Patches are transferred and uploaded to the subclouds prematurely during patch orchestration

[Bo Yuan Chang]

Brief Description
-----------------
Patches are uploaded and started in the subcloud before checking if the subcloud is patchable which prevents subsequent patch orchestration.

Severity
--------
Major

Steps to Reproduce
------------------
Start a patch orchestration while one or more subclouds have a management affecting alarm.

Expected Behavior
------------------
The orchestrated patching fails for the subcloud(s) with the management affecting alarm. However, there's no patching in progress alarm on the subcloud after the patch strategy is complete and removed.

Actual Behavior
----------------
Subclouds with the management affecting alarm end up with an additional alarm - the patching in progress alarm which is also management affecting which prevents subsequent patch orchestration on these subclouds.

Reproducibility
---------------
Readily reproducible.

System Configuration
--------------------
Distributed Cloud system

Branch/Pull Time/Commit
-----------------------
December 12, 2021 master build

Last Pass
---------
Not sure if this test case has been covered before.

Timestamp/Logs
--------------
Issue is readily reproducible

Test Activity
-------------
Developer Testing

Workaround
----------
N/A

[Bo Yuan Chang (cby19961020)]

Gerrit review: https://review.opendev.org/c/starlingx/distcloud/+/823716
Commit: https://opendev.org/starlingx/distcloud/commit/e3a19ade738a151ad3d0a1ed14ccfba3ec6c55a8

[Ghada Khalil (gkhalil)]

@Bo, in the future, please make sure to link the LP in the commit msg using Closes-Bug