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

v1.13 Backports 2023-12-14 [Author Submitted] #29869

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

jrfastab
Copy link
Contributor

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

29801

@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.13 This PR represents a backport for Cilium 1.13.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Dec 13, 2023
@jrfastab jrfastab marked this pull request as ready for review December 13, 2023 20:53
@jrfastab jrfastab requested a review from a team as a code owner December 13, 2023 20:53
@jrfastab
Copy link
Contributor Author

/test-backport-1.13

@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 Dec 14, 2023
…publish"

[ upstream commit 0d1ee39 ]
This reverts commit c9ea7a5.

This works around a condition where restarting the agent uses a new
IP for Cilium Internal IP. But, it turns out this is because of an
incorrect set helm chart option in our reproducer. When configured
correctly we require CiliumInternalIP to reused so this patch is not
necessary. In fact it complicates the code so lets drop it.

The helm option is cleanState. It must be set to false

  cleanState=false

Note that cleanState="false" is a string type and will default to
true because of bool typing. Creating a subtle and broke config.

Signed-off-by: John Fastabend <john.fastabend@gmail.com>
@michi-covalent
Copy link
Contributor

michi-covalent commented Dec 15, 2023

/test-backport-1.13

Job 'Cilium-PR-K8s-1.23-kernel-4.19' failed:

Click to show.

Test Name

K8sDatapathServicesTest Checks E/W loadbalancing (ClusterIP, NodePort from inside cluster, etc) Tests NodePort inside cluster (kube-proxy) with the host firewall and externalTrafficPolicy=Local

Failure Output

FAIL: Request from k8s1 to service tftp://[fd04::12]:30260/hello failed

Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.23-kernel-4.19/551/

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-1.23-kernel-4.19 so I can create one.

Then please upload the Jenkins artifacts to that issue.

@michi-covalent
Copy link
Contributor

michi-covalent commented Dec 15, 2023

/mlh new-flake Cilium-PR-K8s-1.23-kernel-4.19

👍 created #29912

@michi-covalent
Copy link
Contributor

/test-1.23-4.19

1 similar comment
@michi-covalent
Copy link
Contributor

/test-1.23-4.19

@michi-covalent michi-covalent merged commit bae5aec into v1.13 Dec 15, 2023
140 checks passed
@michi-covalent michi-covalent deleted the pr/jrfastab/backport-v113 branch December 15, 2023 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.13 This PR represents a backport for Cilium 1.13.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
No open projects
Status: Released
Development

Successfully merging this pull request may close these issues.

None yet

4 participants