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

[Edge Case] Creation/Health Timeout not respected #811

Open
Tracked by #724
himanshu-kun opened this issue May 4, 2023 · 1 comment
Open
Tracked by #724

[Edge Case] Creation/Health Timeout not respected #811

himanshu-kun opened this issue May 4, 2023 · 1 comment
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/bug Bug lifecycle/rotten Nobody worked on this for 12 months (final aging stage) priority/3 Priority (lower number equals higher priority)

Comments

@himanshu-kun
Copy link
Contributor

How to categorize this issue?

/area quality
/kind bug
/priority 3

What happened:
The creationTimeout / healthTimeout is not respected in the following cases , as per code analysis :

Case 1:

  • machine obj = Pending , Node = NotReady , node conditions changed
    • Result : timeout not checked in this reconcile
    • if node conditions keep on changing , then timeout never checked

Case 2:

  • machine obj = Pending , Node = Ready, node.gardener.cloud/critical-components-not-ready taint present on node, node conditions changed
    • Result : timeout not checked
    • if node conditions keep on changing , then timeout never checked

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • Kubernetes version (use kubectl version):
  • Cloud provider or hardware configuration:
  • Others:
    any mcm provider vendoring mcm <= v0.49.0
@himanshu-kun himanshu-kun added the kind/bug Bug label May 4, 2023
@gardener-robot gardener-robot added area/quality Output qualification (tests, checks, scans, automation in general, etc.) related priority/3 Priority (lower number equals higher priority) labels May 4, 2023
@himanshu-kun
Copy link
Contributor Author

Post grooming issue

If node conditions keep flapping , then cloneDirty will become true in every reconcile, so timeout checking is always skipped.

should be dealt in controller runtime refactor.

@gardener-robot gardener-robot added the lifecycle/stale Nobody worked on this for 6 months (will further age) label Jan 19, 2024
@gardener-robot gardener-robot added lifecycle/rotten Nobody worked on this for 12 months (final aging stage) and removed lifecycle/stale Nobody worked on this for 6 months (will further age) labels Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/bug Bug lifecycle/rotten Nobody worked on this for 12 months (final aging stage) priority/3 Priority (lower number equals higher priority)
Projects
None yet
Development

No branches or pull requests

2 participants