Re-scaffold the project for v0.4.0, removing the Golang control-plane #278
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
shaneutt
added
area/maintenance
size/XXL
k8s-ci-robot
added
cncf-cla: yes
shaneutt
force-pushed
the
shaneutt/v0.4.0-re-scaffolding
branch
from
c153043
to
d47d453
Compare
shaneutt
force-pushed
the
shaneutt/v0.4.0-re-scaffolding
branch
from
d47d453
to
fd1c2d9
Compare
k8s-ci-robot
added
the
do-not-merge/work-in-progress
k8s-ci-robot
removed
the
do-not-merge/work-in-progress
shaneutt
force-pushed
the
shaneutt/v0.4.0-re-scaffolding
branch
from
c3494a7
to
0ebb6f3
Compare
shaneutt
added
the
tide/merge-method-rebase
shaneutt
force-pushed
the
shaneutt/v0.4.0-re-scaffolding
branch
2 times, most recently
from
ee9a290
to
803a7c6
Compare
Signed-off-by: Shane Utt <[email protected]>
The integration with bpfd is quite old, and in fact the project has since been renamed to bpfman. We have an open issue to add a bpfman integration, but in the meantime we can remove the old code. Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
Signed-off-by: Shane Utt <[email protected]>
shaneutt
force-pushed
the
shaneutt/v0.4.0-re-scaffolding
branch
from
803a7c6
to
0e5a589
Compare
aryan9600
approved these changes
Sep 16, 2024
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aryan9600, shaneutt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
shaneutt
added
the
lgtm
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Yep, this PR is huge, sorry about that.
This cleans up the repository in several ways and removes the Golang control-plane in support of the upcoming v0.4.0 Milestone wherein we intend to switch to Rust for the control-plane.
This new foundation is intended to make the repository less confusing for newcomers and provide a bit more focus. It almost definitely breaks a couple of things here and there, however I would recommend we fix that forward rather than spend a lot of time here as the risk is zero for this project.
It does leave the Golang integration tests behind for a bit longer, as these are still useful as we continue to build the Rust control-plane to validate our new code.