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.

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

Prevent Cloud-init erasing LINKDELAY #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dougszumski
Copy link
Member

@dougszumski dougszumski commented Aug 6, 2019

Cloud-init uses default generated network config to try and configure
a link for DHCP. This overwrites the LINKDELAY setting, which is required
in some cases to bring up the link.

This prevents the configuration of a udev rule by cloud init and the addition of the MAC address to the ifcfg script for the active interface. DNS still appears to be configured, but is missing the 'configured by Cloud init header'. None of these issues appear to prevent the image booting, running cloud-init and enabling the user to log in on a test node connected to a single network.

Cloud-init uses default generated network config to try and configure
a link for DHCP. This overwrites the LINKDELAY setting, which is required
in some cases to bring up the link.
@@ -8,3 +8,5 @@ set -o pipefail

# Add a LINKDELAY value to /etc/sysconfig/network
echo -e "# Allow extra time for link up\nLINKDELAY=${CENTOS_LINKUP_EXTRA_SECS}" >> /etc/sysconfig/network
# Prevent Cloudinit writing over LINKDELAY setting
echo -e "network: {config: disabled}" >> /etc/cloud/cloud.cfg.d/99-disable-custom-net-conf.cfg

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like quite a drastic thing to always do. Network bootstrapping is a big part of cloud-init. Would it not also break network configuration provided via config-drive?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants