Bug 1131230

Summary: hammer content-host package install/remove/upgrade fails
Product: Red Hat Satellite Reporter: sthirugn <sthirugn>
Component: HammerAssignee: Adam Price <adprice>
Status: CLOSED CURRENTRELEASE QA Contact: sthirugn <sthirugn>
Severity: high Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: bbuckingham, jhutar, mbacovsk, pstudeni
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/7143
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-09-11 12:20:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1105276    

Description sthirugn@redhat.com 2014-08-18 18:23:38 UTC
Description of problem:
hammer package install failed

Version-Release number of selected component (if applicable):
GA Snap 5 - Satellite-6.0.4-RHEL-6-20140813.2

* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.23-1.el6_5.noarch
* candlepin-common-1.0.1-1.el6_5.noarch
* candlepin-scl-1-5.el6_4.noarch
* candlepin-scl-quartz-2.1.5-5.el6_4.noarch
* candlepin-scl-rhino-1.7R3-1.el6_4.noarch
* candlepin-scl-runtime-1-5.el6_4.noarch
* candlepin-selinux-0.9.23-1.el6_5.noarch
* candlepin-tomcat6-0.9.23-1.el6_5.noarch
* elasticsearch-0.90.10-4.el6sat.noarch
* foreman-1.6.0.40-1.el6sat.noarch
* foreman-compute-1.6.0.40-1.el6sat.noarch
* foreman-gce-1.6.0.40-1.el6sat.noarch
* foreman-libvirt-1.6.0.40-1.el6sat.noarch
* foreman-ovirt-1.6.0.40-1.el6sat.noarch
* foreman-postgresql-1.6.0.40-1.el6sat.noarch
* foreman-proxy-1.6.0.27-1.el6sat.noarch
* foreman-selinux-1.6.0.6-1.el6sat.noarch
* foreman-vmware-1.6.0.40-1.el6sat.noarch
* katello-1.5.0-28.el6sat.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.6-1.el6sat.noarch
* katello-installer-0.0.59-1.el6sat.noarch
* openldap-2.4.23-34.el6_5.1.x86_64
* openldap-devel-2.4.23-34.el6_5.1.x86_64
* pulp-katello-0.3-3.el6sat.noarch
* pulp-nodes-common-2.4.0-0.30.beta.el6sat.noarch
* pulp-nodes-parent-2.4.0-0.30.beta.el6sat.noarch
* pulp-puppet-plugins-2.4.0-0.30.beta.el6sat.noarch
* pulp-puppet-tools-2.4.0-0.30.beta.el6sat.noarch
* pulp-rpm-plugins-2.4.0-0.30.beta.el6sat.noarch
* pulp-selinux-2.4.0-0.30.beta.el6sat.noarch
* pulp-server-2.4.0-0.30.beta.el6sat.noarch
* python-ldap-2.3.10-1.el6.x86_64
* ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
* ruby193-rubygem-runcible-1.1.0-2.el6sat.noarch


How reproducible:
Always


Steps to Reproduce:

hammer package install failed

# hammer -v -u admin -p changeme content-host package install --content-host sat6.redhat.com --organization-id=3 --packages='["emacs","firefox"]'
[ERROR 2014-08-18 10:34:01 API] 500 Internal Server Error

# hammer -v -u admin -p changeme content-host package install --content-host sat6.redhat.com --organization-id=3 --packages=emacs,firefox
[ERROR 2014-08-18 10:34:01 API] 500 Internal Server Error

# hammer -v -u admin -p changeme content-host package install --content-host sat6.redhat.com --organization-id=3 --packages='[emacs,firefox]'
[ERROR 2014-08-18 10:34:01 API] 500 Internal Server Error


Actual results:
Failed as shown above

Expected results:
Package install should work fine

Additional info:

Comment 2 sthirugn@redhat.com 2014-08-18 18:39:31 UTC
Looks like all content-host package commands fail except upgrade-all. 

See below:

hammer> content-host package remove --content-host=sat6.redhat.com --organization-id=3 --packages='["emacs","firefox"]'
Could not remove packages:
  undefined method `each' for "[\"emacs\",\"firefox\"]":String

hammer> content-host package upgrade  --content-host=sat6.redhat.com  --organization-id=3 --packages='["emacs","firefox"]'
Could not upgrade packages:
  undefined method `each' for "[\"emacs\",\"firefox\"]":String

hammer> content-host package upgrade-all  --content-host=sat6.redhat.com  --organization-id=3
[...................................................................] [100%]
Task 85163923-c030-4abf-bf44-0d17306d5329: success
m2crypto-0.21.1.pulp-10.el6sat.x86_64

Comment 3 sthirugn@redhat.com 2014-08-18 18:46:52 UTC
Filed https://bugzilla.redhat.com/show_bug.cgi?id=1131238 for the issue mentioned in Comment 2. 

Note: Please do not close one of these bugs saying it is a dupe of another.  QE need to validate both the scenarios.

Comment 4 sthirugn@redhat.com 2014-08-18 18:47:53 UTC
Ignore Comment 3 and Comment 4.  Those were supposed to go in an other bug. My bad!

Comment 5 Adam Price 2014-08-18 19:35:56 UTC
Created redmine issue http://projects.theforeman.org/issues/7143 from this bug

Comment 6 Pavel Studeník 2014-08-19 15:24:36 UTC
*** Bug 1131559 has been marked as a duplicate of this bug. ***

Comment 7 Adam Price 2014-08-19 16:47:46 UTC
this is related to a change that occurred in apipie-rails.

dustint submitted a patch [1] to hammer-cli to fix the creation of param arrays.

[1] https://github.com/theforeman/hammer-cli/pull/138

Comment 8 Adam Price 2014-08-19 22:13:09 UTC
we're waiting on a new build of hammer-cli (i'm assuming version 0.1.3) from @mbacovsk to get into Satellite6 before we move this to POST. thanks!

Comment 9 Martin Bacovsky 2014-08-20 12:41:20 UTC
Hammer 0.1.3 (with the fix) was just released upstream. @jmontleo was notified.

Comment 10 sthirugn@redhat.com 2014-08-23 14:05:23 UTC
Moving to ON_QA as it looks like the array input for cli is fixed.

Comment 11 sthirugn@redhat.com 2014-08-23 14:06:39 UTC
Verified.

hammer> content-host package install --content-host cloud-qe-22.idm.lab.bos.redhat.com  --organization-id=3 --packages=emacs,firefox
[............................................................] [100%]


hammer> content-host package remove --content-host cloud-qe-22.idm.lab.bos.redhat.com  --organization-id=3 --packages=emacs,firefox
[............................................................] [100%]

hammer> content-host package upgrade --content-host cloud-qe-22.idm.lab.bos.redhat.com  --organization-id=3 --packages=emacs,firefox
[............................................................] [100%]

Version Tested:
GA Snap 6 - Satellite-6.0.4-RHEL-6-20140820.1

* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.23-1.el6_5.noarch
* candlepin-common-1.0.1-1.el6_5.noarch
* candlepin-scl-1-5.el6_4.noarch
* candlepin-scl-quartz-2.1.5-5.el6_4.noarch
* candlepin-scl-rhino-1.7R3-1.el6_4.noarch
* candlepin-scl-runtime-1-5.el6_4.noarch
* candlepin-selinux-0.9.23-1.el6_5.noarch
* candlepin-tomcat6-0.9.23-1.el6_5.noarch
* elasticsearch-0.90.10-6.el6sat.noarch
* foreman-1.6.0.41-1.el6sat.noarch
* foreman-compute-1.6.0.41-1.el6sat.noarch
* foreman-gce-1.6.0.41-1.el6sat.noarch
* foreman-libvirt-1.6.0.41-1.el6sat.noarch
* foreman-ovirt-1.6.0.41-1.el6sat.noarch
* foreman-postgresql-1.6.0.41-1.el6sat.noarch
* foreman-proxy-1.6.0.29-1.el6sat.noarch
* foreman-selinux-1.6.0.7-1.el6sat.noarch
* foreman-vmware-1.6.0.41-1.el6sat.noarch
* katello-1.5.0-29.el6sat.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.6-1.el6sat.noarch
* katello-installer-0.0.60-1.el6sat.noarch
* openldap-2.4.23-34.el6_5.1.x86_64
* openldap-devel-2.4.23-34.el6_5.1.x86_64
* pulp-katello-0.3-3.el6sat.noarch
* pulp-nodes-common-2.4.0-0.30.beta.el6sat.noarch
* pulp-nodes-parent-2.4.0-0.30.beta.el6sat.noarch
* pulp-puppet-plugins-2.4.0-0.30.beta.el6sat.noarch
* pulp-puppet-tools-2.4.0-0.30.beta.el6sat.noarch
* pulp-rpm-plugins-2.4.0-0.30.beta.el6sat.noarch
* pulp-selinux-2.4.0-0.30.beta.el6sat.noarch
* pulp-server-2.4.0-0.30.beta.el6sat.noarch
* python-ldap-2.3.10-1.el6.x86_64
* ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
* ruby193-rubygem-runcible-1.1.0-2.el6sat.noarch

Comment 12 Bryan Kearney 2014-09-11 12:20:03 UTC
This was delivered with Satellite 6.0 which was released on 10 September 2014.