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.11] Backport of #18431 #18509

Merged
merged 3 commits into from
Jan 20, 2022

Conversation

christarazi
Copy link
Member

Once this PR is merged, you can update the PR labels via:

$ for pr in 18431; do contrib/backporting/set-labels.py $pr done 1.11; done

[ upstream commit 043a6fb ]

Tidy them up for a future commit to add on to it.

Signed-off-by: Chris Tarazi <chris@isovalent.com>
[ upstream commit 88647b4 ]

Previously, this option was set arbitrarily inside the
DeployCiliumAndDNS() helper, which is used quite extensively throughout
the test suite. We might as well add it as a default Helm option, so
that it's used consistently.

Signed-off-by: Chris Tarazi <chris@isovalent.com>
[ upstream commit 8e052d7 ]

Now these options are set by default inside
DeployCiliumOptionsAndDNS(), so they no longer need to be set
explicitly.

Signed-off-by: Chris Tarazi <chris@isovalent.com>
@christarazi christarazi requested a review from a team as a code owner January 18, 2022 00:30
@christarazi christarazi added backport/1.11 This PR represents a backport for Cilium 1.11.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Jan 18, 2022
@christarazi
Copy link
Member Author

christarazi commented Jan 18, 2022

/test-backport-1.11

Job 'Cilium-PR-K8s-1.21-kernel-5.4' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sDatapathConfig AutoDirectNodeRoutes Check connectivity with sockops and direct routing

Failure Output

FAIL: Error creating resource /home/jenkins/workspace/Cilium-PR-K8s-1.21-kernel-5.4/src/github.com/cilium/cilium/test/k8sT/manifests/l3-policy-demo.yaml: Cannot retrieve cilium pod cilium-fcprz policy revision: cannot get revision from json output 'Defaulted container "cilium-agent" out of: cilium-agent, mount-cgroup (init), clean-cilium-state (init)

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.21-kernel-5.4 so I can create a new GitHub issue to track it.

Job 'Cilium-PR-K8s-1.18-kernel-4.9' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sServicesTest Checks service across nodes with L7 policy Tests NodePort with L7 Policy

Failure Output

FAIL: Request from k8s1 to service http://[fd04::11]:30065 failed

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.18-kernel-4.9 so I can create a new GitHub issue to track it.

@christarazi christarazi changed the title v1.11 backports 2022-01-17 [v1.11] Backport of #18431 Jan 18, 2022
@christarazi
Copy link
Member Author

5.4 hit #17069.

@christarazi
Copy link
Member Author

christarazi commented Jan 18, 2022

/mlh new-flake Cilium-PR-K8s-1.18-kernel-4.9

👍 created #18530

@christarazi christarazi added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jan 20, 2022
@kkourt kkourt merged commit 51e56c3 into cilium:v1.11 Jan 20, 2022
@christarazi christarazi deleted the pr/v1.11-backport-2022-01-17 branch January 20, 2022 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.11 This PR represents a backport for Cilium 1.11.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
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants