Skip to content

Conversation

aanm
Copy link
Member

@aanm aanm commented Jul 27, 2023

No description provided.

@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Jul 27, 2023
@aanm aanm requested review from a team as code owners July 27, 2023 14:41
@aanm aanm requested review from christarazi and bimmlerd July 27, 2023 14:41
@aanm
Copy link
Member Author

aanm commented Jul 27, 2023

/test

@christarazi christarazi added area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow release-note/ci This PR makes changes to the CI. and removed release-note/misc This PR makes changes that have no direct user impact. labels Jul 27, 2023
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/update-releases branch from c827ff8 to 400745a Compare July 27, 2023 22:08
@aanm aanm requested a review from a team as a code owner July 27, 2023 22:08
@aanm
Copy link
Member Author

aanm commented Jul 27, 2023

/test

@aanm aanm requested a review from pchaigno July 27, 2023 22:09
@pchaigno
Copy link
Member

Any specific reason why I was requested a review?

@@ -243,13 +243,6 @@ func InstallAndValidateCiliumUpgrades(kubectl *helpers.Kubectl, oldHelmChartVers
"hubble.relay.image.repository": "quay.io/cilium/hubble-relay-ci",
"clustermesh.apiserver.image.repository": "quay.io/cilium/clustermesh-apiserver-ci",
}
if helpers.RunsOn419OrLaterKernel() {
Copy link
Member Author

Choose a reason for hiding this comment

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

@pchaigno because you added this. I wanted to make sure there's nothing left.

@aanm aanm merged commit 2e294ac into main Jul 28, 2023
@aanm aanm deleted the pr/update-releases branch July 28, 2023 22:33
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/CI-improvement Topic or proposal to improve the Continuous Integration workflow release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants