opendev/system-config
Code Issues Proposed changes

Move static definition of forwarding.conf later in script

Browse Source 
While the original implementation may have been to save the nameservers
given by the provider, we now just use a static nameserver.  As such we
can define the contents of forwarding.conf later and help avoid any
potential race conditions, such as that in bug 1373505.

Change-Id: I921cca03a6915f1e7b471d1bf90b82abcb1a0606
related-bug: 1373505
This commit is contained in:
Bob Ball 2014-09-24 17:50:50 +01:00 committed by Clark Boylan
parent 0b4568c7c3
commit e4237ccc79
1 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
modules/openstack_project/files/nodepool/scripts/prepare_node.sh
View File

@ -23,13 +23,6 @@ PYTHON3=${4:-false}
PYPY=${5:-false}
ALL_MYSQL_PRIVS=${6:-false}
# Save the nameservers configured by our provider.
cat >/tmp/forwarding.conf <<EOF
forward-zone:
name: "."
forward-addr: 8.8.8.8
EOF
sudo hostname $HOSTNAME
# Fedora image doesn't come with wget
if [ -f /usr/bin/yum ]; then
@ -48,9 +41,12 @@ else
-e "class {'openstack_project::single_use_slave': install_users => false, sudo => $SUDO, bare => $BARE, python3 => $PYTHON3, include_pypy => $PYPY, all_mysql_privs => $ALL_MYSQL_PRIVS, ssh_key => '$NODEPOOL_SSH_KEY', }"
fi
# The puppet modules should install unbound. Take the nameservers
# that we ended up with at boot and configure unbound to forward to
# them.
# The puppet modules should install unbound. Set up some nameservers.
cat >/tmp/forwarding.conf <<EOF
forward-zone:
name: "."
forward-addr: 8.8.8.8
EOF
sudo mv /tmp/forwarding.conf /etc/unbound/
sudo chown root:root /etc/unbound/forwarding.conf
sudo chmod a+r /etc/unbound/forwarding.conf