[openstack] add block_device_mapping_v2

[tnaoto]

fix block_device_mapping_v2

http://docs.openstack.org/api/openstack-compute/2/content/POST_block_device_mapping_v2_createServer__v2__tenant_id__servers_ext-os-block-device-mapping-v2-boot.html

[geemus]

@mwhagedorn could you review? Thanks!

[mwhagedorn]

@geemus.. will do
On Jul 24, 2014, at 12:53 PM, Wesley Beary notifications@github.com wrote:

@mwhagedorn could you review? Thanks!

—
Reply to this email directly or view it on GitHub.

[tnaoto]

@geemus @mwhagedorn
Hello!
I change this PR.
[block_device_mapping] "and" [block_device_mapping_v2]

How about this PR?

[geemus]

@mdarby @smashwilson - could one of you review? Thanks!

[smashwilson]

💄 The indentation here is rather un-Ruby-like, and (in my opinion) obscures the structure somewhat. Also, you're missing a hash inside the :block_device_mapping_v2 array, and technically :image_ref is required, too. How about:

vm = compute_client.servers.create(
  :name => name,
  :flavor_ref => flavor,
  :image_ref => image,
  :block_device_mapping_v2 => [
    {
      :device_name => "/dev/sda1",
      :source_type => "volume",
      :destination_type => "volume",
      :delete_on_termination => false,
      :uuid => cinder_uddi,
      :boot_index => 0
    }
  ]
)

[tnaoto]

Thank you.
I was again as pointed out .

[tnaoto]

@smashwilson Please review, I have fixed it.

[tnaoto]

@geemus How about this PR?

[mdarby]

👍

[geemus]

Thanks!