Skip to content

Conversation

tklauser
Copy link
Member

Remove various leftovers of vagrant-based k8s e2e tests. These were moved to LVH-based testing. See individual commit messages for details.

tklauser added 3 commits May 21, 2025 13:46
This target was only useful for vagrant VM based development/testing.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
The last remaining user of this script was removed in commit
306b022 ("drop vagrant and Jenkins references").

Signed-off-by: Tobias Klauser <tobias@cilium.io>
This was used to run cilium-operator as a systemd service in vagrant VM
based tests. We now only use systemd services in the runtime tests but
these don't run cilium-operator, so the service file is obsolete and can
be removed.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser requested review from a team as code owners May 21, 2025 11:49
@tklauser tklauser added kind/cleanup This includes no functional changes. release-note/ci This PR makes changes to the CI. labels May 21, 2025
@tklauser tklauser requested review from Artyop, aditighag and borkmann May 21, 2025 11:49
@tklauser
Copy link
Member Author

/test

@tklauser tklauser enabled auto-merge May 21, 2025 11:49
The k8s based tests were moved to other testing infrastructure and the
remaining runtime tests don't need to build these images. Thus the
respective provisioning logic is obsolete and can be removed.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser force-pushed the pr/tklauser/vagrant-cleanup branch from 39e0e1c to 52384ce Compare May 21, 2025 12:12
@tklauser
Copy link
Member Author

/test

@tklauser tklauser requested a review from aanm May 23, 2025 12:39
@tklauser tklauser added this pull request to the merge queue May 26, 2025
Merged via the queue into main with commit 25f8447 May 26, 2025
295 of 305 checks passed
@tklauser tklauser deleted the pr/tklauser/vagrant-cleanup branch May 26, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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