Skip to content

datapath: Move linuxNodeHandler IPsec functions to their own file #28941

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

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

pchaigno
Copy link
Member

@pchaigno pchaigno commented Nov 2, 2023

This pull request has no functional changes.

It simply moves all the linuxNodeHandler functions that pertain to IPsec to a new file, ipsec.go. This will ease review assignments by ensuring that we don't require an IPsec review on non-IPsec code and vice versa.

Marking for backports everywhere because it should be easy to backport, is low risk, and will help avoid backporting conflicts in the future.

@pchaigno pchaigno added area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. release-note/misc This PR makes changes that have no direct user impact. sig/community Impacts contribution workflow, guidelines, and tools. needs-backport/1.12 labels Nov 2, 2023
@pchaigno pchaigno marked this pull request as ready for review November 2, 2023 11:14
@pchaigno pchaigno requested review from a team as code owners November 2, 2023 11:14
Copy link
Contributor

@gentoo-root gentoo-root left a comment

Choose a reason for hiding this comment

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

Offtopic: git diff HEAD^:pkg/datapath/linux/node.go HEAD:pkg/datapath/linux/ipsec.go behaves really weird. Instead of just showing the removed lines with -, it produces stuff like:

        }
-       return errs
-}
... more lines deleted ...
+       return errs
 }

@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 Nov 3, 2023
@pchaigno pchaigno added dont-merge/bad-bot To prevent MLH from marking ready-to-merge. and removed ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Nov 4, 2023
This commit has no functional changes.

It simply moves all the linuxNodeHandler functions that pertain to IPsec
to a new file, ipsec.go. This will ease review assignments by ensuring
that we don't require an IPsec review on non-IPsec code and vice versa.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno force-pushed the separate-ipsec-files branch from 77a2db6 to dbcf494 Compare November 6, 2023 08:03
@pchaigno pchaigno removed the dont-merge/bad-bot To prevent MLH from marking ready-to-merge. label Nov 6, 2023
@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 Nov 6, 2023
@pchaigno pchaigno merged commit 1900f4a into cilium:main Nov 6, 2023
@pchaigno pchaigno deleted the separate-ipsec-files branch November 6, 2023 11:18
@jibi jibi mentioned this pull request Nov 7, 2023
15 tasks
@jibi jibi added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 labels Nov 7, 2023
@jibi jibi mentioned this pull request Nov 7, 2023
7 tasks
@jibi jibi mentioned this pull request Nov 7, 2023
6 tasks
@github-actions github-actions bot added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.13 backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/community Impacts contribution workflow, guidelines, and tools.
Projects
No open projects
Status: Released
Status: Released
Development

Successfully merging this pull request may close these issues.

4 participants