+
Skip to content

CD for dynamic development environments #2054

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 8 commits into from
May 26, 2022
Merged

CD for dynamic development environments #2054

merged 8 commits into from
May 26, 2022

Conversation

jgreat
Copy link
Contributor

@jgreat jgreat commented May 25, 2022 •
edited
Loading

Motivation

CI/CD process to test version upgrades in development environments. Previous release -> current version -> block version upgrades.

This is refresh of work that was approved but abandoned in PRs: #1811 #1829 #1848

  1. Deploy 1.1.3
  2. Tests: fog-distribution, mobilecoind, fog-test-client
  3. Deploy 1.2.0 block_version 0
  4. Tests: mobilecoind, fog-test-client
  5. Deploy 1.2.0 block_version 2
  6. Tests: mobilecoind, fog-test-client, mint-config-tx, mint-tx, mobilecoind-json, drain_accounts, fog-test-client token 0, fog-test-client token 1

Summary of changes

  • GitHub Actions workflows
  • Standardize caching in custom actions (reduce copy/pasta)
  • Refresh of helm charts.
  • Refresh of docker images.
  • create utilities and various helpers for environment bootstrap and testing

@jgreat jgreat requested review from a team, MCrank and joekottke May 25, 2022 17:58
@jgreat
Copy link
Contributor Author

jgreat commented May 25, 2022

Note that [skip ci] seems to have overrode the test results, see status on commit b32540f.

jcape
jcape approved these changes May 25, 2022
View reviewed changes
Copy link
Contributor

@jcape jcape left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dockerignore changes LGTM ;-)

cbeck88
cbeck88 approved these changes May 26, 2022
View reviewed changes
Copy link
Contributor

@cbeck88 cbeck88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reviewed the test/*.sh scripts and the util/*.sh scripts, they look good to me

@jgreat jgreat merged commit d2718bc into release/v1.2.0 May 26, 2022
@jgreat jgreat deleted the feature/cd-2-v1.2.0 branch May 26, 2022 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joekottke joekottke Awaiting requested review from joekottke

@MCrank MCrank Awaiting requested review from MCrank

2 more reviewers

@jcape jcape jcape approved these changes

@cbeck88 cbeck88 cbeck88 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

Blockchain Core
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jgreat @jcape @cbeck88
点击 这是indexloc提供的php浏览器服务，不要输入任何密码和下载