-
Bug
-
Resolution: Done
-
Normal
-
None
-
None
-
None
-
None
-
False
-
None
-
False
-
OCPSTRAT-854 - Support Heterogeneous NodePools Within HyperShift
-
-
-
Hypershift Sprint 245, Hypershift Sprint 246, Hypershift Sprint 247, Hypershift Sprint 249, Hypershift Sprint 250, Hypershift Sprint 251, Hypershift Sprint 252, Hypershift Sprint 253, Hypershift Sprint 254
-
0
-
0
-
0
User Story:
As a user of HyperShift, I would like the UX around the `arch` flag validation improved so that it results in a smoother UX experience. The problem today is we default Arch to `amd64`, but then throw an invalid status message on the NodePool CRD if it's not blank and the platform is not AWS.
UPDATE:
This is not just a UX improvement. It prevents cluster creation:
- lastTransitionTime: "2023-11-27T05:10:07Z"
message: 'Arch flag is not supported for platform: KubeVirt'
observedGeneration: 1
reason: ValidationFailed
status: "False"
type: ValidArchPlatform
The above error is treated as a warning during testing made us ignore it.
Acceptance Criteria:
DEFINE once path forward decided; SEE Engineering Details for more details.
Description of criteria:
(optional) Out of Scope:
Detail about what is specifically not being delivered in the story
Engineering Details:
At a minimum we should remove the empty `Arch` flag check here.
What about modifying the section to something like this:
// Validate modifying CPU arch support for platform if (nodePool.Spec.Arch != "amd64") && (nodePool.Spec.Platform.Type != hyperv1.AWSPlatform) { SetStatusCondition(&nodePool.Status.Conditions, hyperv1.NodePoolCondition{ Type: hyperv1.NodePoolValidArchPlatform, Status: corev1.ConditionFalse, Reason: hyperv1.NodePoolInvalidArchPlatform, Message: fmt.Sprintf("modifying CPU arch from 'amd64' not supported for platform: %s", nodePool.Spec.Platform.Type), ObservedGeneration: nodePool.Generation, }) }
This requires/does not require a design proposal.
This requires/does not require a feature gate.
- clones
-
HOSTEDCP-1113 Improve UX Around Arch Flag Validation
-
- Closed
-
- links to