resources: virt: Remove unnecessary early exit from CheckApply

I don't think this early exit is necessary any more, since the main
CheckApply function really just spawns out to the different sub workers
which all individually check the apply variable.

If I'm wrong, we can revert this. It was @roidelapluie that noticed the
check here to begin with.
This commit is contained in:
James Shubin
2017-02-25 21:29:08 -05:00
parent 79923a939b
commit c480bd94db

View File

@@ -836,11 +836,6 @@ func (obj *VirtRes) CheckApply(apply bool) (bool, error) {
// FIXME: should we wait to ensure machine is booted before continuing?
// it may be useful to wait for guest agent to hotplug some ram or cpu!
if !apply {
return false, nil
}
// remaining apply portion
// mem & cpu checks...
if !obj.absent {
if c, err := obj.attrCheckApply(apply, dom); err != nil {