Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[202111][Arista] Fix arista-net initramfs renaming logic #10625

Merged
merged 2 commits into from
Apr 20, 2022
+31 −15
Merged

[202111][Arista] Fix arista-net initramfs renaming logic #10625

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4bb3a27
[Arista] rename management interface in initrd (#9856)
Staphylo Mar 21, 2022
d01d88c
[Arista] Fix arista-net initramfs hook
Staphylo Apr 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
[Arista] Fix arista-net initramfs hook 
The interface renaming logic fails if one interface is missing.
Because of the `set -e` the whole initramfs hook would abort early on
error.
This change fixes the current behavior to make sure missing interfaces
are properly skipped and ensure existing interface are renamed.
@Staphylo
Staphylo committed Apr 20, 2022
commit d01d88c1e523178106c0dd4814495da4803f00ba
4 changes: 3 additions & 1 deletion files/initramfs-tools/arista-net
View file
Original file line number Diff line number Diff line change
@@ -47,7 +47,9 @@ arista_net_rename() {
local new_name="$2"
local from_name="$3"
devname=$(arista_net_devname "$device_path" "$from_name")
[ -n "$devname" ] && ip link set "$devname" name "$new_name"
if [ -n "$devname" ]; then
ip link set "$devname" name "$new_name"
fi
}

# Sets the MAC address to the value passed by Aboot through /proc/cmdline