Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: engine_page: Display correctly proto for Manage by
......................................................................

engine_page: Display correctly proto for Manage by

On status page, we have the "Manage by" string to
show to users which oVirt Engine the node is registered.
This patch will add a validation which protocol it used during
the registration (http or https) and put it in the display.

Change-Id: Iecfc53bab7e5f6ab226ded7f00bd6d140b6846b1
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1065401
Signed-off-by: Douglas Schilling Landgraf <dougsl...@redhat.com>
---
M src/engine_page.py
1 file changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node-plugin-vdsm 
refs/changes/10/26410/1

diff --git a/src/engine_page.py b/src/engine_page.py
index 1525f57..bd4a364 100644
--- a/src/engine_page.py
+++ b/src/engine_page.py
@@ -59,7 +59,12 @@
 
     if cfg["server"] is not None:
         server_url = [unicode(info) for info in [cfg["server"], cfg["port"]] 
if info]
-        engine_data = "oVirt Engine http://%s"; % ":".join(server_url)
+        port, sslPort = compatiblePort(cfg["port"])
+        if sslPort:
+            proto = "https"
+        else:
+            proto = "http"
+        engine_data = "oVirt Engine %s://%s" % (proto,":".join(server_url))
 
     mgmt = Management()
     mgmt.update(engine_data, mgmtIface, None)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iecfc53bab7e5f6ab226ded7f00bd6d140b6846b1
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to