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

Fix new-style device naming from Network Manager on RHEL/CentOS 7 #617

Merged
merged 1 commit into from
Jul 9, 2016

Conversation

legal90
Copy link
Contributor

@legal90 legal90 commented Jun 20, 2016

Fixes #388 and Parallels/vagrant-parallels#265
Details about the root of issue: Parallels/vagrant-parallels#265 (comment)

The solution has been taken from Boxcutter: boxcutter/centos@be4174e

It might look not very clear, so any other solutions are welcome. The current implementation does the following (only on RHEL/CentOS 7):

  • Cleans up the NetworkManager state and settings before the box packaging
  • Appends /etc/rc.d/rc.local by the script, which fixes names of network devices in NM configuration
  • Being executed only on the first boot, the script mentioned above removes itself from the /etc/rc.d/rc.local.

I've verified it with centos-7.2-x86_64.json template and parallels provider

@yankcrime
Copy link

👍 For this - have tested using VMware Fusion and it solves the issue as described.

@nabheet
Copy link

nabheet commented Jun 27, 2016

I have validated that this fix resolves the issue that I discovered with vagrant getting stuck on setting hostname while bringing up a CentOS 7.2 VM using the Parallels provider.

Basically, I can confirm that this is true: I've verified it with centos-7.2-x86_64.json template and parallels provider.

Hopefully this will be merged soon.

@legal90
Copy link
Contributor Author

legal90 commented Jun 28, 2016

@cheeseplus @rickard-von-essen @yzl We need your LGTMs to merge this patch :)

@cheeseplus
Copy link
Contributor

+1

@cheeseplus cheeseplus merged commit 2bd5061 into chef:master Jul 9, 2016
@legal90 legal90 deleted the fix-nm-device branch July 12, 2016 09:11
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.

opscode-centos-7.1 Virtualbox box not able to NFS mount
5 participants