Bug 1867239

Summary: hammer content-view version incremental-update fails with ISE
Product: Red Hat Satellite Reporter: Stephen Wadeley <swadeley>
Component: Content ViewsAssignee: satellite6-bugs <satellite6-bugs>
Status: CLOSED ERRATA QA Contact: Stephen Wadeley <swadeley>
Severity: high Docs Contact:
Priority: unspecified    
Version: 6.8.0CC: mawerner, pcreech, sajha
Target Milestone: 6.8.0Keywords: Regression, Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tfm-rubygem-katello-3.16.0.3-1 Doc Type: If docs needed, set a value
Doc Text:
hammer content-view version incremental-update fails with ISE
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-10-27 13:05:24 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:

Description Stephen Wadeley 2020-08-07 19:48:51 UTC
Description of problem:

These commands:

hammer -d content-view version incremental-update --content-view-version-id 9 --errata-ids 5
hammer -d content-view version incremental-update --content-view-version-id 9 --errata-ids 5  --lifecycle-environment-ids 8

fail to with:
 500 Internal Server Error
"undefined method `first' for nil:NilClass"

Version-Release number of selected component (if applicable):
6,8 Snap 11

[root@dhcp-2-186 ~]# rpm -q satellite
satellite-6.8.0-0.7.beta.el7sat.noarch
[root@dhcp-2-186 ~]# rpm -q tfm-rubygem-katello
tfm-rubygem-katello-3.16.0-0.16.rc5.1.el7sat.noarch

How reproducible:
fails with API commands and hammer commands

Steps to Reproduce:
1. Create CV with fake errata
2. Create another empty CV
3. Try to use incremental-update command to add an erratuim to a version of CV

Actual results:

hammer -d shows:

[ERROR 2020-08-07T12:04:04 API] 500 Internal Server Error
[DEBUG 2020-08-07T12:04:04 API] {
    "displayMessage" => "undefined method `first' for nil:NilClass",
            "errors" => [
        [0] "undefined method `first' for nil:NilClass"
    ]
}

Expected results:

CV version is created with errratum

Additional info:

also fails when using API

Comment 6 Samir Jha 2020-08-27 17:13:43 UTC
Created redmine issue http://projects.theforeman.org/issues/30725 from this bug

Comment 7 Bryan Kearney 2020-08-29 00:05:46 UTC
Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/30725 has been resolved.

Comment 9 Stephen Wadeley 2020-09-03 14:47:40 UTC
Hello

further to comment 0

my step 2 is not complete


An incremental update is copying the errata from Library to a CVV
The errata needs to belong to some repository that is part of the CVV 

Clearing Known Issue flag as this bug was in fact just an error handling problem, not a defect in supported function as I first thought.

Thank you

Comment 10 Stephen Wadeley 2020-09-05 11:31:07 UTC
Hello

I created CV with fake errata
I created empty CV and published it

~]# hammer content-view info --id 308
ID:                           308
Name:                         empty
Label:                        empty
<snip>
Versions:                     
 1) ID:        360
    Version:   1.0
    Published: 2020/09/05 11:22:56


~]# hammer content-view version incremental-update --content-view-version-id 360 --errata-ids 9343
[.........................................................................................................] [100%]
Content View: empty version 1.1
Added Content:

We see no content was added, which is expected as per comment 9, but now no 500 Internal Server Error,
so this bug is verified

Interesting to note, an incremental update is made, just without the erratum

Versions:                     
 1) ID:        360
    Version:   1.0
    Published: 2020/09/05 11:22:56
 2) ID:        361
    Version:   1.1
    Published: 2020/09/05 11:23:28

thank you

Comment 13 errata-xmlrpc 2020-10-27 13:05:24 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Important: Satellite 6.8 release), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:4366

Comment 14 errata-xmlrpc 2020-10-27 13:08:10 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Important: Satellite 6.8 release), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:4366