Actions
Bug #7461
closedCan't add puppet modules to a content view
Description
It seems uuid was renamed to id, but it still validates for 'uuid'.
[cloud-user@katello-devel-1410862301 ~]$ hammer -u admin -p changeme content-view puppet-module add --organization="Default Organization" --content-view="Test CV" --id=$uuid
Could not add the puppet module:
Validation failed: Invalid puppet module parameters specified. Either 'uuid' or 'name' and 'author' must be specified.
[cloud-user@katello-devel-1410862301 ~]$ hammer -u admin -p changeme content-view puppet-module add --help
/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
Usage:
hammer content-view puppet-module add [OPTIONS]
Options:
--author AUTHOR author of the puppet module
--content-view CONTENT_VIEW_NAME Content view name
--content-view-id CONTENT_VIEW_ID content view numeric identifier
--id ID id of the puppet module to associate
--name NAME name of the puppet module
--organization ORGANIZATION_NAME Organization name to search by
--organization-id ORGANIZATION_ID organization ID
--organization-label ORGANIZATION_LABEL Organization label to search by
-h, --help print help
Updated by Eric Helms about 10 years ago
- Difficulty set to easy
- Triaged changed from No to Yes
Updated by The Foreman Bot about 10 years ago
- Status changed from New to Ready For Testing
- Target version set to 56
- Pull request https://github.com/Katello/katello/pull/4684 added
- Pull request deleted (
)
Updated by Eric Helms about 10 years ago
- Status changed from Ready For Testing to New
- Pull request added
- Pull request deleted (
https://github.com/Katello/katello/pull/4684)
Updated by Elyézer Rezende about 10 years ago
Automation had another issue related to adding a puppet module which is the lookup by name. I have tried using the uuid like the example and it seems to be working check below:
# LANG=en_US.UTF-8 hammer -v -u tVEDVN -p UPzoH9McYd --output csv puppet-module list --repository-id='11' ID,Name,Author,Version 6a8e65b6-4cbb-4d22-b04e-21c40f03df8c,httpd,5UbZ3r0,0.2.0 be6ae990-bbf2-4a89-ad1c-c2f74c9eb51f,cron,5UbZ3r0,0.0.1 d7a92a29-2a22-4f53-a763-37ce1e73ce5e,pureftpd,5UbZ3r0,1.0.2 f44d1439-d866-48c4-86ff-2ee7b51c7953,samba,5UbZ3r0,0.2.0 # Name lookup # LANG=en_US.UTF-8 hammer -v -u tVEDVN -p UPzoH9McYd --output csv content-view puppet-module add --content-view-id='8' --name='httpd' /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-03 07:19:04 Exception] Error: content_view_puppet_module not found Could not add the puppet module: Error: content_view_puppet_module not found [ERROR 2014-10-03 07:19:04 Exception] HammerCLIForeman::ResolverError (content_view_puppet_module not found): /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/id_resolver.rb:140:in `pick_result' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/id_resolver.rb:136:in `find_resource' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/id_resolver.rb:114:in `get_id' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/id_resolver.rb:108:in `content_view_puppet_module_id' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/commands.rb:143:in `send' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/commands.rb:143:in `get_resource_id' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/commands.rb:132:in `get_identifier' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/commands.rb:180:in `customized_options' /usr/lib/ruby/gems/1.8/gems/hammer_cli_foreman-0.1.3/lib/hammer_cli_foreman/commands.rb:185:in `request_params' /usr/lib/ruby/gems/1.8/gems/hammer_cli_katello-0.0.6/lib/hammer_cli_katello/content_view_puppet_module.rb:35:in `request_params' /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/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 # ID lookup, passed uuid # LANG=en_US.UTF-8 hammer -v -u tVEDVN -p UPzoH9McYd --output csv content-view puppet-module add --content-view-id='8' --id='6a8e65b6-4cbb-4d22-b04e-21c40f03df8c'/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 Message,Id,Name Puppet module added to content view,3,httpd
Updated by Eric Helms almost 9 years ago
- Translation missing: en.field_release set to 86
Updated by Eric Helms over 8 years ago
- Translation missing: en.field_release changed from 86 to 144
Updated by Eric Helms over 8 years ago
- Translation missing: en.field_release changed from 144 to 168
Updated by Eric Helms over 8 years ago
- Translation missing: en.field_release changed from 168 to 114
Updated by Ian Ballou about 5 years ago
This appears to have been fixed since I cannot reproduce it:
$ hammer content-view puppet-module add --organization="Default Organization" --content-view="Puppeteering" --id=1
Puppet module added to content view.
Updated by Ian Ballou about 5 years ago
- Status changed from Closed to Resolved
- Target version changed from Katello Backlog to Katello Recycle Bin
Actions