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

Add support for gaiad v5.0.0 on CI #1175

Closed
5 tasks done
andynog opened this issue Jul 12, 2021 · 1 comment · Fixed by #1176
Closed
5 tasks done

Add support for gaiad v5.0.0 on CI #1175

andynog opened this issue Jul 12, 2021 · 1 comment · Fixed by #1176
Assignees
@andynog
Labels
I: CI/CD Internal: related to CI/CD pipeline O: tests Objective: Test more aspect of the relayer
Milestone
08.2021

Comments

@andynog
Copy link
Contributor

andynog commented Jul 12, 2021

Crate

CI/CD

Summary

Cosmos Hub will upgrade soon to Gaia v5.0.0. Need to add support for this version in the CI pipeline.

Problem Definition

Need end-to-end (e2e) tests to support Gaia v5.0.0

Proposal

Add gaiad v5.0.0 to CI and upload the Docker images to the registry.

Acceptance Criteria

e2e runs on gaia v5.0.0

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@andynog andynog added O: tests Objective: Test more aspect of the relayer I: CI/CD Internal: related to CI/CD pipeline labels Jul 12, 2021
@andynog andynog added this to the 07.2021 milestone Jul 12, 2021
@andynog andynog self-assigned this Jul 12, 2021
andynog added a commit that referenced this issue Jul 12, 2021
@andynog
Adding gaiad v5.0.0 support to CI e2e (#1175)
7f5fd0f
@andynog andynog mentioned this issue Jul 12, 2021
Merged
5 tasks
andynog added a commit that referenced this issue Jul 12, 2021
@andynog
Fix gaiad version for relayer service (#1175)
bf26c74
andynog added a commit that referenced this issue Jul 13, 2021
@andynog
Adding additional log output for relayer container (#1175)
18479b3
@andynog
Copy link
Contributor Author

andynog commented Jul 13, 2021

It's still failing e2e tests on CI. I've done some local testing and I could reproduce the error that is showing in Github Actions but I don't know the root cause yet. I've ran the docker containers (ibc-0, ibc-1, relayer) locally but the e2e fails with same error.

If I run the dev-env script to start local gaiad process in my machine using a local v5.0.0 gaiad binary I can execute the first command (update-client) locally so I don't think it's an issue with v5.0.0 but maybe some docker configuration.

Still troubleshooting.

@adizere adizere modified the milestones: 07.2021, 08.2021 Aug 3, 2021
andynog added a commit that referenced this issue Aug 6, 2021
@andynog
Upgrade to gaia v5.0.5 (#1175)
fcbc6e5
adizere added a commit that referenced this issue Aug 10, 2021
@andynog @romac @adizere
Adding gaiad v5.0.5 support on CI (#1176)
2122c86 
* Adding gaiad v5.0.0 support to CI e2e (#1175)

* Fix gaiad version for relayer service (#1175)

* Adding additional log output for relayer container (#1175)

* Upgrade to gaia v5.0.5 (#1175)

* Removing gaia v5.0.0 files (#1176)

* add changelog

Co-authored-by: Romain Ruetschi <romain@informal.systems>
Co-authored-by: Adi Seredinschi <adi@informal.systems>
hu55a1n1 pushed a commit to hu55a1n1/hermes that referenced this issue Sep 13, 2022
@andynog @romac @adizere
Adding gaiad v5.0.5 support on CI (informalsystems#1176)
299c660 
* Adding gaiad v5.0.0 support to CI e2e (informalsystems#1175)

* Fix gaiad version for relayer service (informalsystems#1175)

* Adding additional log output for relayer container (informalsystems#1175)

* Upgrade to gaia v5.0.5 (informalsystems#1175)

* Removing gaia v5.0.0 files (informalsystems#1176)

* add changelog

Co-authored-by: Romain Ruetschi <romain@informal.systems>
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
Assignees

@andynog andynog

Labels
I: CI/CD Internal: related to CI/CD pipeline O: tests Objective: Test more aspect of the relayer
Projects
None yet
Milestone
08.2021
Development

Successfully merging a pull request may close this issue.

Adding gaiad v5.0.5 support on CI informalsystems/hermes
2 participants
@adizere @andynog