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(zombienet): docker img version to use in merge queues for bridges #3337

Merged
merged 3 commits into from
Feb 16, 2024

Conversation

pepoviola
Copy link
Contributor

Fix image tag to use in merge queues tests, currently is failing because is using the wrong tag (e.g https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/5224342).

Thx!

@pepoviola pepoviola added R0-silent Changes should not be mentioned in any release notes T10-tests This PR/Issue is related to tests. T15-bridges This PR/Issue is related to bridges. labels Feb 15, 2024
@pepoviola pepoviola requested a review from a team as a code owner February 15, 2024 12:27
Copy link
Contributor

@serban300 serban300 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this ! Sorry, I was thinking it wouldn't work in before_script since we use DOCKER_IMAGES_VERSION as part of the image name.

Comment on lines 9 to 11
rules:
- !reference [.build-refs, rules]
- if: $CI_COMMIT_REF_NAME =~ /^gh-readonly-queue.*$/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should delete this if the variable assignment has been moved.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should keep, since this rule make these test only run on merge queues.

Copy link
Contributor

@serban300 serban300 Feb 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These tests should also run in the CI. And they seem to be running. If this rule makes them only run in merge queues it's by mistake.

@pepoviola
Copy link
Contributor Author

Thanks for fixing this ! Sorry, I was thinking it wouldn't work in before_script since we use DOCKER_IMAGES_VERSION as part of the image name.

Ahh, that true. We should export the final var here. I will fix it.

@serban300
Copy link
Contributor

@pepoviola I see that there are still some errors. Are you working on them ? If not I can try to fix them since it's a bridge testing error. Please let me know how you prefer.

@pepoviola
Copy link
Contributor Author

pepoviola commented Feb 16, 2024

Hi @serban300, thanks for the heads up. Let me check and fix it.
Thx!


Update: Now is working
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/5235654
https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/5235655

@pepoviola pepoviola added this pull request to the merge queue Feb 16, 2024
Merged via the queue into master with commit 25988be Feb 16, 2024
129 of 130 checks passed
@pepoviola pepoviola deleted the jv-fix-bridges-ci-image branch February 16, 2024 16:25
ordian added a commit that referenced this pull request Feb 19, 2024
* master: (41 commits)
  Add Coretime to Westend (#3319)
  removed `pallet::getter` from `pallet-sudo` (#3370)
  gossip-support: add unittests for update authorities (#3258)
  [FRAME Core] remove unnecessary overrides while using derive_impl for frame_system (#3317)
  Update coretime-westend bootnodes (#3380)
  `im-online` removal cleanup: remove off-chain storage (#2290)
  Bump the known_good_semver group with 1 update (#3379)
  Fix documentation dead link (#3372)
  Make `sp-keystore` `no_std`-compatible and fix the `build-runtimes-polkavm` CI job (#3363)
  Remove unused `im-online` weights (#3373)
  Ensure referenda `TracksInfo` is sorted (#3325)
  rpc server: add rate limiting middleware (#3301)
  do not block finality for "disabled" disputes (#3358)
  fix(zombienet): docker `img` version to use in merge queues for bridges (#3337)
  Various nits and alignments for SP testnets found during bumping `polkadot-fellows` repo (#3359)
  Add broker pallet to `coretime-westend` (#3272)
  remove recursion limit (#3348)
  Update subkey README.md (#3355)
  Bump the known_good_semver group with 6 updates (#3347)
  Document LocalKeystore insert method (#3336)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T10-tests This PR/Issue is related to tests. T15-bridges This PR/Issue is related to bridges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants