Project

General

Profile

Actions

Bug #7461

closed

Can't add puppet modules to a content view

Added by Stephen Benjamin over 9 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Hammer
Target version:
Difficulty:
easy
Triaged:
Yes
Fixed in Releases:
Found in Releases:

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
Actions #1

Updated by Eric Helms over 9 years ago

  • Difficulty set to easy
  • Triaged changed from No to Yes
Actions #2

Updated by The Foreman Bot over 9 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 ()
Actions #3

Updated by Eric Helms over 9 years ago

  • Status changed from Ready For Testing to New
  • Pull request added
  • Pull request deleted (https://github.com/Katello/katello/pull/4684)
Actions #4

Updated by Eric Helms over 9 years ago

  • Target version changed from 56 to 58
Actions #5

Updated by Elyézer Rezende over 9 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
Actions #6

Updated by Eric Helms over 9 years ago

  • Target version changed from 58 to 59
Actions #7

Updated by Eric Helms over 9 years ago

  • Target version changed from 59 to 61
Actions #8

Updated by Eric Helms over 9 years ago

  • Target version changed from 61 to 62
Actions #9

Updated by Eric Helms about 9 years ago

  • Target version changed from 62 to 63
Actions #10

Updated by Eric Helms about 9 years ago

  • Target version changed from 63 to 66
Actions #11

Updated by Eric Helms about 9 years ago

  • Target version deleted (66)
Actions #12

Updated by Eric Helms over 8 years ago

  • translation missing: en.field_release set to 86
Actions #13

Updated by Eric Helms about 8 years ago

  • translation missing: en.field_release changed from 86 to 144
Actions #14

Updated by Eric Helms almost 8 years ago

  • translation missing: en.field_release changed from 144 to 168
Actions #15

Updated by Eric Helms almost 8 years ago

  • translation missing: en.field_release changed from 168 to 114
Actions #16

Updated by Ian Ballou over 4 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.
Actions #17

Updated by Ian Ballou over 4 years ago

  • Status changed from New to Closed
Actions #18

Updated by Ian Ballou over 4 years ago

  • Status changed from Closed to Resolved
  • Target version changed from Katello Backlog to Katello Recycle Bin
Actions

Also available in: Atom PDF