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.9 backports 2021-10-04 #17532

Merged
merged 6 commits into from
Oct 7, 2021
Merged

v1.9 backports 2021-10-04 #17532

merged 6 commits into from
Oct 7, 2021

Conversation

glibsm
Copy link
Member

@glibsm glibsm commented Oct 4, 2021

PRs skipped due conflicts:

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

$ for pr in 15450 16969 16264 17424 17417 17489; do contrib/backporting/set-labels.py $pr done 1.9; done

tgraf and others added 6 commits October 4, 2021 22:17
[ upstream commit 10c301d ]

These warnings describe situations that can happen regularly and are no
cause of concern. Convert them into debug log messages.

Signed-off-by: Thomas Graf <thomas@cilium.io>

Signed-off-by: Glib Smaga <code@gsmaga.com>
[ upstream commit a8b3480 ]

It was not clear if kernel v5.8 has problem with libceph or if 5.8 fixes the problem.
Redo the sentence based on feedback to make it more clear and easy to read.

Signed-off-by: Ville Ojamo <bluikko@users.noreply.github.com>

Signed-off-by: Glib Smaga <code@gsmaga.com>
[ upstream commit 27fd5cc ]

Signed-off-by: Stijn Smits <stijn@stijn98s.nl>

Signed-off-by: Glib Smaga <code@gsmaga.com>
[ upstream commit 83d30de ]

Having these environment variables allows the cherry-pick script to be
used on other projects that are not Cilium.

Signed-off-by: André Martins <andre@cilium.io>

Signed-off-by: Glib Smaga <code@gsmaga.com>
[ upstream commit 9e4d84b ]

The Kubernetes' client User-Agent was never set and it would always
fallback to the default value. This commit fixes this issue and now all
Cilium components will correctly present their User-Agent.

Fixes: b31ed33 ("Add k8s client qps and burst as cli flags for the operator")
Signed-off-by: André Martins <andre@cilium.io>

Signed-off-by: Glib Smaga <code@gsmaga.com>
[ upstream commit f6f2406 ]

coredns < 1.7.0 has a bug that makes the services resolution to become
out-of-sync with the last state from Kubernetes in case coredns
suffers from a disconnection with kube-apiserver [1]. This bug is fixed on
all versions equal and above 1.7.0. [2]

In our CI this affects all Kubernetes jobs 1.18 and below and can result
in flaky tests that have the result in the following similar logs:

```
service IP retrieved from DNS (10.101.253.144) does not match the IP for the service stored in Kubernetes (10.108.15.225)
```

[1] coredns/coredns#3587
[2] coredns/coredns#3924

Signed-off-by: André Martins <andre@cilium.io>

Signed-off-by: Glib Smaga <code@gsmaga.com>
@glibsm glibsm requested a review from a team as a code owner October 4, 2021 23:09
@glibsm glibsm requested review from aanm and tgraf October 4, 2021 23:09
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.9 kind/backports This PR provides functionality previously merged into master. labels Oct 4, 2021
Copy link
Member

@aanm aanm left a comment

Choose a reason for hiding this comment

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

I've reviewed all commits

@aanm aanm removed the request for review from tgraf October 4, 2021 23:17
@aanm aanm unassigned tgraf Oct 4, 2021
@glibsm
Copy link
Member Author

glibsm commented Oct 4, 2021

/test-backport-1.9

@glibsm
Copy link
Member Author

glibsm commented Oct 6, 2021

GKE failure for 1.9 is a known tracked issue #17282

@glibsm glibsm added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 6, 2021
@errordeveloper errordeveloper merged commit efa8cc7 into v1.9 Oct 7, 2021
@errordeveloper errordeveloper deleted the pr/v1.9-backport-2021-10-04 branch October 7, 2021 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

6 participants