Skip to content

Commit 0d39705

Browse files
authored
Merge pull request #1319 from stackhpc/fix/no-instance-secgroups
🐛 Add security groups to ports only
2 parents 3abfe14 + 5b98665 commit 0d39705

File tree

2 files changed

+0
-4
lines changed

2 files changed

+0
-4
lines changed

pkg/cloud/services/compute/instance.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,6 @@ func (s *Service) createInstanceImpl(eventObject runtime.Object, openStackCluste
235235
AvailabilityZone: instanceSpec.FailureDomain,
236236
Networks: portList,
237237
UserData: []byte(instanceSpec.UserData),
238-
SecurityGroups: securityGroups,
239238
Tags: instanceSpec.Tags,
240239
Metadata: instanceSpec.Metadata,
241240
ConfigDrive: &instanceSpec.ConfigDrive,

pkg/cloud/services/compute/instance_test.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -582,9 +582,6 @@ func TestService_ReconcileInstance(t *testing.T) {
582582
"imageRef": imageUUID,
583583
"flavorRef": flavorUUID,
584584
"availability_zone": failureDomain,
585-
"security_groups": []map[string]interface{}{
586-
{"name": workerSecurityGroupUUID},
587-
},
588585
"networks": []map[string]interface{}{
589586
{"port": portUUID},
590587
},

0 commit comments

Comments
 (0)