Skip to content

pkg/datapath: Remove unused feature maps #13150

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 14, 2020
Merged

pkg/datapath: Remove unused feature maps #13150

merged 1 commit into from
Sep 14, 2020

Conversation

brb
Copy link
Member

@brb brb commented Sep 11, 2020 •
edited
Loading

Remove ipmasq, source range check and session affinity related maps if corresponding features are not enabled.

Fix #11085
Fix #12841
Fix #11267

@brb brb added kind/bug This is a bug in the Cilium logic. pending-review area/daemon Impacts operation of the Cilium daemon. release-note/bug This PR fixes an issue in a previous release of Cilium. labels Sep 11, 2020
@brb brb requested a review from a team September 11, 2020 08:51
@brb brb mentioned this pull request Sep 11, 2020
Merged
@pchaigno
Copy link
Member

test-me-please

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 11, 2020
@aanm aanm added the dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. label Sep 11, 2020
@brb @borkmann
pkg/datapath: Remove unused feature maps
f621976 
Remove ipmasq, source range check and session affinity related maps if
corresponding features are not enabled.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann borkmann force-pushed the pr/brb/rm-lbmap-if-not-used branch from 8735682 to f621976 Compare September 14, 2020 10:06
@borkmann borkmann removed the dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. label Sep 14, 2020
@borkmann
Copy link
Member

test-me-please

@borkmann
Copy link
Member

restart-gke

@borkmann borkmann merged commit f191f9e into master Sep 14, 2020
@borkmann borkmann deleted the pr/brb/rm-lbmap-if-not-used branch September 14, 2020 14:26
@kkourt kkourt mentioned this pull request Sep 16, 2020
Merged
@joestringer joestringer mentioned this pull request Oct 1, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@borkmann borkmann borkmann approved these changes

@joestringer joestringer joestringer approved these changes

@pchaigno pchaigno pchaigno approved these changes

Assignees
No one assigned
Labels
area/daemon Impacts operation of the Cilium daemon. kind/bug This is a bug in the Cilium logic. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@brb @pchaigno @borkmann @joestringer @kkourt @aanm