Skip to content

Commit 6ae5d6e

Browse files
authored
Merge pull request #1117 from ioito/automated-cherry-pick-of-#1116-upstream-release-3.12
Automated cherry pick of #1116: fix(azure): azure nic delete
2 parents 0ed883a + f1e7d33 commit 6ae5d6e

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

pkg/multicloud/azure/host.go

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,14 @@ func (self *SHost) CreateVM(desc *cloudprovider.SManagedVMCreateConfig) (cloudpr
7272

7373
instance, err := self.zone.region._createVM(desc, nic.ID)
7474
if err != nil {
75-
self.zone.region.DeleteNetworkInterface(nic.ID)
75+
cloudprovider.Wait(time.Minute*2, time.Minute*6, func() (bool, error) {
76+
e := self.zone.region.DeleteNetworkInterface(nic.ID)
77+
if e == nil {
78+
return true, nil
79+
}
80+
log.Errorf("delete nic %s error: %v", nic.ID, err)
81+
return false, nil
82+
})
7683
return nil, err
7784
}
7885
instance.host = self
@@ -122,7 +129,8 @@ func (self *SRegion) _createVM(desc *cloudprovider.SManagedVMCreateConfig, nicId
122129
"NetworkProfile": map[string]interface{}{
123130
"NetworkInterfaces": []map[string]string{
124131
map[string]string{
125-
"Id": nicId,
132+
"Id": nicId,
133+
"deleteOption": "Delete",
126134
},
127135
},
128136
},

0 commit comments

Comments
 (0)