Skip to content

[candi] remove excessive netcat calls from d8-shutdown-inhibitor #17153

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
Merged
name212 merged 1 commit into from
Dec 18, 2025
Merged

[candi] remove excessive netcat calls from d8-shutdown-inhibitor #17153

name212 merged 1 commit into from
Dec 18, 2025

Conversation

@armanokka
Copy link
Contributor

@armanokka armanokka commented Dec 18, 2025 •
edited
Loading

Description

remove excessive netcat calls from d8-shutdown-inhibitor

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

remove excessive netcat calls from d8-shutdown-inhibitor

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

No need

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: candi
type: fix
summary: remove excessive netcat calls from d8-shutdown-inhibitor
impact_level: default 

section: node-manager
type: fix
summary: remove excessive netcat calls from d8-shutdown-inhibitor
impact_level: default

@armanokka armanokka added this to the v1.74.0 milestone Dec 18, 2025
@github-actions github-actions bot added area/cluster-and-infrastructure Pull requests that update infra modules go Pull requests that update Go code labels Dec 18, 2025
@penul4ik penul4ik modified the milestones: v1.74.0, v1.74.1 Dec 18, 2025
@armanokka
[candi] remove excessive netcat calls from d8-shutdown-inhibitor
322f5fb 
Signed-off-by: Arman Toreniyazov <armangokka@gmail.com>
@armanokka armanokka force-pushed the fix-nc-shutdown-inhibitor branch from 91f9e6a to 322f5fb Compare December 18, 2025 12:16
@armanokka armanokka added the e2e/run/yandex-cloud Run e2e tests in Yandex Cloud label Dec 18, 2025
@deckhouse-BOaTswain
Copy link
Collaborator

deckhouse-BOaTswain commented Dec 18, 2025 •
edited
Loading

🟢 e2e: Yandex.Cloud for deckhouse:fix-nc-shutdown-inhibitor succeeded in 41m36s.

Workflow details

Yandex.Cloud-WithoutNAT-Containerd-1.32 - Connection string: ssh redos@51.250.87.89

🟢 e2e: Yandex.Cloud, Containerd, Kubernetes 1.32 succeeded in 40m9s.

@github-actions github-actions bot removed the e2e/run/yandex-cloud Run e2e tests in Yandex Cloud label Dec 18, 2025
@name212 name212 marked this pull request as ready for review December 18, 2025 19:56
@name212 name212 self-requested a review as a code owner December 18, 2025 19:56
@name212 name212 merged commit 8d9554c into main Dec 18, 2025
68 of 93 checks passed
@name212 name212 deleted the fix-nc-shutdown-inhibitor branch December 18, 2025 19:56
@deckhouse-BOaTswain deckhouse-BOaTswain mentioned this pull request Dec 18, 2025
Merged
@armanokka armanokka mentioned this pull request Dec 22, 2025
Merged
4 tasks
@deckhouse-BOaTswain deckhouse-BOaTswain mentioned this pull request Dec 23, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@name212 name212 name212 approved these changes

Assignees

@armanokka armanokka

Labels

area/cluster-and-infrastructure Pull requests that update infra modules go Pull requests that update Go code

Projects

None yet

Milestone

v1.74.1

Development

Successfully merging this pull request may close these issues.

5 participants

@armanokka @deckhouse-BOaTswain @name212 @penul4ik