Daniel Erez has submitted this change and it was merged. Change subject: webadmin: avoid disconnect server on attach domain ......................................................................
webadmin: avoid disconnect server on attach domain During the "Import Domain" flow (mostly driven by the UI) the execution of AttachStorageDomainToPool and DisconnectStorageServerConnection were involved a race that in some cases resulted in the attach operation to fail (since the server was disconnected). Now after attachStorageToDataCenter we avoid postImportFileStorage (since the storage should remain connected), and we go straight to onFinish because any issue in AttachStorageDomainToPool is handled by the default error dialog. This fix is just removing the race between the calls and it's not intended to address other bugs that are present in the flow (for example what happens if the user closes the web browser during the chain of execution? who is cleaning up the storage connections? etc.) Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=994463 Change-Id: Ie880c25014c5292ad64ed94ad9c126fdd048d7bc Signed-off-by: Federico Simoncelli <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18649 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie880c25014c5292ad64ed94ad9c126fdd048d7bc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
