🐛 Fix Available condition for cluster when workers are not required#13366
🐛 Fix Available condition for cluster when workers are not required#13366apedriza wants to merge 1 commit intokubernetes-sigs:mainfrom
Available condition for cluster when workers are not required#13366Conversation
Signed-off-by: apedriza <adripedriza@gmail.com>
|
This PR is currently missing an area label, which is used to identify the modified component when generating release notes. Area labels can be added by org members by writing Please see the labels list for possible areas. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @apedriza. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
The condition was intentionally set to true. I think it's not correct to signal some kind of error by setting this to false if there are no workers. (we made similar decisions across a few conditions and objects) |
What this PR does / why we need it:
When a cluster is created,
WorkersAvailable,ControlPlaneMachinesReadyorWorkerMachinesReadyconditions aretruebut there are no replicas at that moment yet. It can be confusing to initially check the status based on those conditions.Due to the changes made, now that
WorkerAvailablecondition is only set totrueif there is any worker machine, include the condition in the clusterAvailablecondition calculation only in there are any required worker.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #