Bug #5873

Hammer list actions don't resolve ids for optional parameters

Added by Tomáš Strachota over 3 years ago. Updated over 3 years ago.

Status:Closed
Priority:Normal
Assigned To:Tomáš Strachota
Category:Foreman commands
Target version:Foreman - Sprint 24
Difficulty: Bugzilla link:
Found in release: Pull request:
Story points-
Velocity based estimate-

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 over 3 years ago

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

Revision 4774508b
Added by Tomas Strachota over 3 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 over 3 years ago

  • Status changed from New to Assigned
  • Assigned To set to Tomáš Strachota
  • Target version set to Sprint 24

#2 Updated by Tomáš Strachota over 3 years ago

  • Status changed from Assigned to Ready For Testing

#3 Updated by Anonymous over 3 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