Closed sermakov-orion closed 10 months ago
LGTM though i'm not sure about the completeness of the changes that need to go to the actual host, @almusil wdyt?
@michalskrivanek let me add bit more clarity on why I decided not to go to the actual host in case when there no changes other than labels. The HostSetupNetworks VDSM command is invoked with parameters created by the HostSetupNetworksCommand.createSetupNetworksParameters method. And if you walk through this and underlying methods you may notice that they use everything from getParameters() but not getParametes().getLables() and getParameters().getRemovedLables(). So, It looks that HostSetupNetworks does not require any changes in labels.
Hi,
thank you for the patch. Please add an explanation to the commit message so it is clear in the history why this change was required.
Hi @almusil. I did this change as well as other mentioned here. Could you please do another round of the review?
/ost
/ost basic-suite-master el9stream
Signed-off-by: Stepan Ermakov sermakov@orionsoft.ru
Changes introduced with this PR
This PR fixes the following issue: an error occurs while assigning labels on host network interfaces: "Error while executing action HostSetupNetworks: Unexpected exception" Steps to reproduce:
Expected behavior The changes applied with no errors Current behavior The error message is displayed: "Error while executing action HostSetupNetworks: Unexpected exception". The new label was not added to the host network.
Background
if (noChangesDetected())
statement).if (hasNetworkChanges())
that verifies whether the HostSetupNetworks call to the host VDSM is needed or not.Are you the owner of the code you are sending in, or do you have permission of the owner?
y