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

ipaddress gem requirement missing from readme #128

Closed
rski opened this issue Nov 18, 2015 · 4 comments
Closed

ipaddress gem requirement missing from readme #128

rski opened this issue Nov 18, 2015 · 4 comments

Comments

@rski
Copy link
Member

rski commented Nov 18, 2015

As of this pull request, puppet-network uses ipaddress.

Not having it installed on the puppet agent will cause puppet agent -t to fail with an error like

Could not autoload puppet/type/network_config: cannot load such file -- ipaddress

running sudo gem install ipaddress on the agent fixes this.

This requirement is not reflected in the Readme. I will work on a pull request that will add to the README an explanation of this along with an example on how to install this gem using puppet.

@rski
Copy link
Member Author

rski commented Nov 18, 2015

Also, maybe it's required on the master too. I'll look into this a bit more and make sure I know what is needed where, but this gem is definitely required somewhere at some point for puppet-network to work.

@igalic
Copy link
Contributor

igalic commented Nov 18, 2015

that should probably have an autorequire
…but then again, an autorequire isn't gonna really install things…

but another question: how, exactly, do you run a puppet infra without the ipaddress gem??

@rski
Copy link
Member Author

rski commented Nov 18, 2015

@igalic well, we use the ipaddress gem now.

still, I'd like to be able to able to install ipaddress on a node using puppet rather than some other way before puppet runs. Or this could a horrible hack that should be avoided and the gem should be installed using some other way after all.

@rski
Copy link
Member Author

rski commented Nov 18, 2015

It doesn't look like what I described in #129 will work so I might just close that issue. However I think that the docs should explicitly say that the ipaddress gem is required.

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

No branches or pull requests

2 participants