SPLAT-2238: Cleaned up vSphere zonal jobs from using hard coded pools - #81215
Merged
Conversation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
SPLAT-2238
Changes
Summary by CodeRabbit
This PR updates OpenShift CI’s vSphere IPI configuration flow (the vSphere VCM path) to generate zone/failure-domain information dynamically from VCM-provided
platform.json, eliminating reliance on hard-coded vSphere zone/pool definitions.Practically:
zonesfromplatform.json’sfailureDomains(when present), using replica-aware selection capped toCONTROL_PLANE_REPLICAS/COMPUTE_NODE_REPLICASand configured pool sizing viaPOOL_COUNT(defaulted to4in the relevant chains/workflows).jq) for the control-plane (CP_ZONES_BLOCK) and compute (W_ZONES_BLOCK) and injects them into the generatedinstall-configplatform sections (including the compact size variant), inserting the zone blocks afterdiskSizeGB: 120.ipi-conf-vsphere-zoneschain; migration guidance is added to use the VCM-based flow (e.g., via thevsphere-elasticcluster profile).e2e-vsphere-ovn-zonesjob configuration are updated to setPOOL_COUNT, ensuring zone generation follows the updated pool-count-driven behavior.