Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: Show correct error message when can't connect to VM
......................................................................
webadmin: Show correct error message when can't connect to VM
When it's not possible to connect to a VM cause of no available protocol
show a correct error message that does not contain the protocol name (leads
to NPE)
Change-Id: I8725429705e49cf506a80495fc510c169a515d60
Signed-off-by: Tal Nisan <tnisan(a)redhat.com>
Bug-Url:
https://bugzilla.redhat.com/1295728
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/VmConsolesImpl.java
M
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java
2 files changed, 4 insertions(+), 2 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, approved
Tal Nisan: Verified
Jenkins CI: Passed CI tests
Michal Skrivanek: Looks good to me, but someone else must approve
Arik Hadas: Looks good to me, but someone else must approve
--
To view, visit
https://gerrit.ovirt.org/51372
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8725429705e49cf506a80495fc510c169a515d60
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>