Merge pull request #3964 from brendandburns/fixer

Reverse the order of detaching and removing the pd disk.
This commit is contained in:
Satnam Singh 2015-01-29 16:47:27 -08:00
commit 08ce8e80ad

View File

@ -234,9 +234,6 @@ func (pd *gcePersistentDisk) TearDown() error {
return err
}
refCount--
if err := os.RemoveAll(pd.GetPath()); err != nil {
return err
}
// If refCount is 1, then all bind mounts have been removed, and the
// remaining reference is the global mount. It is safe to detach.
if refCount == 1 {
@ -244,5 +241,8 @@ func (pd *gcePersistentDisk) TearDown() error {
return err
}
}
if err := os.RemoveAll(pd.GetPath()); err != nil {
return err
}
return nil
}