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

Disable NetworkManager in RHEL for Bridge interfaces and some RHEL testing improvements #61

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

patdowney
Copy link

Started out wanting to be able to disable NetworkManager from managing the bridge interfaces. Turned into a yak.

  • Turns off NetworkManager management by default on RHEL

Also:

  • Adds epel repository for testing on centos as the lxc package isn't available in default distribution
  • Fixes up the syntax in the Gemfile and update the versions of the gems and retire the vagrant gem.
  • Adds centos targets in test-kitchen
  • Parameterises the rhel_bridge some more

@chrisroberts
Copy link
Contributor

👍 The centos networking was a super quick hack just to get something working, so this is awesome. I have an old bridger cookbook laying around that needs some love to bring it into this century, and would help clean this up and make it more robust. This is a great start getting it cleaned up!

If you have time to rebase this on develop branch and submit PR on that, that would be great. Otherwise I'll just do it locally and you'll see this closed without merge (just FYI).

Cheers!

* Specify Ruby version as 2.1.5
* Don't use vagrant gem anymore
* Use rubygems version of test-kitchen
LXC isn't included in the standard repositories
but it is included in EPEL, so we need to refer to it
so that we can download the LXC packages
Also replaces the hw-ops 12.04 (precise) box with the default 12.04
box from opscode as the hw-ops url no longer works.

centos-6.4 doesn't work properly as the version of dnsmasq that
comes with it has a bug
(https://www.redhat.com/archives/rdo-list/2014-December/msg00068.html)
Also tries to use consistent references to attributes,
no longer use `default_config`
NetworkManager wasn't playing nicely with the bridge
so this commit parameterises the choice and disables
it by default
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