Moti Asayag has uploaded a new change for review. Change subject: engine: Reorder network removal from external provider ......................................................................
engine: Reorder network removal from external provider When an external network is removed from the system, it should be removed first from the external provider to prevent the disappear and reappear of the network in the UI. Change-Id: I6befd1d510073f8364155a6dd2fa8140e1fb989c Bug-Url: https://bugzilla.redhat.com/1076909 Signed-off-by: Moti Asayag <masa...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/RemoveNetworkCommand.java 1 file changed, 4 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/00/30200/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/RemoveNetworkCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/RemoveNetworkCommand.java index 1e26af6..2233176 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/RemoveNetworkCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/RemoveNetworkCommand.java @@ -48,6 +48,10 @@ protected void executeCommand() { setStoragePoolId(getNetwork().getDataCenterId()); + if (getParameters().isRemoveFromNetworkProvider() && getNetwork().isExternal()) { + removeExternalNetwork(); + } + TransactionSupport.executeInNewTransaction(new TransactionMethod<Void> () { @Override public Void runInTransaction() { @@ -58,10 +62,6 @@ return null; } }); - - if (getParameters().isRemoveFromNetworkProvider() && getNetwork().isExternal()) { - removeExternalNetwork(); - } setSucceeded(true); } -- To view, visit http://gerrit.ovirt.org/30200 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6befd1d510073f8364155a6dd2fa8140e1fb989c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masa...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
