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

Upgrade protos, upgrade gaiad in CI [Vega upgrade] #1408

Closed
7 tasks done
Tracked by #1418
adizere opened this issue Oct 4, 2021 · 2 comments · Fixed by #1438 or #1557
Closed
7 tasks done
Tracked by #1418

Upgrade protos, upgrade gaiad in CI [Vega upgrade] #1408

adizere opened this issue Oct 4, 2021 · 2 comments · Fixed by #1438 or #1557
Assignees
Labels
I: dependencies Internal: related to dependencies
Milestone

Comments

@adizere
Copy link
Member

adizere commented Oct 4, 2021

Crate

Summary

Gaiad RC for 6.0.0 is out https://github.com/cosmos/gaia/releases/tag/v6.0.0-rc2

This is the first RC for Vega update, which will go live on the hub soon (should have been in Q3 2021).
https://github.com/cosmos/gaia/blob/main/docs/roadmap/cosmos-hub-roadmap-2.0.md#vega-upgrade-expected-q3-2021

Proposal

We should prepare for 6.0.0:


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@adizere adizere added the I: dependencies Internal: related to dependencies label Oct 4, 2021
@adizere adizere added this to the 11.2021 milestone Oct 4, 2021
@adizere adizere changed the title Upgrade protos, upgrade gaiad in CI Upgrade protos, upgrade gaiad in CI [Vega upgrade] Oct 5, 2021
@ancazamfir ancazamfir self-assigned this Oct 29, 2021
@romac romac added the P-high label Nov 2, 2021
romac added a commit that referenced this issue Nov 8, 2021
romac added a commit that referenced this issue Nov 8, 2021
* Fix broken link in .changelog folder

* Fix changelog entry for #1408
andynog added a commit that referenced this issue Nov 9, 2021
@andynog
Copy link
Contributor

andynog commented Nov 9, 2021

Running the e2e test locally against gaia v6.0.0-rc2, the relayer test fails with:

Not sure yet if it's the e2e setup or something with the gaia future release v6.0.0, but the running the e2e tests locally against v5.0.8 (current release) doesn't fail.

{"timestamp":"2021-11-09T20:11:36.075264Z","level":"TRACE","fields":{"message":"[ibc-0 -> ibc-1:07-tendermint-1] finished misbehaviour verification after 5.074784876s"},"threadId":"ThreadId(22)"}
{"timestamp":"2021-11-09T20:11:36.130906Z","level":"TRACE","fields":{"message":"light client verification","trusted":"1-85","target":"1-96"},"threadId":"ThreadId(19)"}
{"timestamp":"2021-11-09T20:11:36.376720Z","level":"TRACE","fields":{"message":"light client verification","trusted":"1-78","target":"1-85"},"threadId":"ThreadId(19)"}
{"timestamp":"2021-11-09T20:11:36.624447Z","level":"TRACE","fields":{"message":"light client verification","trusted":"1-64","target":"1-78"},"threadId":"ThreadId(19)"}
{"timestamp":"2021-11-09T20:11:36.876632Z","level":"TRACE","fields":{"message":"light client verification","trusted":"1-61","target":"1-64"},"threadId":"ThreadId(19)"}
{"timestamp":"2021-11-09T20:11:37.105728Z","level":"TRACE","fields":{"message":"[ibc-1 -> ibc-0:07-tendermint-0] finished misbehaviour verification after 6.293314424s"},"threadId":"ThreadId(20)"}
2021-11-09 20:11:45 [   DEBUG] /usr/bin/hermes -c /relayer/simple_config.toml --json query channel end ibc-1 transfer channel-2
2021-11-09 20:11:45 [   DEBUG] {"result":{"connection_hops":["connection-1"],"ordering":"Unordered","remote":{"channel_id":null,"port_id":"transfer"},"state":"Init","version":"ics20-1"},"status":"success"}
2021-11-09 20:11:45 [   DEBUG] ChannelEnd(connection_hops=['connection-1'], ordering='Unordered', remote=Remote(channel_id=None, port_id='transfer'), state='Init', version='ics20-1')
2021-11-09 20:11:45 [   DEBUG] Status of channel end channel-2: ChannelEnd(connection_hops=['connection-1'], ordering='Unordered', remote=Remote(channel_id=None, port_id='transfer'), state='Init', version='ics20-1')

Traceback (most recent call last):
  File "/relayer/e2e/run.py", line 276, in <module>
    main()
  File "/relayer/e2e/run.py", line 262, in main
    channel.passive_channel_start_then_init(
  File "/relayer/e2e/e2e/channel.py", line 590, in passive_channel_start_then_init
    verify_state(c, ibc1, ibc0, ibc1_chan_id, port_id)
  File "/relayer/e2e/e2e/channel.py", line 565, in verify_state
    ibc0_chan_end = query_channel_end(c, ibc0, port_id, ibc0_chan_id)
  File "/relayer/e2e/e2e/channel.py", line 528, in query_channel_end
    res = cmd.run(c).success()
  File "/relayer/e2e/e2e/cmd.py", line 61, in run
    l.debug(' '.join(full_cmd))
TypeError: sequence item 9: expected str instance, NoneType found
```

@uhvyuhs77
Copy link

Sweet

romac pushed a commit that referenced this issue Nov 24, 2021
…future release) (#1557)

* Logic to support e2e tests for current gaia v5.0.8 and future gaia v6.0.0-rc1 #1408

* Rename workflows to reflect e2e current and future gaia #1408

* Remove v6.0.0-rc1 since there is a rc2 #1408

* Updated to gaia v6.0.0-rc2 #1408

* Modify logic to handle gaia v6.0.0 output #1408

* Add version to workflows #1557

* Adding gaia v6.0.0-rc3 to CI (#1557)

* Removed gaia v6.0.0-rc2 from CI (#1557)
adizere added a commit that referenced this issue Nov 25, 2021
* Logic to support e2e tests for current gaia v5.0.8 and future gaia v6.0.0-rc1 #1408

* Rename workflows to reflect e2e current and future gaia #1408

* Remove v6.0.0-rc1 since there is a rc2 #1408

* Updated to gaia v6.0.0-rc2 #1408

* Modify logic to handle gaia v6.0.0 output #1408

* Add version to workflows #1557

* Adding gaia v6.0.0-rc3 to CI (#1557)

* Removed gaia v6.0.0-rc2 from CI (#1557)

* Updating to gaia v6.0.0 (#1557)

* Remove v6.0.0-rc3 (#1557)

* Missing step in ci Readme

* changelog

Co-authored-by: Adi Seredinschi <adi@informal.systems>
hu55a1n1 pushed a commit to hu55a1n1/hermes that referenced this issue Sep 13, 2022
* Fix broken link in .changelog folder

* Fix changelog entry for informalsystems#1408
hu55a1n1 pushed a commit to hu55a1n1/hermes that referenced this issue Sep 13, 2022
…future release) (informalsystems#1557)

* Logic to support e2e tests for current gaia v5.0.8 and future gaia v6.0.0-rc1 informalsystems#1408

* Rename workflows to reflect e2e current and future gaia informalsystems#1408

* Remove v6.0.0-rc1 since there is a rc2 informalsystems#1408

* Updated to gaia v6.0.0-rc2 informalsystems#1408

* Modify logic to handle gaia v6.0.0 output informalsystems#1408

* Add version to workflows informalsystems#1557

* Adding gaia v6.0.0-rc3 to CI (informalsystems#1557)

* Removed gaia v6.0.0-rc2 from CI (informalsystems#1557)
hu55a1n1 pushed a commit to hu55a1n1/hermes that referenced this issue Sep 13, 2022
* Logic to support e2e tests for current gaia v5.0.8 and future gaia v6.0.0-rc1 informalsystems#1408

* Rename workflows to reflect e2e current and future gaia informalsystems#1408

* Remove v6.0.0-rc1 since there is a rc2 informalsystems#1408

* Updated to gaia v6.0.0-rc2 informalsystems#1408

* Modify logic to handle gaia v6.0.0 output informalsystems#1408

* Add version to workflows informalsystems#1557

* Adding gaia v6.0.0-rc3 to CI (informalsystems#1557)

* Removed gaia v6.0.0-rc2 from CI (informalsystems#1557)

* Updating to gaia v6.0.0 (informalsystems#1557)

* Remove v6.0.0-rc3 (informalsystems#1557)

* Missing step in ci Readme

* changelog

Co-authored-by: Adi Seredinschi <adi@informal.systems>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: dependencies Internal: related to dependencies
Projects
None yet
6 participants