MCO-2036: Enable boot image updates by default for vsphere and Azure#5583
MCO-2036: Enable boot image updates by default for vsphere and Azure#5583djoshy wants to merge 8 commits intoopenshift:mainfrom
Conversation
|
@djoshy: This pull request references MCO-2036 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. 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. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: djoshy The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@djoshy: This pull request references MCO-2036 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. 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. |
|
/payload-job periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-vsphere-mco-disruptive periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-gcp-mco-disruptive periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-aws-mco-disruptive |
|
@djoshy: trigger 4 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/fcd17e20-f636-11f0-9b60-3c66bfc66382-0 |
|
/hold we also need to reconcile the disruptive boot image tests when we enable these platforms by default, similar to openshift/origin#30712 |
|
/payload-job-with-prs periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-1of2 #5583 |
|
@djoshy: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/7b688eb0-f874-11f0-804b-b09532f7bee8-0 |
|
@djoshy: This PR was included in a payload test run from #5583
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/7b688eb0-f874-11f0-804b-b09532f7bee8-0 |
|
/payload-abort |
|
/payload-job-with-prs periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-1of2 #5428 |
|
@djoshy: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b1a46080-f874-11f0-9b01-9954da0d5f2e-0 |
|
/payload-job-with-prs periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-2of2 #5428 |
|
@djoshy: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b51b37c0-f874-11f0-8ef4-e8cbff9ca2ef-0 |
|
/payload-job-with-prs periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-vsphere-mco-disruptive-techpreview-1of2 #5428 |
|
/payload-job-with-prs periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-vsphere-mco-disruptive-techpreview-2of2 #5428 |
|
@djoshy: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/c0bf6970-f874-11f0-9e49-5df462546e97-0 |
|
@djoshy: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/c3f7b110-f874-11f0-9f8c-7baaab3813ed-0 |
|
/payload-abort |
4812e55 to
64a18b3
Compare
|
@djoshy: The following test failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
|
/payload-job periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-1of2 periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-2of2 periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-vsphere-mco-disruptive-techpreview-1of2 periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-vsphere-mco-disruptive-techpreview-2of2 |
|
@djoshy: trigger 4 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/cd9efc20-f886-11f0-8155-2ad31ce2de83-0 |
64a18b3 to
1a3bcbf
Compare
|
/payload-job periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-1of2 periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-2of2 |
|
@djoshy: trigger 2 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/f82ed910-fac0-11f0-8f83-481c4b491350-0 |
|
@djoshy: This pull request references MCO-2036 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. 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. |
This change adds logic to populate the BootImageSkewEnforcementStatus field in the MachineConfiguration status based on spec configuration, platform support, and cluster version information.
This adds new unit tests for TestSyncMachineConfiguration to test the BootImageSkewEnforcementStatus sync logic added in the previous commit.
This commit updates the machine-set-boot-image controller to track and update the BootImageSkewEnforcementStatus when in Automatic mode.
This commit implements upgrade blocking when boot image version skew exceeds acceptable limits, via the ClusterOperator Upgradeable condition.
This commit adds unit tests for the new Upgradeable guards added in the previous commit.
This commit ensures that the boot image controller state is acceptable before checking the skew. This check is only done in Automatic mode.
|
Putting this in draft so pushes don't run CI |
1a3bcbf to
b1a79c0
Compare
|
/test unit |
b1a79c0 to
17e92e2
Compare
|
/payload-job-with-prs periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-1of2 #5428 |
|
@djoshy: An error was encountered. No known errors were detected, please see the full error message for details. Full error message.
unable to get additional pr info from string: #5428: string: #5428 doesn't match expected format: org/repo#number
Please contact an administrator to resolve this issue. |
|
/payload-abort |
|
/payload-job periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-1of2 periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-azure-mco-disruptive-techpreview-2of2 periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-vsphere-mco-disruptive-techpreview-1of2 periodic-ci-openshift-machine-config-operator-release-4.22-periodics-e2e-vsphere-mco-disruptive-techpreview-2of2 |
|
@djoshy: trigger 4 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b11d76d0-fe12-11f0-9e3c-4a79235e6424-0 |
- What I did
This PR enables boot image updates by default for Azure and vsphere, and adjusts any test behavior to account for this. This PR is based on the skew enforcement MVP(#5428) and should only merge after that PR has landed.
- How to verify it
An installation on this PR should show the managedBootImageStatus MachineConfiguration object set to
Allfor MAPI MachineSet objects:As a result, boot image updates should took place automatically without a
spec.managedBootImagesfield defined. Any edits to the boot images defined in the MachineSet will be stomped back to the reference defined by thecoreos-bootimagesconfigmap.