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

Templated network configuration #56

Merged

Conversation

michaeltchapman
Copy link

Remove puppet-network and dependencies, move networking definition into the preseed late command to get a 'free' reboot (this avoids relying on networking restart doing the right thing, which is not a given) and use templating to support bonding and other advanced configuration.

https://bugs.launchpad.net/openstack-cisco/folsom/+bug/1159597

Michael Chapman added 2 commits March 28, 2013 04:37
…to the preseed late command to get a 'free' reboot and use templating to support bonding and other advanced configuration. https://bugs.launchpad.net/openstack-cisco/folsom/+bug/1159597
…ope, leading to empty strings in the preseed.
@robertstarmer
Copy link
Contributor

Builds as well.

👍 👍

On Mar 27, 2013, at 9:44 PM, Michael Chapman <notifications@github.commailto:notifications@github.com>
wrote:

Remove puppet-network and dependencies, move networking definition into the preseed late command to get a 'free' reboot (this avoids relying on networking restart doing the right thing, which is not a given) and use templating to support bonding and other advanced configuration.

https://bugs.launchpad.net/openstack-cisco/folsom/+bug/1159597


You can merge this Pull Request by running

git pull https://github.com/michaeltchapman/folsom-manifests dt-multi-node-interfaces

Or view, comment on, or merge it at:

#56

Commit Summary

File Changes

Patch Links:

@iawells
Copy link

iawells commented Mar 28, 2013

+1; let's get it in and everyone hammering on it...

iawells added a commit that referenced this pull request Mar 28, 2013
@iawells iawells merged commit 640faf3 into CiscoSystems:multi-node Mar 28, 2013
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.

None yet

3 participants