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

(QENG-760) support for additional disk for vagrant #303

Merged
merged 1 commit into from
Jun 13, 2014

Conversation

anodelman
Copy link
Contributor

  • if a given vagrant box has a disk_path then either create a new vmdk
    disk or use the provided disk
  • defaults to 50 x 1024 bytes if the disk needs to created

@puppetlabs-jenkins
Copy link
Contributor

💚 Test passed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker%20Combined%20Smoketest/98/

v_file << " vb.name = '#{host.name}'\n"
unless File.exist?(host['disk_path'])
host['disk_path'] = File.join(host['disk_path'], "#{host.name}.vmdk")
v_file << " vb.customize ['createhd', '--filename', '#{host['disk_path']}', '--size', #{host['disk_size'] ||= 500 * 1024}, '--format', 'vmdk']\n"
Copy link
Contributor

Choose a reason for hiding this comment

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

According to the documentation, size is in MB. I recommend changing 500 to 5.

VBoxManage createhd         --filename <filename>
                            --size <megabytes>
                            [--format VDI|VMDK|VHD] (default: VDI)
                            [--variant Standard,Fixed,Split2G,Stream,ESX]

Copy link
Contributor

Choose a reason for hiding this comment

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

I am not sure it matters, but format types are all-caps in the doc vs. lowercase here (vmdk).

@colinPL
Copy link
Contributor

colinPL commented Jun 12, 2014

+1, except I have not run this against vagrant+vb. I have run this PR against the vcloud hypervisor.

- if a given vagrant box has a disk_path then either create a new vmdk
  disk or use the provided disk
- defaults to 50 x 1024 bytes if the disk needs to created
@anodelman
Copy link
Contributor Author

Update to create 5 Gb disk instead of 500. I have tested this with vagrant and seen that it creates and attaches an additional disk.

@puppetlabs-jenkins
Copy link
Contributor

🔴 Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker%20Combined%20Smoketest/103/

colinPL added a commit that referenced this pull request Jun 13, 2014
(QENG-760) support for additional disk for vagrant
@colinPL colinPL merged commit 1a7942c into voxpupuli:master Jun 13, 2014
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.

None yet

3 participants