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

added local yum repo hosting on puppet server when started as centos #6

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

Conversation

paulczar
Copy link

for Centos hosts can put RPM files into repository/yum/ and then run vagrant provision puppet to which will kick off a createrepo to add the RPM into the local repo. centos clients get a sandbox.repo file to auto-point to the yum repo.

also flipped the vagrantfile into a symlink so can easily flip between OS choices.

@@ -101,7 +103,7 @@ be picked up immediately.
To have your module actually applied to one or more of the nodes, edit the
`nodes.pp` file and include your classes...that's it!

Check Your Handiwork
Check Your Handywork
Copy link
Contributor

Choose a reason for hiding this comment

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

Heh - the original was correct here as well. You picked an "archaic variant" to change it to. This I find amusing! :)

@paulczar
Copy link
Author

Handiwork vs Handywork .. . I guess that's the usual conflict between Aussie English vs U.S. English ... you're lucky I didn't change all your Z's to S's and add U's to a bunch of words :)

@phinze
Copy link
Contributor

phinze commented Mar 27, 2013

Ha! I hadn't thought about the Aussie angle. :)

I'm +1 for merging this. @elasticdog you wanna weigh in?

@elasticdog
Copy link
Owner

Sorry for being late to the party on this one...I like the idea, but have some stylistic things that I'd like to clean up. I'd also avoid the symlinked Vagrantfile as editing those is a standard affair when using Vagrant, I'd rather not maintain them when there are version bumps (centos63 -> centos64, etc.), and people's base boxes aren't necessarily going to be named the same.

Give me a bit of time to go through all the commits, and I'll merge once I've cleaned things up. Thanks for the contribution!

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.

3 participants