-
Notifications
You must be signed in to change notification settings - Fork 45
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
Axelar with main merged, fixes, upgrade updates #236
Merged
Merged
Changes from all commits
Commits
Show all changes
127 commits
Select commit
Hold shift + click to select a range
355b126
pin goreleaser to 1.9.2
EricBolten 318442d
add tendermint to grpc gateway
EricBolten 7e2062d
v5 incentives/gravity rc1 (#201)
EricBolten d3f379b
v5 -> v6
EricBolten b2efac9
move to github runners, go to 1.19, mod tidy
EricBolten f511ddd
cosmos-sdk 0.45.13, ibc-go 3.4.0
EricBolten 8de2a87
fix proto go packages to v6
EricBolten 1e0f664
v6 upgrade
EricBolten a169e7d
Update linter for go 1.19
EricBolten cde9b7c
Appease linter
EricBolten 54f4e05
clean build
mvid e726d40
bring over axelar module
mvid d190bfb
redo axelar corks with cork v2
mvid e93514c
rpc endpoints for hitting axelar
mvid bd81a11
cleanup for CLI
mvid 83cde22
clean up CLI
mvid 61f2ac0
sdk down to 0.45.10, tendermint to 0.34.22
EricBolten 94f3b17
golang back down to 1.18
EricBolten c41ba6a
Update v6 upgrade README
EricBolten ec2739e
proposal trasnfered from gravity
mvid 927c250
typo
mvid a885feb
update for heighliner
mvid f2877b3
genesis updates
mvid 465c4c2
governance
mvid ee5fe9d
rename msg field
mvid a7bacc3
some review updates
mvid be1bffb
more review updates
mvid 530fe58
even more review updates!
mvid 9e18aad
proposal updates
mvid 4cc1105
chain configs query
mvid 29fe77e
comment updates
mvid 0eeb0b2
fix issue with ID in key parsing
mvid b438612
add staking mocks
mvid c138859
all existing cork v2 tests pass
mvid 1691acd
add mocks to sc
mvid 4af566d
tests for middleware
mvid c19ea7c
empty payload
mvid 05470ad
prost build
mvid 200ff49
bring in changes from cork v2
mvid 85cce47
actual prost build
mvid d49ed24
include modules
mvid 2069278
namespace issue
mvid 050d855
unique gov prop ids
mvid 2514b01
unused interchain test
mvid 1d61e92
go mod conflict
mvid 51dff16
move scoped axelar keeper
mvid c22b34a
issue with params key table
mvid 90db919
app.go fixes for axelar cork
mvid 06340a7
something is funky with codec registrations
mvid 5bab5af
more prop types
mvid 6fb0c77
all proposals
mvid 494d6d1
unit test pass
mvid 54f88ae
clarify all registered types
mvid 0695e31
newer protos broken for axelar cork
mvid b320c7d
downgrade bufbuild
mvid 75febe0
old version cellar fees
mvid c0f091a
Bolten/test axelar buf (#220)
mvid e339a62
rebuild prost from new (old) buf build
mvid 5e0b04e
compiles
mvid abb11d6
params unit tests
mvid d56e88c
lint pass
mvid cae2365
prost build
mvid 0a5659c
validate disabled
mvid f5a3861
review pass
mvid f2c80e6
only chain id
mvid c047631
more updates
mvid ef1cc7f
import gravity module to pull delegate information
mvid 33dc46b
adjust mocks
mvid ac62ab5
set protocgen back to previous
mvid 86ae9dd
linting
mvid 3757e0e
test configuration correct for scenario
mvid 8f5d968
return proto update to previous style
mvid fe27312
resetting swagger script to previous
mvid c021d9f
another pass
mvid 46ec04f
more changes
mvid 3dc9011
cleanup local script changes for protos
mvid 5a7b6d8
encode id to string on result retrieval
mvid 40bb3c0
revert cellarfee protos
mvid 30d1901
test update for hex
mvid 5fb45e9
cork timeouts
mvid c19a92c
updates for unused cancel msg
mvid 087ca62
several review comments
mvid 6ede3f7
Add AxelarContractCallNonce types and keeper methods
cbrit ad59ab4
Update tests
cbrit be7b58d
Update protos, generate docs
cbrit 83c5456
Add deadline to AxelarCork state and keeper methods
cbrit 2db0718
Add deadline and noce to RelayCork payload
cbrit d213dcf
Initial axelar proxy upgrade types and logic
cbrit 88d230a
Unit tests and rename encoding method
cbrit 1b52a60
Fix proto errors
cbrit 0b3ea21
Make linter happy
cbrit 660b1a6
Upgrade data iterator
cbrit 3ec15dc
Init/ExportGenesis for upgrade data
cbrit 71f0fd8
Tweak keeper comments
cbrit 06c691b
More accurate test name
cbrit 62d975f
Tweak tests
cbrit 6dc13c2
Clarify proxy encoding arg name
cbrit b94c72d
Revert "Clarify proxy encoding arg name"
cbrit ed59e54
CLI tx commands for new proposals
cbrit 6c1bbd5
Proposal boilerplate, queries, CLI, fixes
cbrit 4c25eee
Wire up handlers
cbrit 80d09c4
Improved Axelar call encoding/decoding, add unit test
cbrit 588a286
Add proxy upgrade to IBC packet validation
cbrit 5a845c8
Rename packet validation method
cbrit ba5f880
Add missing validation for upgrade packet
cbrit 7cb8b81
Remove unneeded comment
cbrit 41c68e7
Add new axelar proposal handlers to BasicManager
cbrit bf0fd69
Add message type constant for relay upgrade message
cbrit 9aadba3
Remove unused struct
cbrit d978004
Register missing messages with legacy amino codec
cbrit 01a36af
Add ValidateBasic() for new state and add to default genesis
cbrit a778abf
Update somm_proto crate
cbrit 7965628
Add new query commands to CLI
cbrit e34eb14
Add new tx command to CLI
cbrit 8a40d0f
Lock somm_proto crate to steward compatible version
cbrit d5a0c85
Update Cellar IDs query and add QueryCellarIDsByChainID
cbrit d42d457
Remove duplicate chainID argument from axelar cork ID hash
cbrit b7c4338
Updates
EricBolten 0c2f8d4
Merge branch 'main' into bolten/v7-rc1
EricBolten 5192df4
Update subscribers
EricBolten 5a5ebaa
Update cellars in v7 upgrade
EricBolten fd4fbd0
Update token prices in v7 upgrade
EricBolten 49c5418
Update sizes and Seven Seas domain
EricBolten 17d5500
lint fixes
EricBolten 206158f
Account keeper mocks for Axelar
EricBolten 50656ab
Remove extraneous int parsing tests in auction CLI
EricBolten 9739f67
Delete the v4 upgrade test
EricBolten File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,7 +8,7 @@ linters: | |
enable: | ||
- bodyclose | ||
- deadcode | ||
- depguard | ||
# - depguard | ||
- dogsled | ||
# - errcheck | ||
- goconst | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Probably fine for this PR but I know at least this validator has dropped from the active set. Would it be worth culling inactive validators from this list? I can go through the list if you need me to.
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.
I double-checked all the validators in the upgrade against steward-registry and left in the inactive ones, just to have this mirror the current state we're working with. If you want to spend the time to verify which ones we should remove, go for it, although there isn't really a method for scrubbing dead subscribers post-upgrade unless they delete their subscriber themselves, I don't think. So there will likely be cruft no matter what.
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.
Ah, probably needed then