Lior Vernia has submitted this change and it was merged.

Change subject: webadmin: Warn user if KeystoneAuthUrl wasn't configured
......................................................................


webadmin: Warn user if KeystoneAuthUrl wasn't configured

When testing the connection of a provider that was marked as "Requires
Authentication", I added a check whether the KeystoneAuthUrl was
configured in case the test failed. If it wasn't, the user is hinted
that this might be the cause of the failure.

Change-Id: Ie7c0a8586b4845efbee0d599b31d92ce8158d336
Bug-Url: https://bugzilla.redhat.com/1017538
Signed-off-by: Lior Vernia <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java
M 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ProviderPopupView.ui.xml
4 files changed, 16 insertions(+), 7 deletions(-)

Approvals:
  Lior Vernia: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/25720
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie7c0a8586b4845efbee0d599b31d92ce8158d336
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Lior Vernia <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to