-
Notifications
You must be signed in to change notification settings - Fork 89
API changes to support infra creation #1485
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -48,9 +48,19 @@ type IBMPowerVSMachineSpec struct { | |
// Important: Run "make" to regenerate code after modifying this file | ||
|
||
// ServiceInstanceID is the id of the power cloud instance where the vsi instance will get deployed. | ||
// +kubebuilder:validation:MinLength=1 | ||
// Deprecated: use ServiceInstance instead | ||
ServiceInstanceID string `json:"serviceInstanceID"` | ||
|
||
// serviceInstance is the reference to the Power VS workspace on which the server instance(VM) will be created. | ||
// Power VS workspace is a container for all Power VS instances at a specific geographic region. | ||
// serviceInstance can be created via IBM Cloud catalog or CLI. | ||
// supported serviceInstance identifier in PowerVSResource are Name and ID and that can be obtained from IBM Cloud UI or IBM Cloud cli. | ||
// More detail about Power VS service instance. | ||
// https://cloud.ibm.com/docs/power-iaas?topic=power-iaas-creating-power-virtual-server | ||
// when omitted system will dynamically create the service instance | ||
// +optional | ||
ServiceInstance *IBMPowerVSResourceReference `json:"serviceInstance,omitempty"` | ||
|
||
// SSHKey is the name of the SSH key pair provided to the vsi for authenticating users. | ||
SSHKey string `json:"sshKey,omitempty"` | ||
|
||
|
@@ -121,6 +131,20 @@ type IBMPowerVSMachineSpec struct { | |
// ProviderID is the unique identifier as specified by the cloud provider. | ||
// +optional | ||
ProviderID *string `json:"providerID,omitempty"` | ||
|
||
// Ignition defined options related to the bootstrapping systems where Ignition is used. | ||
// +optional | ||
Ignition *Ignition `json:"ignition,omitempty"` | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why a struct here? right now I see just a version, do we expect anything else in the future? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Took a reference from other provider(aws). |
||
} | ||
|
||
// Ignition defines options related to the bootstrapping systems where Ignition is used. | ||
type Ignition struct { | ||
// Version defines which version of Ignition will be used to generate bootstrap data. | ||
// | ||
// +optional | ||
// +kubebuilder:default="2.3" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why it is set to 2.3 version? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Capi doc says only v2 is tested https://cluster-api.sigs.k8s.io/tasks/experimental-features/ignition#note |
||
// +kubebuilder:validation:Enum="2.3";"3.0";"3.1";"3.2";"3.3";"3.4" | ||
Version string `json:"version,omitempty"` | ||
} | ||
|
||
// IBMPowerVSResourceReference is a reference to a specific PowerVS resource by ID, Name or RegEx | ||
|
Uh oh!
There was an error while loading. Please reload this page.