Bug #5873

Hammer list actions don't resolve ids for optional parameters

Added by Tomáš Strachota about 4 years ago. Updated 7 days ago.

Status:Closed
Priority:Normal
Assignee:Tomáš Strachota
Category:Foreman commands (obsolete)
Target version:1.8.2
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:

Description

For example:
hammer content-host list --environment dev --organization-id 1
leaves environment name without translating it to id.

Associated revisions

Revision df9a38ca
Added by Tomas Strachota about 4 years ago

Fixes #5873 - list actions don't resolve ids for optional parameters

Revision 4774508b
Added by Tomas Strachota about 4 years ago

Refs #5873 - scoped options were not cleaning original options

The former implementation was not scoping correctly:
input {
:option_name => 'NAME',
:option_id => 1,
:option_product_id => 2
}
output scoped for a product {
:option_name => 'NAME',
:option_id => 2
}
Note the name remainig incorrectly in the options.

This patch solves the problem by clearing all potential searchable options.

History

#1 Updated by Tomáš Strachota about 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Tomáš Strachota
  • Target version set to 1.8.2

#2 Updated by Tomáš Strachota about 4 years ago

  • Status changed from Assigned to Ready For Testing

#3 Updated by Anonymous about 4 years ago

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

Applied in changeset hammer-cli-foreman|commit:df9a38cadd7d004d25837e6ced1caa193ebdeef4.

Also available in: Atom PDF