-
Notifications
You must be signed in to change notification settings - Fork 510
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SPLAT-1743: vSphere - add host and vm based zonal #1999
base: master
Are you sure you want to change the base?
SPLAT-1743: vSphere - add host and vm based zonal #1999
Conversation
@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. |
Skipping CI for Draft Pull Request. |
Hello @jcpowermac! Some important instructions when contributing to openshift/api: |
(will re-write the commit message with appropriate details once this PR is closer to being ready) /test build /test e2e-aws-ovn-techpreview /test e2e-upgrade |
3ea73d9
to
b03796b
Compare
/test build |
f1bb716
to
4790ee0
Compare
/test build |
1 similar comment
/test build |
dff5476
to
e1b6bd3
Compare
2975083
to
3f02f0e
Compare
3f02f0e
to
c5634ed
Compare
/test build |
a8ac815
to
197af3a
Compare
/test build |
@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. |
@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. |
@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. |
197af3a
to
c5925b5
Compare
/test unit |
@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. |
@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. |
tests/generator.go
Outdated
@@ -178,6 +179,8 @@ func generateOnCreateTable(onCreateTests []OnCreateTestSpec) { | |||
expectedObj.SetName(gotObj.GetName()) | |||
expectedObj.SetNamespace(gotObj.GetNamespace()) | |||
|
|||
fmt.Println(gotObj) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will reset after review
tests/suite_test.go
Outdated
@@ -37,7 +37,8 @@ func TestAPIs(t *testing.T) { | |||
|
|||
var err error | |||
// this assumes a directory. Because it does, we shall assume a path to the serialized featuregate manifests too. | |||
suites, err = LoadTestSuiteSpecs(filepath.Join("..")) | |||
//suites, err = LoadTestSuiteSpecs(filepath.Join("..")) | |||
suites, err = LoadTestSuiteSpecs(filepath.Join("../config/v1/tests/infrastructures.config.openshift.io")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will reset after review
/lgtm |
c5925b5
to
3944b3e
Compare
New changes are detected. LGTM label has been removed. |
3944b3e
to
69ee49e
Compare
/assign @JoelSpeed |
69ee49e
to
7642ae7
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jcpowermac, rvanderp3 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
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.
7642ae7
to
8cb746e
Compare
@jcpowermac: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
Changes
VSphereHostVMGroupZonal
RegionType
andZoneType
fields toVSpherePlatformFailureDomainSpec
to specify region and zone failure domain types.VSphereFailureDomainAffinity
type that contains three fields required for vm-host zonal:VMGroup
,HostGroup
andVMHostRule
.VSphereFailureDomainAffinity
toVSpherePlatformTopology
VMGroup
in a machine(s) workspace.Additional PRs
SPLAT-1733: Update vSphere and include host/vm group enhancements#1677
SPLAT-1742: vSphere - enable host group based zonal installer#8873
WIP - SPLAT-1781: api bump for host zonal infra change client-go#294
WIP - SPLAT-1780: api bump for host zonal infra change library-go#1782
SPLAT-1799: Add support for vSphere host and vm group based zonal cluster-control-plane-machine-set-operator#325
SPLAT-1800: Add support for vSphere host and vm group based zonal machine-api-operator#1285
SPLAT-1743: vSphere - add host and vm based zonal #1999