Skip to content

Conversation

aanm
Copy link
Member

@aanm aanm commented Aug 14, 2025

This test is currently broken, we should skip it to prevent the entire workflow from failing.

@aanm aanm requested review from a team as code owners August 14, 2025 07:16
@aanm aanm requested a review from Artyop August 14, 2025 07:16
@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 Aug 14, 2025
@aanm aanm force-pushed the pr/fix-k8s-test branch from 3169c5d to fa15489 Compare August 14, 2025 07:50
Copy link
Member

@sayboras sayboras left a comment

Choose a reason for hiding this comment

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

Thanks.

Just a question if we want to have one issue to keep track.

@aanm aanm force-pushed the pr/fix-k8s-test branch 2 times, most recently from ec447d8 to 1b75af1 Compare August 14, 2025 08:36
Some tests are currently broken in our CI, so we should skip them to
prevent the entire workflow from failing.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/fix-k8s-test branch from 1b75af1 to c757083 Compare August 14, 2025 09:00
@aanm
Copy link
Member Author

aanm commented Aug 14, 2025

/test

@aanm aanm enabled auto-merge August 14, 2025 09:32
@aanm aanm added area/CI Continuous Integration testing issue or flake area/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. release-note/ci This PR makes changes to the CI. labels Aug 14, 2025
@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 Aug 14, 2025
@aanm aanm added this pull request to the merge queue Aug 14, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Aug 14, 2025
Merged via the queue into main with commit 492bfe8 Aug 14, 2025
68 of 70 checks passed
@aanm aanm deleted the pr/fix-k8s-test branch August 14, 2025 09:49
@joestringer
Copy link
Member

We probably want to file issues to follow up on fixing these failures as well?

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/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. 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.

4 participants