OCPBUGS-83862: Generate autosizing-disabled machineconfig only for master and worker nodes on OCP 4.20#5885
OCPBUGS-83862: Generate autosizing-disabled machineconfig only for master and worker nodes on OCP 4.20#5885aksjadha wants to merge 2 commits intoopenshift:release-4.20from
Conversation
…ster and worker nodes on OCP 4.20
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: aksjadha 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 @aksjadha. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. 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. |
|
@aksjadha: This pull request references Jira Issue OCPBUGS-83862, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
Instructions 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 openshift-eng/jira-lifecycle-plugin repository. |
|
|
||
| // ensureAutoSizingMachineConfigs ensures auto-sizing MachineConfigs exist for the master and worker MachineConfigPools | ||
| func (ctrl *Controller) ensureAutoSizingMachineConfigs(ctx context.Context) error { | ||
| mcpPools, err := ctrl.mcpLister.List(labels.Everything()) |
There was a problem hiding this comment.
Can you explore if we can get only "master" and "worker" pool at this point instead of dropping it later in the code?
Fixes: OCPBUGS-83862
- What I did
introduced during the upgrade to 4.20 where autosizing was disabled to avoid impact on 4.21.
- How to verify it
- Description for the changelog