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

GHA: correctly test kvstoremesh in conformance-clustermesh #28434

Merged
merged 2 commits into from
Oct 6, 2023

Conversation

giorio94
Copy link
Member

@giorio94 giorio94 commented Oct 6, 2023

This PR fixes a couple of issues slipped in with #28180.

The kvstoremesh image repository was incorrectly set to
`clustermesh-apiserver-ci`. Let's fix it.

Fixes: 4498ec9 (".github: re-use common helm values from a single action")
Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 requested a review from aanm October 6, 2023 08:08
@giorio94 giorio94 requested review from a team as code owners October 6, 2023 08:08
@giorio94 giorio94 requested a review from tklauser October 6, 2023 08:08
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 6, 2023
@giorio94
Copy link
Member Author

giorio94 commented Oct 6, 2023

/ci-clustermesh

@giorio94 giorio94 added area/CI Continuous Integration testing issue or flake area/clustermesh Relates to multi-cluster routing functionality in Cilium. release-note/ci This PR makes changes to the CI. labels Oct 6, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Oct 6, 2023
This setting has been incorrectly dropped in a previous commit, let's
restore it, so that we keep testing kvstoremesh.

Fixes: 4498ec9 (".github: re-use common helm values from a single action")
Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 force-pushed the pr/giorio94/main/gha-clustermesh-kvstoremesh-fix branch from 69171b5 to 2e6c83e Compare October 6, 2023 08:11
@giorio94
Copy link
Member Author

giorio94 commented Oct 6, 2023

/ci-clustermesh

@giorio94
Copy link
Member Author

giorio94 commented Oct 6, 2023

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 6, 2023
@ti-mo ti-mo merged commit 24053dd into main Oct 6, 2023
214 checks passed
@ti-mo ti-mo deleted the pr/giorio94/main/gha-clustermesh-kvstoremesh-fix branch October 6, 2023 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake area/clustermesh Relates to multi-cluster routing functionality in Cilium. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants