Merge pull request #20841 from ashley-cui/wsldeadlock

Fix locking error in WSL machine rm -f
This commit is contained in:
openshift-merge-bot[bot]
2023-11-30 14:00:35 +00:00
committed by GitHub

View File

@ -1591,9 +1591,6 @@ func readWinProxyTid(v *MachineVM) (uint32, uint32, string, error) {
func (v *MachineVM) Remove(name string, opts machine.RemoveOptions) (string, func() error, error) {
var files []string
v.lock.Lock()
defer v.lock.Unlock()
if v.isRunning() {
if !opts.Force {
return "", nil, &machine.ErrVMRunningCannotDestroyed{Name: v.Name}
@ -1603,6 +1600,9 @@ func (v *MachineVM) Remove(name string, opts machine.RemoveOptions) (string, fun
}
}
v.lock.Lock()
defer v.lock.Unlock()
// Collect all the files that need to be destroyed
if !opts.SaveKeys {
files = append(files, v.IdentityPath, v.IdentityPath+".pub")