Closes-Bug: #1541111 : Horizon Console not seen in SM-Provisioned HA …

…setup Novncproxy ip address and port not set correctly ip should be external/internal vip if available. frontend port must be used: 6080 Change-Id: Id850479240922f0119ecf4829d3bd22ec5a0cc52
Juniper · Feb 3, 2016 · 856267c · 856267c
1 parent 2b356e0
commit 856267c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 11 deletions.
diff --git a/contrail/environment/modules/contrail/manifests/contrail_openstack.pp b/contrail/environment/modules/contrail/manifests/contrail_openstack.pp
@@ -77,20 +77,16 @@
 )  {
     # Main code for class
 
-    #slect the novncproxy based on presence of internal_vip
-    if ($internal_vip != ''){
-        $novncproxy_port = '6999'
-        $vnc_proxy_host = $host_control_ip
-    } else {
-        $novncproxy_port = '5999'
-        $vnc_proxy_host = $openstack_mgmt_ip
-    }
+    #select the novncproxy based on presence of internal_vip
 
     if ($external_vip != '') {
+        $vnc_base_url_port = '6080'
         $vnc_base_url_ip = $external_vip
     } elsif ($internal_vip != '' ) {
+        $vnc_base_url_port = '6080'
         $vnc_base_url_ip = $internal_vip
     } else {
+        $vnc_base_url_port = '5999'
         $vnc_base_url_ip = $openstack_mgmt_ip
     }
 
@@ -110,7 +106,7 @@
     class {'::contrail::exec_create_ec2rc_file':}
 
     $nova_params =  {
-      'DEFAULT/novncproxy_base_url' => { value => "http://${vnc_base_url_ip}:${novncproxy_port}/vnc_auto.html"},
+      'DEFAULT/novncproxy_base_url' => { value => "http://${vnc_base_url_ip}:${vnc_base_url_port}/vnc_auto.html"},
       'DEFAULT/ec2_private_dns_show_ip' => { value => 'False' },
     }
     create_resources(nova_config,$nova_params, {} )

diff --git a/contrail/environment/modules/openstack/manifests/common/contrail/nova.pp b/contrail/environment/modules/openstack/manifests/common/contrail/nova.pp
@@ -53,7 +53,7 @@
 
   if ($internal_vip != "" and $internal_vip != undef) {
       class { '::nova::vncproxy':
-	host    => $::openstack::config::controller_address_api,
+	host    => $::contrail::params::host_ip,
 	enabled => $is_controller,
         port => '6999',
       }

diff --git a/contrail/environment/modules/openstack/manifests/common/nova.pp b/contrail/environment/modules/openstack/manifests/common/nova.pp
@@ -90,7 +90,7 @@
     }
 
     class { '::nova::vncproxy':
-      host    => $::openstack::config::controller_address_api,
+      host    => $::contrail::params::host_ip,
       enabled => $is_controller,
       port => '6999',
     }