Bug 2162130

Summary:	hammer acs show does not show any SSL related fields
Product:	Red Hat Satellite	Reporter:	Vladimír Sedmík <vsedmik>
Component:	Alternate Content Sources	Assignee:	Chris Roberts <chrobert>
Status:	CLOSED ERRATA	QA Contact:	Jeremy Lenz <jlenz>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	6.13.0	CC:	chrobert, jlenz, pcreech
Target Milestone:	6.13.0	Keywords:	Triaged
Target Release:	Unused
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:	rubygem-hammer_cli_katello-1.7.3-1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2023-05-03 13:24:33 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 Vladimír Sedmík 2023-01-18 20:47:22 UTC

Description of problem:
`hammer acs show` does not show any SSL related fields (verify-ssl, ssl CA cert, ssl client cert, ssl client key) and use-http-proxies.


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


How reproducible:
always


Steps to Reproduce:
1. hammer acs show --id <id>


Actual results:
[root@sat ~]# hammer alternate-content-source show --id 165
ID:                            165
Name:                          Custom yum ACS
Label:                         Custom_yum_ACS
Description:                   
Base URL:                      https://fixtures.coolproject.org/
Content type:                  yum
Alternate content source type: custom
Subpaths:                      
 1) : rpm-zchunk/
Smart proxies:                 
 1) Id:              1
    Name:            satellite.redhat.com
    URL:             https://satellite.redhat.com:9090
    Download policy: on_demand


Expected results:
It might be useful to display them when they are set.

Comment 1 Brad Buckingham 2023-01-23 14:37:18 UTC

Hi Vladimir,

Does the UI or API show the SSL fields?

Thanks!

Comment 2 Vladimír Sedmík 2023-01-24 12:40:38 UTC

Hey Brad,

yes, UI and API show the fields, just the hammer does not pick them

# curl -sku admin:password -H "Content-type: application/json" -X GET https://$(hostname)/katello/api/alternate_content_sources/171 | json_reformat
{
    ...
    "verify_ssl": false,
    "use_http_proxies": false,
    "id": 171,
    "ssl_ca_cert": null,
    "ssl_client_cert": {
        "id": 2,
        "name": "RHUI"
    },
    "ssl_client_key": {
        "id": 2,
        "name": "RHUI"
    },
    ...
}

Comment 3 Bryan Kearney 2023-02-13 20:03:17 UTC

Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/36052 has been resolved.

Comment 5 Jeremy Lenz 2023-03-13 19:34:50 UTC

Verified in 6.13 snap 14

1. I have the correct version of hammer-cli-katello. Clearing needinfo.

[root@dhcp-3-54 ~]# rpm -qa | grep hammer*
rubygem-hammer_cli_katello-1.7.3-1.el8sat.noarch

2. Verify SSL, SSL CA Cert, SSL Client Cert, and SSL Client Key are all shown in the hammer output.

[root@dhcp-3-54 ~]# hammer alternate-content-source show --id 1
ID:                            1
Name:                          myCustom
Label:                         myCustom
Description:                   
Base URL:                      http://google.com
Content type:                  yum
Alternate content source type: custom
Verify SSL:                    true
SSL CA Cert:                   
 1) Id:   1
    Name: Debug Cert dhcp-3-54.vms.sat.rdu2.redhat.com
SSL Client Cert:               
 1) Id:   1
    Name: Debug Cert dhcp-3-54.vms.sat.rdu2.redhat.com
SSL Client Key:                
 1) Id:   1
    Name: Debug Cert dhcp-3-54.vms.sat.rdu2.redhat.com
Subpaths:                      

Smart proxies:                 
 1) Id:              1
    Name:            dhcp-3-54.vms.sat.rdu2.redhat.com
    URL:             https://dhcp-3-54.vms.sat.rdu2.redhat.com:9090
    Download policy: on_demand

Comment 8 errata-xmlrpc 2023-05-03 13:24:33 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