Skip to content
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

connectivity: enable outside to nodeport via IPv6 when IPsec is enabled #1581

Merged

Conversation

jschwinger233
Copy link
Member

Enable these test cases as cilium/cilium#23461 has been resolved.

Signed-off-by: Zhichuan Liang gray.liang@isovalent.com

Enable these test cases as cilium/cilium#23461
has been resolved.

Signed-off-by: Zhichuan Liang <gray.liang@isovalent.com>
@jschwinger233 jschwinger233 force-pushed the gray/enable-outside-nodeport-ipsec branch from 779676d to 87970c3 Compare May 8, 2023 04:10
@jschwinger233 jschwinger233 temporarily deployed to ci May 8, 2023 04:11 — with GitHub Actions Inactive
@jschwinger233
Copy link
Member Author

jschwinger233 commented May 8, 2023

Steps to run the affected test cases:

  1. ./contrib/scripts/kind.sh "" 3 "" "" "iptables" "dual"
  2. make kind-image
  3. k apply -f test/k8s/manifests/ipsec_secret.yaml -n kube-system
  4. cilium install --wait --chart-directory=../cilium/install/kubernetes/cilium --helm-set=image.repository=localhost:5000/cilium/cilium-dev --helm-set=image.useDigest=false --helm-set=image.tag=local --helm-set=operator.image.repository=localhost:5000/cilium/operator --helm-set=operator.image.tag=local --helm-set operator.image.suffix= --helm-set=operator.image.useDigest=false --rollback=false --config monitor-aggregation=none --nodes-without-cilium=kind-worker3 --helm-set-string=kubeProxyReplacement=disabled --helm-set-string=tunnel=vxlan --helm-set=ipv6.enabled=true --helm-set=bpf.masquerade=false --helm-set=operator.image.pullPolicy=IfNotPresent --helm-set=image.pullPolicy=IfNotPresent --encryption ipsec
  5. ./cilium connectivity test -v --test north-south-loadbalancing --datapath

@jschwinger233 jschwinger233 marked this pull request as ready for review May 8, 2023 04:15
@jschwinger233 jschwinger233 requested a review from a team as a code owner May 8, 2023 04:15
@jschwinger233 jschwinger233 requested a review from asauber May 8, 2023 04:15
@tklauser tklauser merged commit d598f34 into cilium:main May 8, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants