Skip to content

Commit

Permalink
overlay/fcos: pin legacy ifnames on existing machines
Browse files Browse the repository at this point in the history
This introduces a `coreos-keep-legacy-ifnames` service to pin
network interface names to the legacy scheme on machines
where `99-default.link` is missing.
This is in order to ensure that those machines do not jump
into the new scheme via auto-updates as soon as FCOS starts
shipping such unit in the OS.
  • Loading branch information
lucab committed Jun 24, 2020
1 parent a353909 commit 70366d3
Show file tree
Hide file tree
Showing 3 changed files with 53 additions and 0 deletions.
3 changes: 3 additions & 0 deletions overlay.d/15fcos/usr/lib/systemd/system-preset/45-fcos.preset
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,6 @@ enable fedora-coreos-pinger.service
# Provide information if no ignition is provided
enable coreos-check-ignition-config.service
enable coreos-check-ssh-keys.service
# Pinning script for legacy interface names, see
# https://github.com/coreos/fedora-coreos-tracker/issues/484
enable coreos-keep-legacy-ifnames.service
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
[Unit]
Description=CoreOS Keep Legacy Network Interface Names
Documentation=https://github.com/coreos/fedora-coreos-tracker/issues/484
RequiresMountsFor=/boot
ConditionPathExists=!/usr/lib/systemd/network/99-default.link
# NOTE: we do not conditionalize on ConditionKernelCommandLine= here because
# kernel args might have been tweaked manually just for this boot; we still
# want to change the BLS entries in that case.

[Service]
Type=oneshot
RemainAfterExit=yes
ExecStart=/usr/libexec/coreos-keep-legacy-ifnames

[Install]
WantedBy=multi-user.target
34 changes: 34 additions & 0 deletions overlay.d/15fcos/usr/libexec/coreos-keep-legacy-ifnames
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
#!/bin/bash
# This is a pinning script to keep legacy ifnames on machines that were
# installed on an OS version where `99-default.link` was missing due to
# https://github.com/coreos/fedora-coreos-tracker/issues/484

set -euo pipefail

injected=0

if test -e /usr/lib/systemd/network/99-default.link; then
echo "unit /usr/lib/systemd/network/99-default.link found, no actions performed"
exit 0;
fi

for f in /boot/loader/entries/*.conf; do
options=$(grep '^options ' "$f" | cut -f2- -d' ')

# If it is already specified, do not touch whatever value is there.
# This is in order to avoid messing with user customization, and to
# make the logic idempotent.
if grep -q "net.ifnames" <<< "$options"; then
continue
fi

# Otherwise, make sure we stay on legacy ifnames.
sed -e "/^options / s/$/ net.ifnames=0/" -i "$f"
echo "$(basename "$f"): injected net.ifnames=0"
injected=1
done

if [[ $injected -eq 0 ]]; then
echo "existing net.ifnames karg detected, no actions performed"
fi

0 comments on commit 70366d3

Please sign in to comment.