Bug #6465
closedhammer activation-key update should not require an organization-id when have the --id option
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1114109
Description of problem:
When updating an activation key passing its ID, the organization option should not be required.
- hammer --version
hammer (0.1.1) * hammer_cli_foreman (0.1.1) * hammer_cli_katello (0.0.4)
How reproducible:
Aways
Steps to Reproduce:
1. Create an activation key using hammer
2. Try to update its name passing --id and --new-name options
Actual results:
The activation key is not updated and an exception is raised
Expected results:
The activation key should be updated.
- LANG=en_US hammer -v -u admin -p changeme --output csv activation-key create --name='ue99k5' --organization-id='169' --usage-limit='unlimited'
Message,Id,Name
Activation key created,35,ue99k5
- LANG=en_US hammer -v -u admin -p changeme activation-key info --id='35'
Name: ue99k5
ID: 35
Description:
Lifecycle Environment:
Content View:
Host Collections:
- LANG=en_US hammer -v -u admin -p changeme --output csv activation-key update --id='35' --new-name='glHZmTDUdzxiYFJ'
[ERROR 2014-06-27 14:02:31 Exception] Error: Missing options to search organization
Could not update the activation key:
Error: Missing options to search organization
[ERROR 2014-06-27 14:02:31 Exception]
HammerCLIForeman::MissingSeachOptions (Missing options to search organization):
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/id_resolver.rb:115:in `search_options'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/id_resolver.rb:88:in `find_resource'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/id_resolver.rb:82:in `get_id'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/id_resolver.rb:76:in `organization_id'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/commands.rb:133:in `send'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/commands.rb:133:in `get_resource_id'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/commands.rb:164:in `request_params'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/commands.rb:161:in `each'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/commands.rb:161:in `request_params'
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.1/lib/hammer_cli/./apipie/command.rb:42:in `send_request'
/usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.1/lib/hammer_cli_foreman/commands.rb:154:in `send_request'
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.1/lib/hammer_cli/./apipie/command.rb:33:in `execute'
/usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/command.rb:67:in `run'
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.1/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:67:in `run'
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.1/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:67:in `run'
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.1/lib/hammer_cli/./apipie/../abstract.rb:22:in `run'
/usr/lib/ruby/gems/1.8/gems/clamp-0.6.2/lib/clamp/command.rb:125:in `run'
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.1/bin/hammer:100
/usr/bin/hammer:19:in `load'
/usr/bin/hammer:19
Updated by dustin tsang almost 11 years ago
- Target version set to 48
- Difficulty set to easy
Updated by The Foreman Bot over 10 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/hammer-cli-katello/pull/202 added
Updated by dustin tsang over 10 years ago
- Status changed from Ready For Testing to Rejected
Updated by Eric Helms over 10 years ago
- Translation missing: en.field_release set to 13
Updated by Eric Helms about 10 years ago
- Translation missing: en.field_release deleted (
13)
Updated by Eric Helms almost 9 years ago
- Translation missing: en.field_release set to 166