Bug 2166966

Summary: Candlepin 4.2.13 changes [:content][:id] to ['contentId']
Product: Red Hat Satellite Reporter: Jeremy Lenz <jlenz>
Component: Subscription ManagementAssignee: Ian Ballou <iballou>
Status: CLOSED ERRATA QA Contact: Chris Roberts <chrobert>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.13.0CC: chrobert, osousa, pcreech
Target Milestone: 6.13.0Keywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tfm-rubygem-katello-4.7.0.9-1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2023-05-03 13:24:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
screenshot showing issue fixed none

Description Jeremy Lenz 2023-02-03 18:08:34 UTC
We need to adapt our code to the new environment data scheme.

Without it content view publishes will fail.

Comment 1 Jeremy Lenz 2023-02-03 18:08:37 UTC
Created from redmine issue https://projects.theforeman.org/issues/36033

Comment 2 Jeremy Lenz 2023-02-03 18:08:38 UTC
Upstream bug assigned to iballou

Comment 4 Jeremy Lenz 2023-02-06 15:17:46 UTC
No user-facing APIs changed. Candlepin's API response from GET /environments/{env_id} changed between CP 4.1 and 4.2. This was called by Katello during content view publish.

old API: https://petstore.swagger.io/?url=https://raw.githubusercontent.com/candlepin/candlepin/candlepin-4.1-HOTFIX/api/candlepin-api-spec.yaml#/Environment/getEnvironment
new API: https://petstore.swagger.io/?url=https://raw.githubusercontent.com/candlepin/candlepin/master/api/candlepin-api-spec.yaml#/environment/getEnvironment

To reproduce, attempt to publish a content view. The issue will be present on any build that uses Candlepin 4.2.0 - 4.2.11 without this patch. The CV publish will fail with 'Error: undefined method `[]' for nil:NilClass.'

This fix isn't needed for Candlepin 4.1 AFAIU, so I don't think it should be needed for 6.12.z.

Comment 6 Chris Roberts 2023-03-20 15:35:29 UTC
[root@sat613snap ~]# rpm -qa | grep candlepin
candlepin-4.2.13-1.el8sat.noarch
candlepin-selinux-4.2.13-1.el8sat.noarch

Able to publish a content view with RH repos and custom content and also promote it to a new lifecycle env. See screenshots attached

Moving to VERIFIED.

Comment 7 Chris Roberts 2023-03-20 15:36:19 UTC
Created attachment 1952110 [details]
screenshot showing issue fixed

Comment 10 errata-xmlrpc 2023-05-03 13:24:54 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.13 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-2023:2097