Bug #21768

Support NIL values in searchable options

Added by Martin Bacovsky about 1 month ago. Updated 6 days ago.

Status:Closed
Priority:Normal
Assigned To:Martin Bacovsky
Category:Foreman commands
Target version:Team Ivan - Iteration 25
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/hammer-cli-foreman/pull/346, https://github.com/theforeman/hammer-cli/pull/264
Story points-
Velocity based estimate-

Description

Currently setting the NIL (unset) values is supported only for common options. The searchable options won't resolve properly:

hammer host update --id 83 --domain NIL
Could not update the host:
  Error: domain not found

Requested result is the domain_id to be unset


Related issues

Related to Hammer CLI - Bug #17252: Unable to set empty parameter via hammer Closed 11/07/2016

Associated revisions

Revision 4928d442
Added by Martin Bacovsky 20 days ago

Refs #21768 - Make option sources overridable

Revision c61d1365
Added by Martin Bacovsky 6 days ago

Fixes #21768 - Allow nil searchables

Adds ability to correctly resolve name params set to NIL.

For better coexistence with hammer defaults the logic
was moved to option sources which is now preferred place
for manipulation with options. As a result when options
or all_options is called anywhere in the code it should
return fully resolved options in final state.

History

#1 Updated by Martin Bacovsky about 1 month ago

  • Related to Bug #17252: Unable to set empty parameter via hammer added

#2 Updated by Martin Bacovsky about 1 month ago

  • Tracker changed from Feature to Bug

#3 Updated by Martin Bacovsky about 1 month ago

  • Target version set to Team Ivan - Iteration 25
  • Assigned To set to Martin Bacovsky
  • Status changed from New to Assigned

#4 Updated by The Foreman Bot about 1 month ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli/pull/264 added

#5 Updated by The Foreman Bot about 1 month ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/346 added

#6 Updated by Martin Bacovsky 6 days ago

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

Also available in: Atom PDF