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 <fsimo...@redhat.com> --- 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 <fsimo...@redhat.com> Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Ayal Baron <aba...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Liron Ar <lara...@redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
