The call to PlannedNode.spent() *must* always occur

- They have been removed from the pendingLaunches, so once we leave this method we have no reference to them
- Arguably the call to spent should take place when removing from the pendingLaunches, but that would change
  the method contract

(cherry picked from commit 252e1296)

core/src/main/java/hudson/slaves/NodeProvisioner.java

@@ -250,9 +250,9 @@ public class NodeProvisioner {
                             for (CloudProvisioningListener cl : CloudProvisioningListener.all()) {
                                 cl.onFailure(f, e);
                             }
+                        } finally {
+                            f.spent();
                         }
-
-                        f.spent();
                     }
 
                     float plannedCapacity = plannedCapacitySnapshot;