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

contrib: Remove deb,rpm packaging #23081

Merged
merged 1 commit into from Mar 8, 2023

Conversation

joestringer
Copy link
Member

These have been bitrotting for a while now (the deb version lists
version 0.9.90), and we have no mechanisms for building, deploying or
testing these packages. The primary mechanism for distributing and using
Cilium is via docker images, so let's focus on making those work really
well and discard these. We can always resurrect these in future via git
if we need to.

@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Jan 13, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jan 13, 2023
@github-actions
Copy link

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Feb 12, 2023
These have been bitrotting for a while now (the deb version lists
version 0.9.90), and we have no mechanisms for building, deploying or
testing these packages. The primary mechanism for distributing and using
Cilium is via docker images, so let's focus on making those work really
well and discard these. We can always resurrect these in future via git
if we need to.

Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer
Copy link
Member Author

/test

@github-actions github-actions bot removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Feb 25, 2023
@joestringer joestringer marked this pull request as ready for review March 1, 2023 03:55
@joestringer joestringer requested review from a team as code owners March 1, 2023 03:55
Copy link
Member

@aditighag aditighag left a comment

Choose a reason for hiding this comment

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

The clean-up makes sense to me. I didn't even know we had these as I've always used docker images.

@joestringer joestringer merged commit a91e20a into cilium:master Mar 8, 2023
@joestringer joestringer deleted the submit/remove-bin-packages branch March 8, 2023 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants