api icon indicating copy to clipboard operation
api copied to clipboard

SPLAT-1743: vSphere - add host and vm based zonal

Open jcpowermac opened this issue 1 year ago • 20 comments

Changes

  • New feature gate VSphereHostVMGroupZonal
  • Add RegionType and ZoneType fields to VSpherePlatformFailureDomainSpec to specify region and zone failure domain types.
  • Add VSphereFailureDomainAffinity type that contains three fields required for vm-host zonal: VMGroup, HostGroup and VMHostRule.
  • Add VSphereFailureDomainAffinity to VSpherePlatformTopology
  • Create additional unit tests to cover the introduction of vm-host zonal provisioning types.
  • Add VMGroup in a machine(s) workspace.

Additional PRs

  • https://github.com/openshift/enhancements/pull/1677

  • https://github.com/openshift/installer/pull/8873

  • https://github.com/openshift/client-go/pull/294

  • https://github.com/openshift/library-go/pull/1782

  • https://github.com/openshift/cluster-control-plane-machine-set-operator/pull/325

  • https://github.com/openshift/machine-api-operator/pull/1285

  • https://github.com/openshift/api/pull/1999

jcpowermac avatar Aug 14 '24 13:08 jcpowermac

@jcpowermac: This pull request references SPLAT-1743 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.18.0" version, but no target version was set.

In 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.

openshift-ci-robot avatar Aug 14 '24 13:08 openshift-ci-robot

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

openshift-ci[bot] avatar Aug 14 '24 13:08 openshift-ci[bot]

Hello @jcpowermac! Some important instructions when contributing to openshift/api: API design plays an important part in the user experience of OpenShift and as such API PRs are subject to a high level of scrutiny to ensure they follow our best practices. If you haven't already done so, please review the OpenShift API Conventions and ensure that your proposed changes are compliant. Following these conventions will help expedite the api review process for your PR.

openshift-ci[bot] avatar Aug 14 '24 13:08 openshift-ci[bot]

(will re-write the commit message with appropriate details once this PR is closer to being ready)

/test build /test e2e-aws-ovn

/test e2e-aws-ovn-techpreview

/test e2e-upgrade /test e2e-upgrade-minor /test images /test integration /test unit /test verify /test verify-client-go /test verify-crd-schema /test verify-deps

jcpowermac avatar Aug 14 '24 13:08 jcpowermac

/test build /test images /test integration /test unit /test verify /test verify-client-go /test verify-crd-schema /test verify-deps

jcpowermac avatar Aug 20 '24 20:08 jcpowermac

/test build /test images /test integration /test unit /test verify /test verify-client-go /test verify-crd-schema /test verify-deps

jcpowermac avatar Aug 27 '24 14:08 jcpowermac

/test build /test images /test integration /test unit /test verify /test verify-client-go /test verify-crd-schema /test verify-deps

jcpowermac avatar Aug 27 '24 17:08 jcpowermac

/test build /test images /test integration /test unit /test verify /test verify-client-go /test verify-crd-schema /test verify-deps

jcpowermac avatar Sep 20 '24 20:09 jcpowermac

/test build /test images /test integration /test unit /test verify /test verify-client-go /test verify-crd-schema /test verify-deps

jcpowermac avatar Sep 23 '24 14:09 jcpowermac

@jcpowermac: This pull request references SPLAT-1743 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.18.0" version, but no target version was set.

In response to this:

Changes

Additional PRs

https://github.com/openshift/api/pull/1999 https://github.com/openshift/installer/pull/8873 https://github.com/openshift/client-go/pull/294 https://github.com/openshift/library-go/pull/1782 https://github.com/openshift/cluster-control-plane-machine-set-operator/pull/325 https://github.com/openshift/machine-api-operator/pull/1285

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.

openshift-ci-robot avatar Sep 23 '24 14:09 openshift-ci-robot

@jcpowermac: This pull request references SPLAT-1743 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.18.0" version, but no target version was set.

In response to this:

Changes

Additional PRs

https://github.com/openshift/enhancements/pull/1677 https://github.com/openshift/api/pull/1999 https://github.com/openshift/installer/pull/8873 https://github.com/openshift/client-go/pull/294 https://github.com/openshift/library-go/pull/1782 https://github.com/openshift/cluster-control-plane-machine-set-operator/pull/325 https://github.com/openshift/machine-api-operator/pull/1285

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.

openshift-ci-robot avatar Sep 23 '24 14:09 openshift-ci-robot

@jcpowermac: This pull request references SPLAT-1743 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.18.0" version, but no target version was set.

In response to this:

Changes

  • New feature gate VSphereHostVMGroupZonal
  • Add RegionType and ZoneType fields to VSpherePlatformFailureDomainSpec to specify region and zone failure domain types.
  • Add VSphereFailureDomainAffinity type that contains three fields required for vm-host zonal: VMGroup, HostGroup and VMHostRule.
  • Add VSphereFailureDomainAffinity to VSpherePlatformTopology
  • Create additional unit tests to cover the introduction of vm-host zonal provisioning types.

Additional PRs

https://github.com/openshift/enhancements/pull/1677 https://github.com/openshift/api/pull/1999 https://github.com/openshift/installer/pull/8873 https://github.com/openshift/client-go/pull/294 https://github.com/openshift/library-go/pull/1782 https://github.com/openshift/cluster-control-plane-machine-set-operator/pull/325 https://github.com/openshift/machine-api-operator/pull/1285

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.

openshift-ci-robot avatar Sep 23 '24 14:09 openshift-ci-robot

/test unit

jcpowermac avatar Sep 23 '24 15:09 jcpowermac

@jcpowermac: This pull request references SPLAT-1743 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.18.0" version, but no target version was set.

In response to this:

Changes

  • New feature gate VSphereHostVMGroupZonal
  • Add RegionType and ZoneType fields to VSpherePlatformFailureDomainSpec to specify region and zone failure domain types.
  • Add VSphereFailureDomainAffinity type that contains three fields required for vm-host zonal: VMGroup, HostGroup and VMHostRule.
  • Add VSphereFailureDomainAffinity to VSpherePlatformTopology
  • Create additional unit tests to cover the introduction of vm-host zonal provisioning types.

Additional PRs

  • https://github.com/openshift/enhancements/pull/1677

  • https://github.com/openshift/installer/pull/8873

  • https://github.com/openshift/client-go/pull/294

  • https://github.com/openshift/library-go/pull/1782

  • https://github.com/openshift/cluster-control-plane-machine-set-operator/pull/325

  • https://github.com/openshift/machine-api-operator/pull/1285

  • https://github.com/openshift/api/pull/1999

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.

openshift-ci-robot avatar Sep 23 '24 15:09 openshift-ci-robot

@jcpowermac: This pull request references SPLAT-1743 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.18.0" version, but no target version was set.

In response to this:

Changes

  • New feature gate VSphereHostVMGroupZonal
  • Add RegionType and ZoneType fields to VSpherePlatformFailureDomainSpec to specify region and zone failure domain types.
  • Add VSphereFailureDomainAffinity type that contains three fields required for vm-host zonal: VMGroup, HostGroup and VMHostRule.
  • Add VSphereFailureDomainAffinity to VSpherePlatformTopology
  • Create additional unit tests to cover the introduction of vm-host zonal provisioning types.
  • Add VMGroup in a machine(s) workspace.

Additional PRs

  • https://github.com/openshift/enhancements/pull/1677

  • https://github.com/openshift/installer/pull/8873

  • https://github.com/openshift/client-go/pull/294

  • https://github.com/openshift/library-go/pull/1782

  • https://github.com/openshift/cluster-control-plane-machine-set-operator/pull/325

  • https://github.com/openshift/machine-api-operator/pull/1285

  • https://github.com/openshift/api/pull/1999

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.

openshift-ci-robot avatar Sep 23 '24 16:09 openshift-ci-robot

/lgtm

rvanderp3 avatar Sep 23 '24 19:09 rvanderp3

/assign @JoelSpeed

jcpowermac avatar Sep 23 '24 19:09 jcpowermac

@JoelSpeed this pr and the enhancement https://github.com/openshift/enhancements/pull/1677 are ready for review when you have a chance

jcpowermac avatar Oct 14 '24 11:10 jcpowermac

@JoelSpeed https://github.com/openshift/api/pull/1999/commits/2ee822a5c01f76da9639f182fac153ab09df51c1 changes from our meeting earlier today

jcpowermac avatar Oct 23 '24 16:10 jcpowermac

/test minor-e2e-upgrade-minor

jcpowermac avatar Nov 04 '24 17:11 jcpowermac

@JoelSpeed I pushed fe164067da0405e64fcbc4cd73d38f09b1a11a7e with changes to CEL, required fields and MinLength. Included some additional unit tests.

jcpowermac avatar Nov 04 '24 18:11 jcpowermac

@JoelSpeed rebased and pushed with the latest changes.

jcpowermac avatar Nov 11 '24 15:11 jcpowermac

@JoelSpeed updated

jcpowermac avatar Nov 21 '24 15:11 jcpowermac

Verify looks like there's a need to re-run the update scripts? Diff in the generated CRDs

JoelSpeed avatar Nov 21 '24 16:11 JoelSpeed

Verify looks like there's a need to re-run the update scripts? Diff in the generated CRDs

It looks like I needed to rebase, just pushed

jcpowermac avatar Nov 21 '24 17:11 jcpowermac

/test verify-crd-schema

JoelSpeed avatar Nov 25 '24 10:11 JoelSpeed

/test minor-images /test e2e-aws-ovn-hypershift /test e2e-upgrade

JoelSpeed avatar Nov 25 '24 12:11 JoelSpeed

/override ci/prow/verify-crd-schema

False positives on NoNewRequiredFields, the parent of these fields is optional and omitempty, therefore the new required field is not a breaking change for existing clients

JoelSpeed avatar Nov 25 '24 13:11 JoelSpeed

@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/verify-crd-schema

In response to this:

/override ci/prow/verify-crd-schema

False positives on NoNewRequiredFields, the parent of these fields is optional and omitempty, therefore the new required field is not a breaking change for existing clients

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 kubernetes-sigs/prow repository.

openshift-ci[bot] avatar Nov 25 '24 13:11 openshift-ci[bot]

Huh, why did the integration test fail now? 🤔

JoelSpeed avatar Nov 26 '24 13:11 JoelSpeed