-
Notifications
You must be signed in to change notification settings - Fork 108
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
chore: upgrade cosmovisor to 1.6.0 #2782
Conversation
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughWalkthroughThe changes made involve updating the Dockerfile for a local network setup by upgrading the version of the Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- Dockerfile-localnet (1 hunks)
Additional comments not posted (1)
Dockerfile-localnet (1)
27-27
: LGTM! But verify compatibility.The version of
cosmovisor
has been correctly updated fromv1.5.0
tov1.6.0
. Ensure that the new version is compatible with the existing infrastructure and does not introduce any breaking changes.The code changes are approved.
Run the following script to verify the compatibility of the new version:
Verification successful
No breaking changes found, but verify configuration handling.
The changelog for
cosmovisor
v1.6.0 does not indicate any breaking changes. However, it includes updates to configuration handling that should be tested to ensure compatibility with your current setup.
- Verify the changes related to the
--cosmovisor-config
flag and configuration file handling in a test environment.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify compatibility of cosmovisor v1.6.0 with existing infrastructure. # Test: Check for any breaking changes or incompatibilities. # Expect: No issues found. # Fetch the changelog for cosmovisor v1.6.0 curl -s https://raw.githubusercontent.com/cosmos/cosmos-sdk/main/tools/cosmovisor/CHANGELOG.md | grep -A 10 "## [v1.6.0]"Length of output: 4606
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2782 +/- ##
===========================================
- Coverage 66.78% 66.61% -0.18%
===========================================
Files 364 369 +5
Lines 20516 20732 +216
===========================================
+ Hits 13702 13810 +108
- Misses 6186 6289 +103
- Partials 628 633 +5 |
Changelog: https://github.com/cosmos/cosmos-sdk/blob/tools/cosmovisor/v1.6.0/tools/cosmovisor/CHANGELOG.md
Closes https://github.com/zeta-chain/infrastructure/issues/1596
Summary by CodeRabbit
cosmovisor
tool to versionv1.6.0
, enhancing the build process and potentially improving runtime behavior.