Project

General

Profile

Bug #7461

Can't add puppet modules to a content view

Added by Stephen Benjamin over 4 years ago. Updated 11 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Hammer
Target version:
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Pull request:
Team Backlog:
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

History

#1 Updated by Eric Helms over 4 years ago

  • Difficulty set to easy
  • Triaged changed from No to Yes

#2 Updated by The Foreman Bot over 4 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 ()

#3 Updated by Eric Helms over 4 years ago

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

#4 Updated by Eric Helms over 4 years ago

  • Target version changed from 56 to 58

#5 Updated by Elyézer Rezende over 4 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

#6 Updated by Eric Helms over 4 years ago

  • Target version changed from 58 to 59

#7 Updated by Eric Helms over 4 years ago

  • Target version changed from 59 to 61

#8 Updated by Eric Helms over 4 years ago

  • Target version changed from 61 to 62

#9 Updated by Eric Helms over 4 years ago

  • Target version changed from 62 to 63

#10 Updated by Eric Helms over 4 years ago

  • Target version changed from 63 to 66

#11 Updated by Eric Helms over 4 years ago

  • Target version deleted (66)

#12 Updated by Eric Helms over 3 years ago

  • Legacy Backlogs Release (now unused) set to 86

#13 Updated by Eric Helms about 3 years ago

  • Legacy Backlogs Release (now unused) changed from 86 to 144

#14 Updated by Eric Helms almost 3 years ago

  • Legacy Backlogs Release (now unused) changed from 144 to 168

#15 Updated by Eric Helms almost 3 years ago

  • Legacy Backlogs Release (now unused) changed from 168 to 114

Also available in: Atom PDF