Project

General

Profile

Actions

Bug #22718

closed

Hammer does not error when not all items in list of ids are resolved

Added by Martin Bacovsky almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
High
Category:
Foreman commands (obsolete)
Difficulty:
Triaged:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description


hammer role create --name a
hammer user-group create --name ug1 --admin true --roles 'a,m'
User group [ug1] created.

But after checking the logs that only role with id = 1 was added.

This is caused by how resolver is counting expected records.
It tries to guess if it is looking up records for resource in singular or plural from the options.
When both (e.g. --name and --names) are present it will pick singular and thus expects just one record.
In the example above the scoped options contain both name = ug1 and names = 'a,m' and the logic fails.

Actions #1

Updated by Martin Bacovsky almost 7 years ago

  • Target version changed from 246 to 249
Actions #2

Updated by The Foreman Bot almost 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/356 added
Actions #3

Updated by Martin Bacovsky almost 7 years ago

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

Updated by Martin Bacovsky almost 7 years ago

  • Translation missing: en.field_release set to 342
Actions

Also available in: Atom PDF