Skip to content

Commit

Permalink
Clean up n-net remnants
Browse files Browse the repository at this point in the history
In I90316208d1af42c1659d3bee386f95e38aaf2c56 support for nova-network
was removed, but some bits remained, fix this up.

Change-Id: Iba7e1785fd0bdf0a6e94e5e03438fc7634621e49
  • Loading branch information
osfrickler committed Aug 21, 2022
1 parent 90e5479 commit ca5f919
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 27 deletions.
2 changes: 0 additions & 2 deletions files/debs/nova
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
conntrack
curl
dnsmasq-base
dnsmasq-utils # for dhcp_release
ebtables
genisoimage # required for config_drive
iptables
Expand Down
2 changes: 0 additions & 2 deletions files/rpms-suse/nova
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
cdrkit-cdrtools-compat # dist:sle12
conntrack-tools
curl
dnsmasq
dnsmasq-utils # dist:opensuse-12.3,opensuse-13.1
ebtables
iptables
iputils
Expand Down
2 changes: 0 additions & 2 deletions files/rpms/nova
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
conntrack-tools
curl
dnsmasq # for q-dhcp
dnsmasq-utils # for dhcp_release
ebtables
genisoimage # not:rhel9 required for config_drive
iptables
Expand Down
14 changes: 0 additions & 14 deletions lib/nova
Original file line number Diff line number Diff line change
Expand Up @@ -107,20 +107,6 @@ NOVA_FILTERS="AvailabilityZoneFilter,ComputeFilter,ComputeCapabilitiesFilter,Ima

QEMU_CONF=/etc/libvirt/qemu.conf

# Set default defaults here as some hypervisor drivers override these
PUBLIC_INTERFACE_DEFAULT=br100
# Set ``GUEST_INTERFACE_DEFAULT`` to some interface on the box so that
# the default isn't completely crazy. This will match ``eth*``, ``em*``, or
# the new ``p*`` interfaces, then basically picks the first
# alphabetically. It's probably wrong, however it's less wrong than
# always using ``eth0`` which doesn't exist on new Linux distros at all.
GUEST_INTERFACE_DEFAULT=$(ip link \
| grep 'state UP' \
| awk '{print $2}' \
| sed 's/://' \
| grep ^[ep] \
| head -1)

# ``NOVA_VNC_ENABLED`` can be used to forcibly enable VNC configuration.
# In multi-node setups allows compute hosts to not run ``n-novnc``.
NOVA_VNC_ENABLED=$(trueorfalse False NOVA_VNC_ENABLED)
Expand Down
10 changes: 3 additions & 7 deletions stack.sh
Original file line number Diff line number Diff line change
Expand Up @@ -1152,7 +1152,8 @@ fi
# ----

if is_service_enabled q-dhcp; then
# Delete traces of nova networks from prior runs
# TODO(frickler): These are remnants from n-net, check which parts are really
# still needed for Neutron.
# Do not kill any dnsmasq instance spawned by NetworkManager
netman_pid=$(pidof NetworkManager || true)
if [ -z "$netman_pid" ]; then
Expand Down Expand Up @@ -1212,12 +1213,7 @@ if is_service_enabled nova; then
echo_summary "Configuring Nova"
init_nova

# Additional Nova configuration that is dependent on other services
# TODO(stephenfin): Is it possible for neutron to *not* be enabled now? If
# not, remove the if here
if is_service_enabled neutron; then
async_runfunc configure_neutron_nova
fi
async_runfunc configure_neutron_nova
fi


Expand Down

0 comments on commit ca5f919

Please sign in to comment.