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

[kube-dns] "prefer_udp" in stub zones #2774

Merged
merged 1 commit into from
Oct 19, 2022

Conversation

zuzzas
Copy link
Contributor

@zuzzas zuzzas commented Oct 19, 2022

Description

Forgot about "prefer_udp" in stub zones in #2413

Why do we need it, and what problem does it solve?

Same as in the previous issue.

What is the expected result?

CoreDNS with enabled stub zones is no longer affected by the aforementioned issue.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: kube-dns
type: fix
summary: Added "prefer_udp" to stub zones.

Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
@zuzzas zuzzas added type/bug area/cluster-and-infrastructure Pull requests that update infra modules labels Oct 19, 2022
@zuzzas zuzzas added this to the v1.38.1 milestone Oct 19, 2022
@zuzzas zuzzas self-assigned this Oct 19, 2022
@zuzzas zuzzas added e2e/use/k8s/1.23 e2e/run/yandex-cloud Run e2e tests in Yandex Cloud labels Oct 19, 2022
@deckhouse-BOaTswain
Copy link
Collaborator

deckhouse-BOaTswain commented Oct 19, 2022

🟢 e2e: Yandex.Cloud for deckhouse:node-local-dns-prefer-udp-stub-zones succeeded in 34m18s.

Workflow details

🟢 e2e: Yandex.Cloud, Containerd, Kubernetes 1.23 succeeded in 33m41s.

@github-actions github-actions bot removed the e2e/run/yandex-cloud Run e2e tests in Yandex Cloud label Oct 19, 2022
@RomanenkoDenys RomanenkoDenys merged commit c1bf28b into main Oct 19, 2022
@RomanenkoDenys RomanenkoDenys deleted the node-local-dns-prefer-udp-stub-zones branch October 19, 2022 16:54
@RomanenkoDenys RomanenkoDenys added the status/backport Backport pr label Oct 21, 2022
github-actions bot pushed a commit that referenced this pull request Oct 21, 2022
Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
deckhouse-BOaTswain added a commit that referenced this pull request Oct 21, 2022
Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>

Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
Co-authored-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
@deckhouse-BOaTswain
Copy link
Collaborator

Cherry pick PR 2799 to the branch release-1.38 successful!

@deckhouse-BOaTswain deckhouse-BOaTswain removed the status/backport Backport pr label Oct 21, 2022
@RomanenkoDenys
Copy link
Member

/backport v1.37

github-actions bot pushed a commit that referenced this pull request Oct 21, 2022
Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
@deckhouse-BOaTswain
Copy link
Collaborator

Cherry pick PR 2800 to the branch release-1.37 successful!

deckhouse-BOaTswain added a commit that referenced this pull request Oct 21, 2022
Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>

Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
Co-authored-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
kvaps pushed a commit that referenced this pull request Oct 21, 2022
Signed-off-by: Andrey Klimentyev <andrey.klimentyev@flant.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cluster-and-infrastructure Pull requests that update infra modules type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants