Project

General

Profile

Bug #13356

OpenStack volumes API v2 needs :name

Added by Daniel Lobato Garcia over 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Compute resources - OpenStack
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Our code in https://github.com/theforeman/foreman/blob/develop/app/models/compute_resources/foreman/model/openstack.rb#L65 uses :display_name as the name attribute to create Openstack volumes.
Notice volume v2 API requires :name, while v1 just needs :display_name
https://github.com/fog/fog/blob/master/lib/fog/openstack/models/volume_v1/volume.rb
https://github.com/fog/fog/blob/master/lib/fog/openstack/models/volume_v2/volume.rb#L17

So we cannot create Volumes on Openstack when they use the v2 api. Fix is simple, just add :name in that hash and it works.

Associated revisions

Revision f5c7a90e (diff)
Added by Daniel Lobato Garcia over 3 years ago

Fixes #13356 - OpenStack volumes API v2 needs :name

Our code in uses :display_name as the name attribute to create
Openstack volumes.

Notice volume v2 API requires :name, while v1 just needs :display_name
https://github.com/fog/fog/blob/master/lib/fog/openstack/models/volume_v1/volume.rb
https://github.com/fog/fog/blob/master/lib/fog/openstack/models/volume_v2/volume.rb#L17

So we cannot create Volumes on Openstack when they use the v2 API.
Fix is simple, just add :name in that hash and it works.

History

#1 Updated by Daniel Lobato Garcia over 3 years ago

  • Subject changed from OpenStack boot_from_volume needs :name to OpenStack volumes API v2 needs :name

#2 Updated by The Foreman Bot over 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/foreman/pull/3117 added

#3 Updated by Daniel Lobato Garcia over 3 years ago

  • Legacy Backlogs Release (now unused) set to 71

Set this to 1.11 as without this fix, Openstack provisioning with volumes is 100% broken

#4 Updated by Daniel Lobato Garcia over 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF