Skip to content

test: ginkgo cleanups #39714

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

Merged
merged 3 commits into from
May 28, 2025
Merged

test: ginkgo cleanups #39714

merged 3 commits into from
May 28, 2025

Conversation

julianwiedmann
Copy link
Member

Rip out a whole bunch of dead ginkgo code.

#38538 removed the problematic test.

Fixes: #29715
Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Delete a whole bunch of unused legacy helper code.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Remove some stale version checks for ancient k8s versions. We're only
testing with newer k8s versions by now.

Also remove one explicit check for the current Cilium version. Now that
we're no longer testing downgrades in ginkgo, this check is obviously true.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann requested a review from a team as a code owner May 26, 2025 04:18
@julianwiedmann julianwiedmann added the area/CI Continuous Integration testing issue or flake label May 26, 2025
@julianwiedmann julianwiedmann requested review from a team as code owners May 26, 2025 04:18
@julianwiedmann julianwiedmann added the kind/cleanup This includes no functional changes. label May 26, 2025
@julianwiedmann julianwiedmann added the release-note/ci This PR makes changes to the CI. label May 26, 2025
@julianwiedmann
Copy link
Member Author

/test

@julianwiedmann julianwiedmann enabled auto-merge May 26, 2025 05:03
@julianwiedmann julianwiedmann added this pull request to the merge queue May 28, 2025
Merged via the queue into main with commit c91b864 May 28, 2025
154 checks passed
@julianwiedmann julianwiedmann deleted the pr/jwi/main/ginkgo-cleanups branch May 28, 2025 01:16
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 kind/cleanup This includes no functional changes. 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