Bug 1048232 - Cli -:Hammer cli command to import classes fails
Summary: Cli -:Hammer cli command to import classes fails
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Provisioning
Version: Nightly
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: Unspecified
Assignee: Tomas Strachota
QA Contact: Tazim Kolhar
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-01-03 13:06 UTC by Tazim Kolhar
Modified: 2019-09-26 18:18 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-07-02 14:14:44 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
foreman -debug (59.11 KB, application/x-xz)
2014-05-12 14:27 UTC, Tazim Kolhar
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 6090 0 None None None 2016-04-22 16:38:57 UTC

Description Tazim Kolhar 2014-01-03 13:06:27 UTC
Description of problem:
Hammer cli command to import classes fails

Version-Release number of selected component (if applicable):
# rpm -qa | grep foreman
 rubygem-hammer_cli_foreman-0.0.15-1.el6.noarch
foreman-compute-1.4.0-0.develop.201401011454git1245d23.el6.noarch
foreman-installer-1.4.0-0.develop.201312272053gitbc2acd9.el6.noarch
foreman-release-1.4.0-0.develop.201401011454git1245d23.el6.noarch
foreman-1.4.0-0.develop.201401011454git1245d23.el6.noarch
foreman-ovirt-1.4.0-0.develop.201401011454git1245d23.el6.noarch
 foreman-proxy-1.4.0-0.develop.201312091021git42bab91.el6.noarch
foreman-postgresql-1.4.0-0.develop.201401011454git1245d23.el6.noarch
 rubygem-foreman_api-0.1.9-1.el6.noarch
 foreman-libvirt-1.4.0-0.develop.201401011454git1245d23.el6.noarch
 foreman-selinux-1.4.0-0.develop.201311220906gite342a0b.el6.noarch

How reproducible:
everytime

Steps to Reproduce:
1.# hammer proxy import_classes --id 1
Puppet classes were imported

2.# hammer -v sc_param list
[ INFO 2014-01-03 07:57:09 Init] Initialization of Hammer CLI (0.0.14) has started...
[ INFO 2014-01-03 07:57:09 Init] Configuration from the file /etc/foreman/cli_config.yml has been loaded
[ INFO 2014-01-03 07:57:09 Modules] Extension module hammer_cli_foreman (0.0.15) loaded
[ INFO 2014-01-03 07:57:09 HammerCLI::MainCommand] Called with options: {"verbose"=>true}
[ INFO 2014-01-03 07:57:09 HammerCLIForeman::SmartClassParameter] Called with options: {}
[ INFO 2014-01-03 07:57:09 HammerCLIForeman::SmartClassParameter::ListCommand] Called with options: {}
[DEBUG 2014-01-03 07:57:09 HammerCLIForeman::SmartClassParameter::ListCommand] Retrieved data: 
[]
No data.

3.# hammer environment sc_params --id 1
No data.


Actual results:
Unable to view the list

Expected results:
List displayed

Additional info:

Comment 1 RHEL Program Management 2014-01-03 13:14:48 UTC
Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.

Comment 5 Tazim Kolhar 2014-05-08 07:17:51 UTC
# hammer proxy import-classes --id 1
Import of puppet classes failed:
  404 Resource Not Found

Comment 6 Dominic Cleal 2014-05-12 13:57:10 UTC
(In reply to Tazim Kolhar from comment #5)
> # hammer proxy import-classes --id 1
> Import of puppet classes failed:
>   404 Resource Not Found

Please attach the foreman-debug archive, output from Hammer's verbose mode, and the output of "hammer proxy list".

Comment 7 Tazim Kolhar 2014-05-12 14:27:35 UTC
Created attachment 894735 [details]
foreman -debug

 hammer proxy list
---|---------------------------------------------|---------------------------------------------------------
ID | NAME                                        | URL                                                     
---|---------------------------------------------|---------------------------------------------------------
1  | ibm-x3650m4-01-vm-16.lab.eng.bos.redhat.com | https://ibm-x3650m4-01-vm-16.lab.eng.bos.redhat.com:9090
---|---------------------------------------------|---------------------------------------------------------

Comment 8 Dominic Cleal 2014-05-12 14:38:05 UTC
Are you sure that foreman-debug file corresponds to you running the command producing the error?  The log is from today and I see no 404 errors as reported in comment #5.

Please also run the command with debugging:
  hammer -d proxy import-classes --id 1

Comment 9 Bryan Kearney 2014-06-06 12:20:27 UTC
Removing the Failed QE Verified flag so that the script will move this to POST when the redmine issue is fixed.

Comment 10 Bryan Kearney 2014-06-11 11:12:33 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/6090 has been closed

Comment 13 Tazim Kolhar 2014-06-17 08:05:21 UTC
VERIFIED

Upload puppet module

# hammer proxy import-classes --id 1
Puppet classes were imported

# hammer -v sc-param list 
---|---------------------|----------------------------------------|----------|-------------
ID | PARAMETER           | DEFAULT VALUE                          | OVERRIDE | PUPPET CLASS
---|---------------------|----------------------------------------|----------|-------------
9  | autoupgrade         |                                        |          |             
28 | autoupgrade         |                                        |          |             
34 | config_file         | ${$ntp::params::config_file}           |          |             
12 | config_file         | ${$ntp::params::ntpdate_config_file}   |          |             
35 | config_file_group   | ${$ntp::params::config_file_group}     |          |             
14 | config_file_mode    | ${$ntp::params::config_file_mode}      |          |             
18 | config_file_owner   | ${$ntp::params::config_file_owner}     |          |             
20 | config_file_replace | true                                   |          |             
27 | defaults_file       | ${$ntp::params::defaults_file}         |          |             
8  | defaults_file       | ${$ntp::params::ntpdate_defaults_file} |          |             
29 | driftfile           | ${$ntp::params::driftfile}             |          |             
16 | enable_statistics   |                                        |          |             
6  | ensure              | present                                |          |             
25 | ensure              | present                                |          |             
21 | interface_ignore    |                                        |          |             
19 | interface_listen    |                                        |          |             
1  | ntpdate_options     | ${$ntp::params::ntpdate_options}       |          |             
15 | ntpd_start_options  | ${$ntp::params::ntpd_start_options}    |          |             
4  | package             | ${$ntp::params::ntpdate_package}       |          |             
22 | package             | ${$ntp::params::package}               |          |             
---|---------------------|----------------------------------------|----------|-------------
List next page? (Y/n): Y
---|--------------------|---------------------------------------|----------|-------------
ID | PARAMETER          | DEFAULT VALUE                         | OVERRIDE | PUPPET CLASS
---|--------------------|---------------------------------------|----------|-------------
24 | query_networks     |                                       |          |             
33 | server_enabled     |                                       |          |             
11 | server_list        | 0.pool.ntp.org1.pool.ntp.org2.pool... |          |             
31 | server_list        | 0.pool.ntp.org1.pool.ntp.org2.pool... |          |             
26 | service_enable     | true                                  |          |             
7  | service_enable     | true                                  |          |             
5  | service_ensure     | running                               |          |             
23 | service_ensure     | running                               |          |             
17 | service_hasrestart | true                                  |          |             
3  | service_hasrestart | true                                  |          |             
10 | service_hasstatus  |                                       |          |             
30 | service_hasstatus  | true                                  |          |             
13 | service_name       | ${$ntp::params::ntpdate_service_name} |          |             
36 | service_name       | ${$ntp::params::service_name}         |          |             
32 | statsdir           |                                       |          |             
2  | sync_hwclock       |                                       |          |             
---|--------------------|---------------------------------------|----------|-------------

Comment 14 Bryan Kearney 2014-07-02 14:09:52 UTC
This was delivered with 6.0.3, which is the Satellite 6 Beta.

Comment 15 Bryan Kearney 2014-07-02 14:10:07 UTC
This was delivered in 6.0.3, the Beta version of Satellite 6.0

Comment 16 Bryan Kearney 2014-07-02 14:14:44 UTC
This was delivered in the Satellite 6 Beta (6.0.3)


Note You need to log in before you can comment on or make changes to this bug.