Bug #6342
closedCompute profile should be used when empty volumes and interfaces passed in compute_attributes
Description
When creating hosts through the API with hammer, it's fairly unavoidable that you get a params hash containing:
"compute_attributes"=>{"volumes_attributes"=>{},"interfaces_attributes"=>{},"nics_attributes"=>{}}
due to hammer not knowing if a compute profile will be used or not via the host group.
set_compute_attributes in Host::Managed checks for compute_attributes.empty?, which will return false when these empty sub-hashes are present. It should be possible to use the compute profile even when passing in this empty data.
Either that, or CR VM creation should be reliable without them set to {}, but it currently isn't.
Updated by Dominic Cleal about 10 years ago
- Related to Feature #4250: Expose compute profiles via API added
Updated by Dominic Cleal about 10 years ago
- Related to Feature #6344: Commands for read operations around compute profiles and attributes added
Updated by Dominic Cleal about 10 years ago
- Status changed from New to Ready For Testing
- Assignee set to Dominic Cleal
- Target version set to 1.8.1
Updated by Anonymous about 10 years ago
- Target version changed from 1.8.1 to 1.8.0
Updated by Dominic Cleal about 10 years ago
- Status changed from Ready For Testing to Assigned
Updated by Anonymous about 10 years ago
- Target version changed from 1.8.0 to 1.7.5
Updated by Dominic Cleal about 10 years ago
- Status changed from Assigned to New
- Assignee deleted (
Dominic Cleal) - Target version deleted (
1.7.5)
Updated by Dominic Cleal almost 9 years ago
- Has duplicate Bug #11829: hammer host create does not send correct compute-attributes information which breaks a condition on the server side added
Updated by Tomáš Strachota almost 9 years ago
- Status changed from New to Assigned
- Assignee set to Tomáš Strachota
Updated by The Foreman Bot almost 9 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2802 added
- Pull request deleted (
)
Updated by Guido Günther over 8 years ago
- Related to Bug #13507: Creating VMs with hammer requires too many parameters added
Updated by Anonymous over 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 47990639ae6b02f541e5d76b392d5549afd74bc7.
Updated by Dominic Cleal over 8 years ago
- Translation missing: en.field_release set to 71
Updated by Dominic Cleal over 8 years ago
- Related to Bug #14179: Interface/NIC compute attributes in API host creation overwritten with compute profile attributes added
Updated by Dominic Cleal over 8 years ago
- Related to Bug #10223: Image provisoning over API doesn't start host automatically added
Updated by Guido Günther over 8 years ago
Anonymous wrote:
Applied in changeset 47990639ae6b02f541e5d76b392d5549afd74bc7.
Could this be fixed on 1.10.x as well? We won't be able to update to 1.11.x in not time so this would be awesome.
Updated by Dominic Cleal over 8 years ago
Sorry, this is a behaviour change and has a bit of risk (one regression so far) so it'll only be in 1.11.
Updated by Dominic Cleal over 8 years ago
- Has duplicate Bug #14485: hammer should consider everything from the "Compute Profile" added
Updated by Bryan Kearney over 8 years ago
- Bugzilla link changed from 1263332 to 1319564