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

Revert "gh/workflow: Run GKE workflow in matrix strategy" #25464

Merged

Conversation

thorn3r
Copy link
Contributor

@thorn3r thorn3r commented May 15, 2023

Reverts #25364 due to CI failures:

The network "default" does not have available private IP space in 10.0.0.0/9

https://github.com/cilium/cilium/actions/runs/4983577990/jobs/8920912559
https://github.com/cilium/cilium/actions/runs/4983089307
https://github.com/cilium/cilium/actions/runs/4983592830
https://github.com/cilium/cilium/actions/runs/4983122154

@nbusseneau mentioned we will need to create ad-hoc subnetworks to prevent this

@thorn3r thorn3r requested review from a team as code owners May 15, 2023 19:31
@thorn3r thorn3r requested a review from aanm May 15, 2023 19:31
@maintainer-s-little-helper
Copy link

Commit 9c152689ba703b8cf1ce2966bfd2a9ea7ee9b296 does not contain "Signed-off-by".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@thorn3r thorn3r requested a review from tklauser May 15, 2023 19:31
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels May 15, 2023
This reverts commit 4a92f1a.

Signed-off-by: Tim Horner <timothy.horner@isovalent.com>
@thorn3r thorn3r force-pushed the revert-25364-pr/brlbil/ci-run-gke-conformance-in-parallel branch from 9c15268 to 534855d Compare May 15, 2023 19:52
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. label May 15, 2023
@thorn3r
Copy link
Contributor Author

thorn3r commented May 15, 2023

force-pushed to add sign-off

@joestringer joestringer added the release-note/ci This PR makes changes to the CI. label May 15, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label May 15, 2023
@joestringer
Copy link
Member

I see this was suggested by @nbusseneau on Slack, and it seems like it should help to stabilize CI for this job. Seems safe since we were previously running successfully with the earlier version. Given that this is a revert for a PR that we believe is breaking CI, I think this is reasonable to merge as-is without further testing or review. CC @brlbil @viktor-kurchenko

@joestringer joestringer merged commit a581786 into main May 15, 2023
45 checks passed
@joestringer joestringer deleted the revert-25364-pr/brlbil/ci-run-gke-conformance-in-parallel branch May 15, 2023 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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

2 participants