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

fix(integration): avoid fail IBC integration tests #4086

Merged
merged 7 commits into from
Apr 18, 2024

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Apr 17, 2024

Description

  • Retry to get the IBC balance if it fails for the IBC integration tests the first time;
  • Split the test from IBC relayer and IBC scaffolded chains;
  • Use a fixed hermes relayer version;

@Pantani Pantani self-assigned this Apr 17, 2024
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs labels Apr 17, 2024
@Pantani Pantani changed the title fix(integration): retry to get the IBC balance if it fails the first time fix(integration): avoid fail IBC integration tests Apr 17, 2024
@Pantani Pantani enabled auto-merge (squash) April 17, 2024 14:08
julienrbrt
julienrbrt previously approved these changes Apr 17, 2024
@Pantani Pantani disabled auto-merge April 17, 2024 23:33
@Pantani Pantani enabled auto-merge (squash) April 17, 2024 23:33
@Pantani Pantani merged commit 9179104 into main Apr 18, 2024
46 checks passed
@Pantani Pantani deleted the fix/retry-ibc-balance-integration-tests branch April 18, 2024 09:21
@julienrbrt
Copy link
Member

@Mergifyio backport release/v28.x.y

Copy link
Contributor

mergify bot commented Apr 25, 2024

backport release/v28.x.y

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Apr 25, 2024
* retry get ibc balance if fails first time for the ibc integration tests

* add changelog

* split ibc relayer and ibc tests

* specify hermes version

* improve ci logs for relayer integration tests

* reset balance output

---------

Co-authored-by: Pantani <Pantani>
Co-authored-by: Julien Robert <julien@rbrt.fr>
(cherry picked from commit 9179104)
julienrbrt added a commit that referenced this pull request Apr 26, 2024
…4109)

* fix(integration): avoid fail IBC integration tests (#4086)

* retry get ibc balance if fails first time for the ibc integration tests

* add changelog

* split ibc relayer and ibc tests

* specify hermes version

* improve ci logs for relayer integration tests

* reset balance output

---------

Co-authored-by: Pantani <Pantani>
Co-authored-by: Julien Robert <julien@rbrt.fr>
(cherry picked from commit 9179104)

* Update changelog.md

---------

Co-authored-by: Danilo Pantani <danpantani@gmail.com>
Co-authored-by: Julien Robert <julien@rbrt.fr>
julienrbrt added a commit that referenced this pull request May 3, 2024
…4109)

* fix(integration): avoid fail IBC integration tests (#4086)

* retry get ibc balance if fails first time for the ibc integration tests

* add changelog

* split ibc relayer and ibc tests

* specify hermes version

* improve ci logs for relayer integration tests

* reset balance output

---------

Co-authored-by: Pantani <Pantani>
Co-authored-by: Julien Robert <julien@rbrt.fr>
(cherry picked from commit 9179104)

* Update changelog.md

---------

Co-authored-by: Danilo Pantani <danpantani@gmail.com>
Co-authored-by: Julien Robert <julien@rbrt.fr>
julienrbrt added a commit that referenced this pull request May 29, 2024
* retry get ibc balance if fails first time for the ibc integration tests

* add changelog

* split ibc relayer and ibc tests

* specify hermes version

* improve ci logs for relayer integration tests

* reset balance output

---------

Co-authored-by: Pantani <Pantani>
Co-authored-by: Julien Robert <julien@rbrt.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:ci CI/CD workflow and automated jobs. component:configs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants