Skip to content

Commit 3142b92

Browse files
authored
Support service instance id (#1652)
1 parent 0a849e2 commit 3142b92

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

api/v1beta2/ibmpowervscluster_webhook.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,10 @@ func (r *IBMPowerVSCluster) validateIBMPowerVSClusterCreateInfraPrereq() *field.
121121
return field.Invalid(field.NewPath("spec.zone"), r.Spec.Zone, "value of zone is empty")
122122
}
123123

124+
if r.Spec.VPC == nil {
125+
return field.Invalid(field.NewPath("spec.vpc"), r.Spec.VPC, "value of VPC is empty")
126+
}
127+
124128
if r.Spec.VPC.Region == nil {
125129
return field.Invalid(field.NewPath("spec.vpc.region"), r.Spec.VPC.Region, "value of VPC region is empty")
126130
}

cloud/scope/powervs_machine.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -162,6 +162,8 @@ func NewPowerVSMachineScope(params PowerVSMachineScopeParams) (scope *PowerVSMac
162162
var serviceInstanceID, serviceInstanceName string
163163
if params.IBMPowerVSMachine.Spec.ServiceInstanceID != "" {
164164
serviceInstanceID = params.IBMPowerVSMachine.Spec.ServiceInstanceID
165+
} else if params.IBMPowerVSMachine.Spec.ServiceInstance != nil && params.IBMPowerVSMachine.Spec.ServiceInstance.ID != nil {
166+
serviceInstanceID = *params.IBMPowerVSMachine.Spec.ServiceInstance.ID
165167
} else {
166168
serviceInstanceName = fmt.Sprintf("%s-%s", params.IBMPowerVSCluster.GetName(), "serviceInstance")
167169
if params.IBMPowerVSCluster.Spec.ServiceInstance != nil && params.IBMPowerVSCluster.Spec.ServiceInstance.Name != nil {

0 commit comments

Comments
 (0)