Project

General

Profile

Actions

Bug #4262

closed

--organization-id is confusing as it requires the organization name and not its id

Added by Og Maciel over 10 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Playing with the gpg subcommand I noticed that organization-id, which is a required argument, doesn't take the organization ID value but the organization name, which is definitely bound to cause confusion. Can we change that to either use the ID (the best solution imho) or rename it to organization-name?

[root@qetello03 katello-deploy]# hammer organization list
------------------------------------------------------------------------
ID | NAME             | LABEL            | DESCRIPTION
------------------------------------------------------------------------
1  | ACME_Corporation | ACME_Corporation | ACME_Corporation Organization
------------------------------------------------------------------------
[root@qetello03 katello-deploy]# hammer gpg list --organization-id 1
404 Resource Not Found
[root@qetello03 katello-deploy]# hammer gpg list --organization-id ACME_Corporation
No data.
Actions #1

Updated by Mike McCune over 10 years ago

+1 to just using the ID. We are working on a consistent name vs ID set of arguments to all the hammer commands but for now this should definitely use the ID like the argument states

Actions #2

Updated by Ohad Levy about 10 years ago

  • Target version set to 1.9.0
Actions #3

Updated by Anonymous about 10 years ago

  • Target version changed from 1.9.0 to 1.8.4
Actions #4

Updated by Adam Price about 10 years ago

  • Project changed from Hammer CLI to Katello
  • Category changed from 74 to Hammer
  • Target version deleted (1.8.4)
Actions #5

Updated by Eric Helms almost 10 years ago

  • Triaged set to Yes
Actions #6

Updated by Eric Helms over 9 years ago

  • Status changed from New to Resolved
Actions #7

Updated by Eric Helms over 9 years ago

  • translation missing: en.field_release set to 13
Actions

Also available in: Atom PDF