Project

General

Profile

Actions

Bug #6342

closed

Compute profile should be used when empty volumes and interfaces passed in compute_attributes

Added by Dominic Cleal almost 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
API
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

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.


Related issues 7 (1 open6 closed)

Related to Foreman - Feature #4250: Expose compute profiles via APIClosedJoseph Magen02/04/2014Actions
Related to Hammer CLI - Feature #6344: Commands for read operations around compute profiles and attributesClosedTomáš StrachotaActions
Related to Foreman - Bug #13507: Creating VMs with hammer requires too many parametersResolved02/01/2016Actions
Related to Foreman - Bug #14179: Interface/NIC compute attributes in API host creation overwritten with compute profile attributesClosedDominic Cleal03/14/2016Actions
Related to Foreman - Bug #10223: Image provisoning over API doesn't start host automaticallyNew04/21/2015Actions
Has duplicate Hammer CLI - Bug #11829: hammer host create does not send correct compute-attributes information which breaks a condition on the server sideDuplicate09/15/2015Actions
Has duplicate Foreman - Bug #14485: hammer should consider everything from the "Compute Profile"Duplicate04/05/2016Actions
Actions #1

Updated by Dominic Cleal almost 10 years ago

  • Related to Feature #4250: Expose compute profiles via API added
Actions #2

Updated by Dominic Cleal almost 10 years ago

  • Related to Feature #6344: Commands for read operations around compute profiles and attributes added
Actions #3

Updated by Dominic Cleal over 9 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Dominic Cleal
  • Target version set to 1.8.1
Actions #4

Updated by Anonymous over 9 years ago

  • Target version changed from 1.8.1 to 1.8.0
Actions #5

Updated by Dominic Cleal over 9 years ago

  • Status changed from Ready For Testing to Assigned
Actions #6

Updated by Anonymous over 9 years ago

  • Target version changed from 1.8.0 to 1.7.5
Actions #7

Updated by Dominic Cleal over 9 years ago

  • Status changed from Assigned to New
  • Assignee deleted (Dominic Cleal)
  • Target version deleted (1.7.5)
Actions #8

Updated by Dominic Cleal over 8 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
Actions #9

Updated by Bryan Kearney over 8 years ago

  • Bugzilla link set to 1263332
Actions #10

Updated by Tomáš Strachota over 8 years ago

  • Status changed from New to Assigned
  • Assignee set to Tomáš Strachota
Actions #11

Updated by The Foreman Bot over 8 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2802 added
  • Pull request deleted ()
Actions #12

Updated by Guido Günther about 8 years ago

  • Related to Bug #13507: Creating VMs with hammer requires too many parameters added
Actions #13

Updated by Anonymous about 8 years ago

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

Updated by Dominic Cleal about 8 years ago

  • translation missing: en.field_release set to 71
Actions #15

Updated by Dominic Cleal about 8 years ago

  • Related to Bug #14179: Interface/NIC compute attributes in API host creation overwritten with compute profile attributes added
Actions #16

Updated by Dominic Cleal about 8 years ago

  • Related to Bug #10223: Image provisoning over API doesn't start host automatically added
Actions #17

Updated by Guido Günther about 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.

Actions #18

Updated by Dominic Cleal about 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.

Actions #19

Updated by Dominic Cleal almost 8 years ago

  • Has duplicate Bug #14485: hammer should consider everything from the "Compute Profile" added
Actions #20

Updated by Bryan Kearney almost 8 years ago

  • Bugzilla link changed from 1263332 to 1319564
Actions

Also available in: Atom PDF