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: update protonet reset workflow for new infra #2033

Merged
merged 2 commits into from
Oct 17, 2024

Conversation

pirtleshell
Copy link
Member

Description

Updates the reset network flow for protonet. The new flow should be more resilient. It consists of two steps:

  1. stopping the chain services & removing all data
  2. reconfiguring the chain & restarting the services

This should keep instances up to date with any infra enhancements going forward.

No changes to the data seeding have been made. The changes do not affect internal testnet. I will be replacing the infra & upgrading that network's flows next.

I have confirmed both the reset workflow & the existing seed workflow succeed via local workflow testing with act:

# save secrets to a local file, github-action.secrets, then...
act workflow_dispatch --secret-file github-action.secrets -W '.github/workflows/cd-protonet.yml'

Comment on lines +4 to +9
on:
workflow_run:
workflows: [Continuous Integration (Kava Master)]
types:
- completed
workflow_dispatch:
Copy link
Member Author

Choose a reason for hiding this comment

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

this workflow supports being triggered by either:

  1. a successful run of the CI on merge to master
  2. manual dispatch via the github UI

configuring it this way allows us to have a single file instead of two copies with different on conditions

@pirtleshell pirtleshell merged commit f38434c into master Oct 17, 2024
14 checks passed
@pirtleshell pirtleshell deleted the rp-update-reset-protonet-ci branch October 17, 2024 17:39
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