Project

General

Profile

Bug #4262

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

Added by Og Maciel over 6 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
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.

History

#1 Updated by Mike McCune over 6 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

#2 Updated by Ohad Levy over 6 years ago

  • Target version set to 1.9.0

#3 Updated by Dmitri Dolguikh over 6 years ago

  • Target version changed from 1.9.0 to 1.8.4

#4 Updated by Adam Price over 6 years ago

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

#5 Updated by Eric Helms over 6 years ago

  • Triaged set to Yes

#6 Updated by Eric Helms about 6 years ago

  • Status changed from New to Resolved

#7 Updated by Eric Helms about 6 years ago

  • Legacy Backlogs Release (now unused) set to 13

Also available in: Atom PDF