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

ci-e2e-upgrade: Make it stable #31895

Merged
merged 5 commits into from
Apr 11, 2024
Merged

ci-e2e-upgrade: Make it stable #31895

merged 5 commits into from
Apr 11, 2024

Conversation

brb
Copy link
Member

@brb brb commented Apr 11, 2024

@maintainer-s-little-helper
Copy link

Commit 6ac07d9 does not match "(?m)^Signed-off-by:".

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

@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 Apr 11, 2024
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

1 similar comment
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

@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 Apr 11, 2024
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

1 similar comment
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

brb added 2 commits April 11, 2024 13:22
This should resolve the flaky north-south-loadbalancing test [1]. See
doc comment for more details.

[1]: #31343

Signed-off-by: Martynas Pumputis <m@lambda.lt>
Until #31622 has been released.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

Get rid of things which are no longer relevant and have been removed in
other workflows.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

Until #30717 has been resolved.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/ci-e2e-upgrade

@brb brb changed the title WIP: ci-e2e-upgrade ci-e2e-upgrade: Make it stable Apr 11, 2024
It became stable enough.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Apr 11, 2024
@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 Apr 11, 2024
@brb
Copy link
Member Author

brb commented Apr 11, 2024

/test

@brb brb marked this pull request as ready for review April 11, 2024 15:16
@brb brb requested review from a team as code owners April 11, 2024 15:16
@brb brb requested a review from brlbil April 11, 2024 15:16
@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 Apr 11, 2024
@joestringer joestringer added this pull request to the merge queue Apr 11, 2024
Merged via the queue into main with commit 28fe12d Apr 11, 2024
263 checks passed
@joestringer joestringer deleted the pr/brb/ci-e2e-upgrade branch April 11, 2024 21:30
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 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

3 participants