From 1dc2ca352531e6727e525c8aff1f3f4c5626f17d Mon Sep 17 00:00:00 2001 From: toby Date: Fri, 26 Oct 2018 20:24:47 +0200 Subject: [PATCH] fixing bug in udev rule writing for mgmt/gw interfaces --- debian/wit-network-config.postinst | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/debian/wit-network-config.postinst b/debian/wit-network-config.postinst index 7b5d61c..3115f76 100755 --- a/debian/wit-network-config.postinst +++ b/debian/wit-network-config.postinst @@ -124,11 +124,11 @@ case "$1" in ## blow we deal with real physical interfaces ## it is crucial that this `ifmac` block is above the rest, since if no mac is returned it will skip the loop and prevent the package install to fail - if [[ $if = mgmt ]] || [[ $if = mgmtgw ]] || [[ $if = ipmigw ]] + if [[ $ifname = mgmt1 ]] || [[ $if = mgmtgw1 ]] || [[ $if = ipmigw1 ]] ## we only support a single interface of these types and they are handled slightly differently in DNS, hence the workaround then - ifmac=$(dig_txt mac.${if}.${HOSTNAME}) || continue ## at this point skip the rest of the loop for interfaces that do not have a mac defined (those are not physical) + ifmac=$(dig_txt mac.${if}.${HOSTNAME}) || continue ## at this point skip the rest of the loop for interfaces that do not have a mac defined (those are basically not configured) else - ifmac=$(dig_txt mac.${ifname}.${HOSTNAME}) || continue ## at this point skip the rest of the loop for interfaces that do not have a mac defined (those are not physical) + ifmac=$(dig_txt mac.${ifname}.${HOSTNAME}) || continue ## at this point skip the rest of the loop for interfaces that do not have a mac defined fi echo 'SUBSYSTEM=="net", ACTION=="add", DRIVERS=="?*", ATTR{address}=="'${ifmac}'", ATTR{type}=="1", NAME="'${ifname}'"' >>$UDEVCONFIG