Skip to content
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.

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

ci: separate deploy logic into charts #3033

Merged
merged 1 commit into from
Sep 15, 2023
Merged

ci: separate deploy logic into charts #3033

merged 1 commit into from
Sep 15, 2023

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented Sep 15, 2023

This PR breaks up the catch-all deploy logic in the existing single helm chart into:

  • network genesis with initial validators
  • fullnode deployments via join
  • metrics scraping pd & tm

So far, it's been tested extensively in a devnet configuration. Next, will convert preview, and then finally testnet.

Towards #2983.

@conorsch conorsch temporarily deployed to smoke-test September 15, 2023 00:10 — with GitHub Actions Inactive
This PR breaks up the catch-all deploy logic in the existing single helm chart into:

  * network genesis with initial validators
  * fullnode deployments via join
  * metrics scraping pd & tm

So far, it's been tested extensively in a devnet configuration. Next,
will convert preview, and then finally testnet.
@conorsch conorsch temporarily deployed to smoke-test September 15, 2023 22:27 — with GitHub Actions Inactive
@conorsch conorsch marked this pull request as ready for review September 15, 2023 22:36
@conorsch conorsch merged commit 887ce02 into main Sep 15, 2023
8 checks passed
@conorsch conorsch deleted the penumbra-helm-charts branch September 15, 2023 22:46
conorsch added a commit that referenced this pull request Sep 15, 2023
Forgot the new binary for applying multiple helm manifests
in parallel. Follow-up to #3033.
conorsch added a commit that referenced this pull request Sep 15, 2023
Pushing to CI to evaluate fixes is not my favorite workflow, but here we
are. Follow-up to #3033.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant