Skip to content

Commit 54b96b5

Browse files
jichenjcmdbooth
authored andcommitted
Delete trunk on failure
Conflicts: pkg/cloud/services/compute/instance_test.go Unit tests have not been backported because they added test cases to unit tests which don't exist in release-0.5. (cherry picked from commit 28f820b)
1 parent d76e19e commit 54b96b5

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

pkg/cloud/services/compute/instance.go

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -486,7 +486,17 @@ func (s *Service) DeleteInstance(eventObject runtime.Object, instance *InstanceS
486486
}
487487

488488
func (s *Service) deletePorts(eventObject runtime.Object, nets []servers.Network) error {
489+
trunkSupported, err := s.isTrunkExtSupported()
490+
if err != nil {
491+
return err
492+
}
493+
489494
for _, n := range nets {
495+
if trunkSupported {
496+
if err = s.networkingService.DeleteTrunk(eventObject, n.Port); err != nil {
497+
return err
498+
}
499+
}
490500
if err := s.networkingService.DeletePort(eventObject, n.Port); err != nil {
491501
return err
492502
}

0 commit comments

Comments
 (0)