Skip to content

Conversation

giorio94
Copy link
Member

@giorio94 giorio94 commented Nov 12, 2024

Once this PR is merged, a GitHub action will update the labels of these PRs:

 35646 35679

[ upstream commit 03bde2b ]

[ backproter's notes: applied the changes to the main Makefile, rather
  than Makefile.kind that did not exist in the v1.15 tree. ]

Simplify locally testing Cilium configured in kvstore mode via the
introduction of a dedicated `kind-kvstore-install-cilium` target,
which takes care of setting up an etcd pod running in host network
(pinned to a control plane node) and configuring Cilium to target it.

The creation and deletion of the etcd pod is delegated to the dedicated
`kind-kvstore-start` and `kind-kvstore-stop` commands.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 added kind/backports This PR provides functionality previously merged into master. backport/1.15 This PR represents a backport for Cilium 1.15.x of a PR that was merged to main. labels Nov 12, 2024
@giorio94
Copy link
Member Author

/ci-e2e-upgrade

@giorio94
Copy link
Member Author

/ci-ipsec-upgrade

@giorio94
Copy link
Member Author

/ci-ipsec-e2e

[ upstream commit 535e7f1 ]

[ backporter's notes: hit multiple conflicts due to different
  surrounding context. Adapted the etcd setup logic to be run
  within the LVH context. Additionally skipped testing kvstore
  in combination with WireGuard, as v1.15 and earlier are
  affected by bugs potentially causing connection disruption
  upon agent restart in this combination (#31985,
  #31979). ]

Modify a few matrix entries of the Cilium E2E Upgrade workflow to
configure Cilium in kvstore mode, to cover this functionality here
as well in addition to the clustermesh workflows. In detail, the
etcd instance is executed as a pod running in host network, which
is setup via the `kind-kvstore-start` makefile target. The matrix
entries are selected trying to cover the most common combinations,
that is native-routing/tunneling, KPR off/on and wireguard off/on,
and avoiding incompatible options (mainly Egress Gateway).

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
[ upstream commit 2056fa9 ]

[ backporter's notes: hit minor conflicts due to different surrounding
  context, adapted as appropriate; additionally configured the matrix
  entries in the individual workflow files, as ipsec/configs.yaml did
  not exist in the v1.15 tree. ]

Modify a few matrix entries of the Conformance IPSec and IPSec Upgrade
workflows to configure Cilium in kvstore mode, to cover this functionality
here as well in addition to the E2E and clustermesh workflows. In detail,
the etcd instance is executed as a pod running in host network, which
is setup via the `kind-kvstore-start` makefile target. The matrix entries
are selected to cover both native routing and tunneling, while avoiding
incompatible options (mainly Egress Gateway and Mutual Auth).

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 force-pushed the pr/v1.15-backport-2024-11-12-10-21 branch from 83c8326 to a248f34 Compare November 12, 2024 11:19
@giorio94
Copy link
Member Author

/test-backport-1.15

@giorio94 giorio94 marked this pull request as ready for review November 12, 2024 12:56
@giorio94 giorio94 requested review from a team as code owners November 12, 2024 12:56
@giorio94 giorio94 requested review from aanm and nebril November 12, 2024 12:56
@sayboras sayboras merged commit e8d3ad6 into v1.15 Nov 12, 2024
253 of 255 checks passed
@sayboras sayboras deleted the pr/v1.15-backport-2024-11-12-10-21 branch November 12, 2024 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.15 This PR represents a backport for Cilium 1.15.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants