8000 add timeout for dns handlser by AndrewGodGivens · Pull Request #13977 · deckhouse/deckhouse · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

add timeout for dns handlser #13977

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add timeout for dns handlser #13977

wants to merge 1 commit into from

Conversation

AndrewGodGivens
Copy link
Contributor
@AndrewGodGivens AndrewGodGivens commented Jun 16, 2025 •
edited
Loading

Description

A timeout was added for DNS checking during smoke tests. And the logging mechanism was improved in this test.

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

This resolves the issue of incorrect test execution when the DNS server does not respond.

Why do we need it in the patch release (if we do)?

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: upmeter
type: chore
summary: Add timeout for dns test and improve logging
impact_level: low

@AndrewGodGivens AndrewGodGivens requested a review from shvgn as a code owner June 16, 2025 06:43
@github-actions github-actions bot added area/monitoring Pull requests that update monitoring modules go Pull requests that update Go code labels Jun 16, 2025
@AndrewGodGivens AndrewGodGivens added this to the v1.71.0 milestone Jun 16, 2025
@AndrewGodGivens AndrewGodGivens force-pushed the update_smoke_mini branch 2 times, most recently from 83ba065 to 37290f7 Compare June 16, 2025 16:37
@AndrewGodGivens
add timeout for dns handlser
79d096f 
Signed-off-by: Andrey Bogdanov <563132@gmail.com>
@ykav-flant ykav-flant added the e2e/run/yandex-cloud Run e2e tests in Yandex Cloud label Jun 19, 2025
@deckhouse-BOaTswain
Copy link
Collaborator
deckhouse-BOaTswain commented Jun 19, 2025 •
edited
Loading

🟢 e2e: Yandex.Cloud for deckhouse:update_smoke_mini succeeded in 36m49s.

Workflow details

Yandex.Cloud-WithoutNAT-Containerd-1.30 - Connection string: ssh redos@89.169.133.9

🟢 e2e: Yandex.Cloud, Containerd, Kubernetes 1.30 succeeded in 35m43s.

@github-actions github-actions bot removed the e2e/run/yandex-cloud Run e2e tests in Yandex Cloud label Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shvgn shvgn Awaiting requested review from shvgn shvgn is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@AndrewGodGivens AndrewGodGivens

Labels
area/monitoring Pull requests that update monitoring modules go Pull requests that update Go code
Projects
None yet
Milestone
v1.71.0
Development

Successfully merging this pull request may close these issues.
3 participants
@AndrewGodGivens @deckhouse-BOaTswain @ykav-flant
0