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

x/group does not propagate events correctly #12887

Closed
damiannolan opened this issue Aug 10, 2022 · 0 comments · Fixed by #12888
Closed

x/group does not propagate events correctly #12887

damiannolan opened this issue Aug 10, 2022 · 0 comments · Fixed by #12888
Assignees

Comments

@damiannolan
Copy link
Member

Summary of Bug

The x/group module does not propagate events correctly to the current context.
This affects many ibc associated messages as relayers depend on certain events, for example hermes channel workers detect channel_open_init events and use this to complete the handshake flow.
In particular when testing an interchain-accounts account registration via x/group proposal events are not propagated to the current context in x/group msg_server.go Exec.

Version

v0.46.0

Steps to Reproduce

Some rough notes for reproducing below using this branch

icad tx group create-group $WALLET_1 test-metadata members.json --home ./data/test-1 --from $WALLET_1

icad q group group-info 1 --home ./data/test-1

icad tx group create-group-policy $WALLET_1 1 policy-meta policy.json --home ./data/test-1

icad q group group-policies-by-group 1 --home ./data/test-1
cosmos1afk9zr2hn2jsac63h4hm60vl9z3e5u69gndzf7c99cqge3vzwjzsfwkgpd

icad tx group submit-proposal proposal.json --home ./data/test-1 --from $WALLET_1

icad q group proposal 1 --home ./data/test-1

icad tx group vote 1 $WALLET_1 --home ./data/test-1 VOTE_OPTION_YES meta --from $WALLET_1
icad tx group vote 1 $WALLET_2 --home ./data/test-1 VOTE_OPTION_YES meta --from $WALLET_2

# fund the group policy address
icad tx bank send $WALLET_1 cosmos1afk9zr2hn2jsac63h4hm60vl9z3e5u69gndzf7c99cqge3vzwjzsfwkgpd 10000000stake --home ./data/test-1

icad tx group exec 1 --from $WALLET_1 --home ./data/test-1 --gas 500000 -b block
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant