Skip to content

Commit feb8b91

Browse files
committed
Add FlavorID to be set by openStackMachineSpecToOpenStackServerSpec
FlavorID can be used in stead of Flavor when specifying machine and server is then created succesfully as well. Additional tests testing Flavor and FlavorID conversion from machine to server. Signed-off-by: Siiri Kemppainen <siiri.kemppainen@est.tech>
1 parent 606fcbd commit feb8b91

File tree

2 files changed

+38
-1
lines changed

2 files changed

+38
-1
lines changed

controllers/openstackmachine_controller.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -484,6 +484,7 @@ func openStackMachineSpecToOpenStackServerSpec(openStackMachineSpec *infrav1.Ope
484484
AdditionalBlockDevices: openStackMachineSpec.AdditionalBlockDevices,
485485
ConfigDrive: openStackMachineSpec.ConfigDrive,
486486
Flavor: openStackMachineSpec.Flavor,
487+
FlavorID: openStackMachineSpec.FlavorID,
487488
IdentityRef: identityRef,
488489
Image: openStackMachineSpec.Image,
489490
RootVolume: openStackMachineSpec.RootVolume,

controllers/openstackmachine_controller_test.go

Lines changed: 37 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ func TestOpenStackMachineSpecToOpenStackServerSpec(t *testing.T) {
117117
},
118118
},
119119
{
120-
name: "Test a OpenStackMachineSpec to OpenStackServerSpec conversion with an additional security group",
120+
name: "Test an OpenStackMachineSpec to OpenStackServerSpec conversion with an additional security group",
121121
spec: &infrav1.OpenStackMachineSpec{
122122
Flavor: ptr.To(flavorName),
123123
Image: image,
@@ -138,6 +138,42 @@ func TestOpenStackMachineSpecToOpenStackServerSpec(t *testing.T) {
138138
UserDataRef: userData,
139139
},
140140
},
141+
{
142+
name: "Test an OpenStackMachineSpec to OpenStackServerSpec conversion with flavor and flavorID specified",
143+
spec: &infrav1.OpenStackMachineSpec{
144+
Flavor: ptr.To(flavorName),
145+
FlavorID: ptr.To(flavorUUID),
146+
Image: image,
147+
SSHKeyName: sshKeyName,
148+
},
149+
want: &infrav1alpha1.OpenStackServerSpec{
150+
Flavor: ptr.To(flavorName),
151+
FlavorID: ptr.To(flavorUUID),
152+
IdentityRef: identityRef,
153+
Image: image,
154+
SSHKeyName: sshKeyName,
155+
Ports: portOpts,
156+
Tags: tags,
157+
UserDataRef: userData,
158+
},
159+
},
160+
{
161+
name: "Test an OpenStackMachineSpec to OpenStackServerSpec conversion with flavorID specified but not flavor",
162+
spec: &infrav1.OpenStackMachineSpec{
163+
FlavorID: ptr.To(flavorUUID),
164+
Image: image,
165+
SSHKeyName: sshKeyName,
166+
},
167+
want: &infrav1alpha1.OpenStackServerSpec{
168+
FlavorID: ptr.To(flavorUUID),
169+
IdentityRef: identityRef,
170+
Image: image,
171+
SSHKeyName: sshKeyName,
172+
Ports: portOpts,
173+
Tags: tags,
174+
UserDataRef: userData,
175+
},
176+
},
141177
}
142178
for i := range tests {
143179
tt := tests[i]

0 commit comments

Comments
 (0)