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

feat: update misbehaviour handling using IBC-Go 7 #1401

Merged
merged 13 commits into from
Nov 15, 2023
Merged

feat: update misbehaviour handling using IBC-Go 7 #1401

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
3ce01f2
nit fix
sainoe Nov 9, 2023
99423b0
doc
sainoe Nov 9, 2023
d55d3f0
improve doc
sainoe Nov 9, 2023
e75013c
super picky nit
sainoe Nov 9, 2023
291fbf7
Merge branch 'feat/ics-misbehaviour-handling' into sainoe/ibc7-misb-c…
sainoe Nov 14, 2023
a8bb748
fix!: drop nil votes in misbehaviour handling (#1404)
sainoe Nov 10, 2023
fda0b0f
fix comments
sainoe Nov 14, 2023
feac2ad
update changelog
sainoe Nov 15, 2023
09a2a38
Update x/ccv/provider/keeper/misbehaviour.go
sainoe Nov 15, 2023
6f44e0e
improve tests
sainoe Nov 15, 2023
30d31f9
lint
sainoe Nov 15, 2023
7374355
udpate traces
sainoe Nov 15, 2023
db94dc3
silly bug
sainoe Nov 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions ...eatures/provider/1340-add-cryptographic-verification-of-equivocation-feature.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
- Add the cryptographic verification of equivocation feature to the provider
(cf. ADR-005 & ADR-013). ([\#1340](https://github.com/cosmos/interchain-
security/pull/1340))
3 changes: 3 additions & 0 deletions ...reaking/provider/1340-add-cryptographic-verification-of-equivocation-feature.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
- Add the cryptographic verification of equivocation feature to the provider
(cf. ADR-005 & ADR-013). ([\#1340](https://github.com/cosmos/interchain-
security/pull/1340))
39 changes: 13 additions & 26 deletions tests/e2e/tracehandler_testdata/consumer-double-sign.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -72,8 +71,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": {
Expand Down Expand Up @@ -117,8 +115,7 @@
"ProviderKeys": {
"carol": "cosmosvalcons1ezyrq65s3gshhx5585w6mpusq3xsj3ayzf4uv6"
},
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -162,8 +159,7 @@
"ProviderKeys": {
"carol": "cosmosvalcons1ezyrq65s3gshhx5585w6mpusq3xsj3ayzf4uv6"
},
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -199,8 +195,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": {
Expand Down Expand Up @@ -258,8 +253,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -277,8 +271,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -328,8 +321,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -351,8 +343,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -378,8 +369,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -401,8 +391,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -431,8 +420,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -454,8 +442,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down
39 changes: 13 additions & 26 deletions tests/e2e/tracehandler_testdata/consumer-misbehaviour.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -66,8 +65,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": {
Expand Down Expand Up @@ -111,8 +109,7 @@
"ProviderKeys": {
"alice": "cosmosvalcons1qmq08eruchr5sf5s3rwz7djpr5a25f7xw4mceq"
},
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -146,8 +143,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": {
Expand Down Expand Up @@ -193,8 +189,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -211,8 +206,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -263,8 +257,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -281,8 +274,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -310,8 +302,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -351,8 +342,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -372,8 +362,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand Down Expand Up @@ -401,8 +390,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": null,
"Proposals": null
Expand All @@ -422,8 +410,7 @@
"ConsumerChains": null,
"AssignedKeys": null,
"ProviderKeys": null,
"ConsumerChainQueueSizes": null,
"GlobalSlashQueueSize": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
"ClientsFrozenHeights": {
"07-tendermint-0": {
Expand Down
3 changes: 0 additions & 3 deletions tests/e2e/tracehandler_testdata/democracy.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -610,10 +610,7 @@
"ProviderKeys": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
<<<<<<< HEAD
"ClientsFrozenHeights": null,
=======
>>>>>>> main
"Proposals": {
"1": {
"RawProposal": {
Expand Down
3 changes: 0 additions & 3 deletions tests/e2e/tracehandler_testdata/democracyRewardsSteps.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -610,10 +610,7 @@
"ProviderKeys": null,
"ConsumerPendingPacketQueueSize": null,
"RegisteredConsumerRewardDenoms": null,
<<<<<<< HEAD
"ClientsFrozenHeights": null,
=======
>>>>>>> main
"Proposals": {
"1": {
"RawProposal": {
Expand Down
Loading
Loading