boxen / puppet-virtualbox

MIT License
9 stars 60 forks source link

Update Virtualbox to 4.2.6 #1

Closed roderik closed 11 years ago

willdurand commented 11 years ago

:+1:

It would be nice to have a parametrized class though.

jgn commented 11 years ago

I've tried this with an install of the earlier version (4.1.20) and Puppet see that the prior version is "still installed" (it isn't), and VirtualBox doesn't get reinstalled with the new version.

$ sudo more /private/var/db/.puppet_pkgdmg_installed_VirtualBox
name: 'VirtualBox'
source: 'http://download.virtualbox.org/virtualbox/4.1.20/VirtualBox-4.1.20-80170-OSX.dmg'

Removing that file did get VirtualBox reinstalled with 4.2.6.

(I'm a little worried that this might be a chronic issue for boxen - the pkgdmg doesn't allow for "ensure: absent," so there doesn't seem to be an obvious way to get Puppet to believe that it isn't installed.)

This won't be a problem for people who are coming to boxen for the first time, but VirtualBox in particular is a moving target and needs to get installed at newer versions frequently (in my experience).

wfarr commented 11 years ago

FWIW, the appdmg and pkgdmg providers are from Puppet upstream (they're included in the default distribution).

If they're not cutting it, we can file bugs/PRs against puppetlabs/puppet to get them fixed.

On Wed, Feb 20, 2013 at 10:19 AM, John Norman notifications@github.comwrote:

I've tried this with an install of the earlier version (4.1.20) and Puppet see that the prior version is "still installed" (it isn't), and VirtualBox doesn't get reinstalled with the new version.

$ sudo more /private/var/db/.puppet_pkgdmg_installed_VirtualBox name: 'VirtualBox' source: ' http://download.virtualbox.org/virtualbox/4.1.20/VirtualBox-4.1.20-80170-OSX.dmg '

Removing that file did get VirtualBox reinstalled with 4.2.6.

(I'm a little worried that this might be a chronic issue for boxen - the pkgdmg doesn't allow for "ensure: absent," so there doesn't seem to be an obvious way to get Puppet to believe that it isn't installed.)

This won't be a problem for people who are coming to boxen for the first time, but VirtualBox in particular is a moving target and needs to get installed at newer versions frequently (in my experience).

— Reply to this email directly or view it on GitHubhttps://github.com/boxen/puppet-virtualbox/pull/1#issuecomment-13806372.

jgn commented 11 years ago

I can imagine why it's there upstream: There are a lot of weird ways to uninstall stuff (VirtualBox being a prime offender -- they just provide a script in the DMG they expect end users to run manually).

Think I'll search their Trac . . .

Here's the way I summed it up in a blog post:

http://7fff.com/2013/02/19/osx-puppet-and-boxen-and-installing-new-software-versions-from-dmgs-virtualbox-for-me/

wfarr commented 11 years ago

Merging this for now, but the "real" bug here is Puppet's upstream behavior around appdmg/pkgdmg.

jgn commented 11 years ago

Hey, awesome! I think it needs a new tag 1.0.1 . . .