search

gposton / vagrant-hiera

MIT License
21 stars 12 forks source link

Suggestion: no need to install puppet/hiera #9

Open ghost opened 11 years ago

ghost commented 11 years ago

I would like to use Ubuntu, but the client uses CentOS. I'm using a Cent vbox with puppet 3 included: http://packages.vstone.eu/vagrant-boxes/centos/6.3/

I wish this gem didn't try to install puppet with apt and dpkg. I ended up having to ln -s /bin/true /bin/dpkg and ln -s /bin/true /bin/apt-get, and mkdir -p /etc/apt/sources.list.d, to get vagrant-hiera to stop failing my vagrant up because it couldn't use apt to install its own puppet...

I would like to request that using debianisms to install puppet be made optional so I don't have to do silly things like symlink a different distro's commands to /bin/true.

joerayme commented 11 years ago

I'd second this - I'm having issues in debian land with the apt packages conflicting with gems installed by default on the box. I think it would be better if it behaved like the Vagrant provisioners: check if hiera exists, and if not, throw an error. See https://github.com/mitchellh/vagrant/blob/master/plugins/provisioners/puppet/provisioner/puppet.rb

gposton commented 11 years ago

I agree that this should be taken out. I just don't have time to make and test the change right now. Feel free to update and submit a pull request, otherwise I'll get around to it soon. Sorry :(