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 almost 11 years ago. Updated over 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 almost 11 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 over 10 years ago

  • Target version set to 1.9.0
Actions #3

Updated by Anonymous over 10 years ago

  • Target version changed from 1.9.0 to 1.8.4
Actions #4

Updated by Adam Price over 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 over 10 years ago

  • Triaged set to Yes
Actions #6

Updated by Eric Helms over 10 years ago

  • Status changed from New to Resolved
Actions #7

Updated by Eric Helms over 10 years ago

  • Translation missing: en.field_release set to 13
Actions

Also available in: Atom PDF