Bug 798299

Summary: Changeset#promote_content failed with NoMethodError: undefined method `[]' for nil:NilClass
Product: Red Hat Satellite Reporter: Og Maciel <omaciel>
Component: Content ManagementAssignee: Ivan Necas <inecas>
Status: CLOSED CURRENTRELEASE QA Contact: Og Maciel <omaciel>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 6.0.1CC: ansmith, inecas, jturner, mmccune
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-22 18:29:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
traceback
none
Script for reproduction sync error due to conflict in sha checksums none

Description Og Maciel 2012-02-28 15:14:48 UTC
Created attachment 566320 [details]
traceback

Description of problem:

As I performed a series of continuous sync and promote cycles for RHEL 6.1, 6.2 repositories (from manifest) and custom provider content, when I attempted to promote both types of content for the first time, the process failed to complete with the error that can be seen in the subject

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. Created SaoPaulo organization with Library > Dev1 > QA1 > GA1 environments
2. Uploaded manifest and selected RPMs repositories for both RHEL 6.1 and 6.2 Servers, x86_64
3. Synchronized content
4. Promoted RHEL content (all of it) to Dev1 environment
5. Synchronized content again
6. Added new custom content pointing to internal BREW repository for SE
7. Synchronized all available repositories
8.  Promoted both RHEL and custom repository content to Dev1

Actual results:

The web ui showed a message stating that the promotion failed.

Expected results:


Additional info: (traceback attached)

2012-02-28T08:50:56-0500: [Worker(delayed_job host:qetello02.usersys.redhat.com pid:19369)] Changeset#promote_content failed with NoMethodError: undefined method `[]' for nil:NilClass - 0 failed attempts
2012-02-28T08:50:56-0500: [Worker(delayed_job host:qetello02.usersys.redhat.com pid:19369)] PERMANENTLY removing Changeset#promote_content because of 1 consecutive failures.
undefined method `[]' for nil:NilClass

Comment 1 Ivan Necas 2012-02-28 15:23:59 UTC
Notes: There was probably change in pulp error reporting API - Katello needs to reflect this changes: moving error from "result" to "exception" and "traceback"

Comment 3 Ivan Necas 2012-02-29 15:59:36 UTC
Created attachment 566577 [details]
Script for reproduction sync error due to conflict in sha checksums

Attaching a script that reproduces the error

Comment 4 Ivan Necas 2012-02-29 16:24:24 UTC
Fixed in commit 5fb97dc158bcdc642eb2a37cf152338ab91faa67

After short investigation it was only using string instead of symbol for accessing pulp errors.

Comment 6 Ivan Necas 2012-03-01 12:42:20 UTC
Notice: this is a fix for correct  handling of pulp error messages, the problem with the error in Pulp is handled in https://bugzilla.redhat.com/show_bug.cgi?id=798301

Comment 7 Mike McCune 2012-03-07 23:43:15 UTC
mass move ON_QA after brewing

Comment 8 Og Maciel 2012-03-09 15:43:29 UTC
Verified:
* candlepin-0.5.24-1.el6.noarch
* candlepin-tomcat6-0.5.24-1.el6.noarch
* katello-0.1.303-1.el6.noarch
* katello-all-0.1.303-1.el6.noarch
* katello-candlepin-cert-key-pair-1.0-1.noarch
* katello-certs-tools-1.0.4-1.el6.noarch
* katello-cli-0.1.102-1.el6.noarch
* katello-cli-common-0.1.102-1.el6.noarch
* katello-common-0.1.303-1.el6.noarch
* katello-configure-0.1.104-1.el6.noarch
* katello-glue-candlepin-0.1.303-1.el6.noarch
* katello-glue-foreman-0.1.303-1.el6.noarch
* katello-glue-pulp-0.1.303-1.el6.noarch
* katello-qpid-broker-key-pair-1.0-1.noarch
* katello-qpid-client-key-pair-1.0-1.noarch
* katello-selinux-0.1.8-1.el6.noarch
* pulp-1.0.0-4.el6.noarch
* pulp-common-1.0.0-4.el6.noarch
* pulp-selinux-server-1.0.0-4.el6.noarch

Comment 10 Mike McCune 2013-08-16 17:51:18 UTC
getting rid of 6.0.0 version since that doesn't exist