Skip to content

Commit 6f9fc85

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Remove VNC server listen addresses seeting to IPv4 from base job"
2 parents df70dfa + b0b80d7 commit 6f9fc85

2 files changed

Lines changed: 2 additions & 6 deletions

File tree

.zuul.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -382,8 +382,6 @@
382382
SWIFT_HASH: 1234123412341234
383383
DEBUG_LIBVIRT_COREDUMPS: true
384384
NOVA_VNC_ENABLED: true
385-
VNCSERVER_LISTEN: 0.0.0.0
386-
VNCSERVER_PROXYCLIENT_ADDRESS: $HOST_IP
387385
devstack_local_conf:
388386
post-config:
389387
$NEUTRON_CONF:
@@ -479,8 +477,6 @@
479477
GLANCE_HOSTPORT: "{{ hostvars['controller']['nodepool']['private_ipv4'] }}:9292"
480478
Q_HOST: "{{ hostvars['controller']['nodepool']['private_ipv4'] }}"
481479
NOVA_VNC_ENABLED: true
482-
VNCSERVER_LISTEN: 0.0.0.0
483-
VNCSERVER_PROXYCLIENT_ADDRESS: $HOST_IP
484480

485481
- job:
486482
name: devstack-ipv6

lib/nova

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -601,8 +601,8 @@ function configure_console_compute {
601601
if is_service_enabled n-novnc || is_service_enabled n-xvnc || [ "$NOVA_VNC_ENABLED" != False ]; then
602602
# Address on which instance vncservers will listen on compute hosts.
603603
# For multi-host, this should be the management ip of the compute host.
604-
VNCSERVER_LISTEN=${VNCSERVER_LISTEN=$NOVA_SERVICE_LOCAL_HOST}
605-
VNCSERVER_PROXYCLIENT_ADDRESS=${VNCSERVER_PROXYCLIENT_ADDRESS=$NOVA_SERVICE_LOCAL_HOST}
604+
VNCSERVER_LISTEN=${VNCSERVER_LISTEN:-$NOVA_SERVICE_LOCAL_HOST}
605+
VNCSERVER_PROXYCLIENT_ADDRESS=${VNCSERVER_PROXYCLIENT_ADDRESS:-$NOVA_SERVICE_LOCAL_HOST}
606606
iniset $NOVA_CPU_CONF vnc server_listen "$VNCSERVER_LISTEN"
607607
iniset $NOVA_CPU_CONF vnc server_proxyclient_address "$VNCSERVER_PROXYCLIENT_ADDRESS"
608608
else

0 commit comments

Comments
 (0)