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.

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

v10 Upgrade Handler #805

Merged
merged 48 commits into from Jun 14, 2023
Merged

v10 Upgrade Handler #805

merged 48 commits into from Jun 14, 2023

Conversation

sampocs
Copy link
Collaborator

@sampocs sampocs commented May 26, 2023 •
edited

Context and purpose of the change

Upgrade handler for v10 upgrade

Brief Changelog

  • Migrated tendermint consensus params [source]
  • Migrated channel capability from stakeibc → icacontroller [source]
  • Pruned expired tendermint clients [source]
  • Migrated icacallback proto from google proto (deprecated) to cosmos proto
  • Reduced STRD staking rewards. Double check me on these numbers!!
  • Set MinInitialDepositRatio
  • Removed an unnecessary setup call from past unit tests
  • Cleaned up makefile and docs
  • Added localhost client for ibc-go v7.1

@github-actions github-actions bot added C:submodules dependencies Pull requests that update a dependency file labels Jun 8, 2023
@riley-stride
Copy link
Contributor

Should I re-review this @sampocs?

This was referenced Jun 12, 2023
Closed
Closed
@riley-stride riley-stride self-requested a review June 14, 2023 00:03
asalzmann
asalzmann approved these changes Jun 14, 2023
View reviewed changes
dockernet/dockerfiles/Dockerfile.relayer Show resolved Hide resolved
dockernet/upgrades/build_old_binary.sh Show resolved Hide resolved
app/app.go Show resolved Hide resolved
app/upgrades/v10/upgrades.go Outdated Show resolved Hide resolved
app/upgrades/v10/upgrades.go Show resolved Hide resolved
app/upgrades/v10/upgrades.go Show resolved Hide resolved
app/upgrades/v10/upgrades.go Outdated Show resolved Hide resolved
app/upgrades/v10/upgrades.go Show resolved Hide resolved
app/upgrades/v10/upgrades.go Show resolved Hide resolved
app/upgrades/v10/upgrades.go Show resolved Hide resolved
@sampocs sampocs merged commit 5514e47 into main Jun 14, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asalzmann asalzmann asalzmann approved these changes

@riley-stride riley-stride riley-stride approved these changes

@shellvish shellvish Awaiting requested review from shellvish

Assignees
No one assigned
Labels
C:app-wiring C:docs C:records C:stakeibc C:submodules C:test dependencies Pull requests that update a dependency file T:build v10
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@sampocs @riley-stride @asalzmann @shellvish