Project

General

Profile

Actions
Copy link

Bug #7813

closed

Four of the activation-key commands accept only activation-key id missing activation key name, organization, organization-id, organization-label arguments

Added by Christine Fouant about 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Christine Fouant
Category:
Hammer
Target version:
Katello 2.1
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
1110475
Pull request:
https://github.com/Katello/hammer-cli-katello/pull/237
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1110475
Description of problem:
Four of the activation-key commands accept only activation-key id missing activation key name, organization, organization-id, organization-label arguments
1. hammer activation-key host-collections
2. hammer activation-key add-subscription
3. hammer activation-key remove-subscription
4. hammer activation-key subscriptions

Version-Release number of selected component (if applicable):
  • apr-util-ldap-1.3.9-3.el6_0.1.x86_64
  • candlepin-0.9.19-1.el6_5.noarch
  • candlepin-scl-1-5.el6_4.noarch
  • candlepin-scl-quartz-2.1.5-5.el6_4.noarch
  • candlepin-scl-rhino-1.7R3-1.el6_4.noarch
  • candlepin-scl-runtime-1-5.el6_4.noarch
  • candlepin-selinux-0.9.19-1.el6_5.noarch
  • candlepin-tomcat6-0.9.19-1.el6_5.noarch
  • elasticsearch-0.90.10-4.el6sat.noarch
  • foreman-1.6.0.17-1.el6sat.noarch
  • foreman-compute-1.6.0.17-1.el6sat.noarch
  • foreman-gce-1.6.0.17-1.el6sat.noarch
  • foreman-libvirt-1.6.0.17-1.el6sat.noarch
  • foreman-ovirt-1.6.0.17-1.el6sat.noarch
  • foreman-postgresql-1.6.0.17-1.el6sat.noarch
  • foreman-proxy-1.6.0.7-1.el6sat.noarch
  • foreman-selinux-1.6.0-4.el6sat.noarch
  • foreman-vmware-1.6.0.17-1.el6sat.noarch
  • katello-1.5.0-26.el6sat.noarch
  • katello-ca-1.0-1.noarch
  • katello-certs-tools-1.5.5-1.el6sat.noarch
  • katello-installer-0.0.48-1.el6sat.noarch
  • openldap-2.4.23-32.el6_4.1.x86_64
  • pulp-katello-0.3-3.el6sat.noarch
  • pulp-nodes-common-2.4.0-0.20.beta.el6sat.noarch
  • pulp-nodes-parent-2.4.0-0.20.beta.el6sat.noarch
  • pulp-puppet-plugins-2.4.0-0.20.beta.el6sat.noarch
  • pulp-puppet-tools-2.4.0-0.20.beta.el6sat.noarch
  • pulp-rpm-plugins-2.4.0-0.20.beta.el6sat.noarch
  • pulp-selinux-2.4.0-0.20.beta.el6sat.noarch
  • pulp-server-2.4.0-0.20.beta.el6sat.noarch
  • python-ldap-2.3.10-1.el6.x86_64
  • ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
  • ruby193-rubygem-runcible-1.1.0-2.el6sat.noarch

How reproducible:
Always

Steps to Reproduce:

  1. hammer activation-key host-collections --help
    Usage:
    hammer activation-key host-collections [OPTIONS]
    Options:
    --id ID resource ID
    -h, --help print help
  1. hammer activation-key add-subscription --help
    Usage:
    hammer activation-key add-subscription [OPTIONS]
    Options:
    --id ID resource ID
    --quantity QUANTITY subscription quantity
    --subscription-id ID subscription ID
    -h, --help print help
  1. hammer activation-key remove-subscription --help
    Usage:
    hammer activation-key remove-subscription [OPTIONS]
    Options:
    --id ID resource ID
    --subscription-id ID subscription ID
    -h, --help print help
  1. hammer activation-key subscriptions --help
    Usage:
    hammer activation-key subscriptions [OPTIONS]
    Options:
    --id ID resource ID
    -h, --help print help

Actual results:
As shown above

Expected results:
The four commands mentioned here should accept activation key name, organization, organization-id, organization-label

Additional info:

Actions
Copy link
 #1

Updated by Elyézer Rezende about 10 years ago

I'm facing the below issue when trying to add a subscription to an activation key, and as per IRC this should be related to this issue. Adding the paste contents here.

# hammer -v -u admin -p changeme  activation-key add-subscription --subscription-id='<subs_id>' --id='3' --quantity='10'
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.3/lib/hammer_cli/./apipie/../abstract.rb:67: warning: already initialized constant DEFAULT_LABEL_INDENT
[ERROR 2014-10-06 09:21:27 Exception] Missing arguments for 'subscriptions'
Could not add subscription to activation key:
  Missing arguments for 'subscriptions'
[ERROR 2014-10-06 09:21:27 Exception] 

ApipieBindings::MissingArgumentsError (Missing arguments for 'subscriptions'):
    /usr/lib/ruby/gems/1.8/gems/apipie-bindings-0.0.10/lib/apipie_bindings/action.rb:62:in `validate!'
    /usr/lib/ruby/gems/1.8/gems/apipie-bindings-0.0.10/lib/apipie_bindings/api.rb:151:in `call'
    /usr/lib/ruby/gems/1.8/gems/apipie-bindings-0.0.10/lib/apipie_bindings/resource.rb:14:in `call'
    /usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.3/lib/hammer_cli/./apipie/command.rb:42:in `send_request'
    /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/commands.rb:164:in `send_request'
    /usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.3/lib/hammer_cli/./apipie/command.rb:33:in `execute'
    /usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/command.rb:68:in `run'
    /usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.3/lib/hammer_cli/./apipie/../abstract.rb:22:in `run'
    /usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/subcommand/execution.rb:11:in `execute'
    /usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/command.rb:68:in `run'
    /usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.3/lib/hammer_cli/./apipie/../abstract.rb:22:in `run'
    /usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/subcommand/execution.rb:11:in `execute'
    /usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/command.rb:68:in `run'
    /usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.3/lib/hammer_cli/./apipie/../abstract.rb:22:in `run'
    /usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/command.rb:126:in `run'
    /usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.3/bin/hammer:106
    /usr/bin/hammer:19:in `load'
    /usr/bin/hammer:19
Actions
Copy link
 #2

Updated by Eric Helms about 10 years ago

  • Category set to Hammer
  • Translation missing: en.field_release set to 14
  • Difficulty set to easy
  • Triaged changed from No to Yes
Actions
Copy link
 #3

Updated by The Foreman Bot about 10 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 58
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/237 added
  • Pull request deleted ()
Actions
Copy link
 #4

Updated by Christine Fouant about 10 years ago

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

Also available in: Atom PDF