[RFE] Ability to configure various libvirt VM parameters about oh-my-vagrant HOT 10 CLOSED

Patches definitely welcome.

As for RAM/CPU's, @goern started this patch in: #84 but then gave up. Please feel free to write it from scratch or fix his. It should be straightforward.

As for additional HDD's, I actually wrote this feature in vagrant-libvirt, so I'm happy to get it in OMV :) You can look at this: https://github.com/purpleidea/puppet-gluster/blob/master/vagrant/Vagrantfile#L483 for how to add disks, assuming the interface hasn't changed.

HTH,
James

from oh-my-vagrant.

PS: technically OMV works with virtualbox now too. So bonus points if your patch adds the virtualbox hooks, although I won't nack the patch if you don't have them.

from oh-my-vagrant.

TBH I did not use VirtualBox for like 7 years now and have no idea how stuff works there this days. Though this does not look like a bad idea to make RAM/CPU/HDD adjustments unified, I will poke around in that direction.

Patch in #84 looks reasonable (well, goern@f2186d5 at least), so I'll use it as a basis for my work, thanks James for pointing that out.

from oh-my-vagrant.

On Fri, May 29, 2015 at 12:27 PM, br0ziliy [email protected] wrote:

TBH I did not use VirtualBox for like 7 years now and have no idea how
stuff works there this days. Though this does not look like a bad idea to
make RAM/CPU/HDD adjustments unified, I will poke around in that direction.

I don't use virtualbox either. Libvirt works so much better! Feel free to
ignore those additions. We'll open an issue for that feature and have a
virtualbox user add that in later.

from oh-my-vagrant.

@br0ziliy ping, still planning on doing this?

Cheers!

from oh-my-vagrant.

Hey James,

Yes, sure - was just a bit busy during RH Summit times. I'm definitively on
it though - it's just the time that is the matter.

On Thu, Jul 2, 2015 at 12:54 PM, James [email protected] wrote:

@br0ziliy https://github.com/br0ziliy ping, still planning on doing
this?

Cheers!

—
Reply to this email directly or view it on GitHub
#92 (comment)
.

Vasiliy Kaygorodov
System Administrator

from oh-my-vagrant.

@br0ziliy @goern

Just added: 92dd418

This should meet everything but the disk configuration aspect. I'll close this ticket for now, until someone actually wants to submit a patch for the disk part.

Please test and confirm this works! I've also tagged this release as 0.0.16 and a COPR build exists too:
https://copr.fedoraproject.org/coprs/purpleidea/oh-my-vagrant/

Cheers!

from oh-my-vagrant.

cc @aweiteka @scollier
Please test... ^^^^

@flavio-fernandes
Can you confirm this works on virtualbox.

Thanks!

from oh-my-vagrant.

@purpleidea Thanks for implementing this - works like a charm (tested with vagrant-libvirt)

from oh-my-vagrant.

@br0ziliy YW! enjoy

from oh-my-vagrant.