Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

revert htb changes due to ongoing issues and missing maintainer #1105

Merged
merged 4 commits into from
Oct 14, 2024

Conversation

h0nIg
Copy link
Contributor

@h0nIg h0nIg commented Oct 11, 2024

see #1097

not required to adjust documentation, since HTB PR was never merged: containernetworking/cni.dev#130

@h0nIg h0nIg force-pushed the patch-2 branch 4 times, most recently from 16ae1a5 to 79649a1 Compare October 11, 2024 10:28
@h0nIg h0nIg marked this pull request as ready for review October 11, 2024 10:29
@h0nIg
Copy link
Contributor Author

h0nIg commented Oct 11, 2024

@squeed ping, merge-able

h0nIg added 3 commits October 11, 2024 13:14
Signed-off-by: h0nIg <h0nIg@users.noreply.github.com>
…clude_subnets_from_traffic_shapping2"

This reverts commit ef076af, reversing
changes made to 5974089.

Signed-off-by: h0nIg <h0nIg@users.noreply.github.com>
Signed-off-by: h0nIg <h0nIg@users.noreply.github.com>
Signed-off-by: h0nIg <h0nIg@users.noreply.github.com>
Copy link
Contributor

@s1061123 s1061123 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you so much for your effort, @h0nIg !

@squeed squeed merged commit c52e02b into containernetworking:main Oct 14, 2024
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants