Bug 1155237

Summary:	Missing Checksum type value in hammer repository info
Product:	Red Hat Satellite	Reporter:	Elyézer Rezende <erezende>
Component:	Hammer	Assignee:	Brad Buckingham <bbuckingham>
Status:	CLOSED ERRATA	QA Contact:	Sachin Ghai <sghai>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	Nightly	CC:	bbuckingham, sghai
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
URL:	http://projects.theforeman.org/issues/8085
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-08-12 05:18:22 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 Elyézer Rezende 2014-10-21 16:14:29 UTC

Description of problem:
Hammer repository info should show the checksum type value. The checksum type could be specified during the creation but the info does not show its value.


Version-Release number of selected component (if applicable):
* hammer_cli_import (0.10.4)
* hammer_cli_katello (0.0.6)
* hammer_cli_foreman (0.1.3)

How reproducible:
Aways


Steps to Reproduce:
1. hammer repository create --checksum-type='sha256' # other options...
2. hammer repository info --id='<repo_id>'

Actual results:
ID:                 248
Name:               UmIHftKrfVbktJW
Label:              UmIHftKrfVbktJW
Organization:       5jWS1S
Red Hat Repository: no
Content Type:       yum
URL:                http://example.org/fakerepos/zoo3/
Publish Via HTTP:   yes
Published At:       http://server.example.com/pulp/repos/5jWS1S/Library/custom/EAiqOxutyBbenTEmFGNU/UmIHftKrfVbktJW
Product:
    ID:   1148
    Name: TCfnIdJoGgwPFKCSFlwi
GPG Key:

Sync:
    Status:
Created:            2014/10/21 16:07:11
Updated:            2014/10/21 16:07:12
Content Counts:

Expected results:
ID:                 248
Name:               UmIHftKrfVbktJW
Label:              UmIHftKrfVbktJW
Organization:       5jWS1S
Red Hat Repository: no
Content Type:       yum
URL:                http://example.org/fakerepos/zoo3/
Publish Via HTTP:   yes
Published At:       http://server.example.com/pulp/repos/5jWS1S/Library/custom/EAiqOxutyBbenTEmFGNU/UmIHftKrfVbktJW
Product:
    ID:   1148
    Name: TCfnIdJoGgwPFKCSFlwi
GPG Key:

Sync:
    Status:
Created:            2014/10/21 16:07:11
Updated:            2014/10/21 16:07:12
Content Counts:
Checksum type:      sha1|sha256|none

Additional info:

Comment 2 Brad Buckingham 2014-10-23 20:25:12 UTC

Created redmine issue http://projects.theforeman.org/issues/8085 from this bug

Comment 3 Brad Buckingham 2014-10-23 20:29:34 UTC

Proposed fix in:
  https://github.com/Katello/hammer-cli-katello/pull/245

hammer> repository info --id 10
ID:                 10
Name:               zoo-sha256
Label:              zoo-sha256
Organization:       Default Organization
Red Hat Repository: no
Content Type:       yum
Metadata Checksum:  sha256
URL:                
Publish Via HTTP:   yes
Published At:       http://fortello.devel/pulp/repos/Default_Organization/Library/custom/zoo/zoo-sha256
Product:            
    ID:   5
    Name: zoo
GPG Key:            

Sync:               
    Status:
Created:            2014/10/23 19:52:23
Updated:            2014/10/23 19:52:29
Content Counts:

Comment 4 Brad Buckingham 2014-10-29 00:37:13 UTC

Fix updated based upon QE feedback:

   https://github.com/Katello/hammer-cli-katello/pull/246

Comment 5 Bryan Kearney 2014-10-29 02:04:51 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/8085 has been closed
-------------
Brad Buckingham
Applied in changeset commit:hammer-cli-katello|2a572bed5f793a9912bd0070705f983e9b121ae8.

Comment 8 Sachin Ghai 2015-02-13 07:58:53 UTC

Verified with:

Satellite-6.1.0-RHEL-6-20150210.0
Sat6.1.0 beta snap2



[root@dhcp201-167 ~]# hammer --username admin --password changeme repository create  --url='http://omaciel.fedorapeople.org/fakerepo01/' --product-id='114' --name='r2' --publish-via-http=1  --content-type='yum' --gpg-key=g_key --checksum-type='sha256' --organization-id=3
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
Repository created
[root@dhcp201-167 ~]# hammer --username admin --password changeme repository list --organization-id=3
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
---|------|---------|--------------|--------------------------------------------
ID | NAME | PRODUCT | CONTENT TYPE | URL                                        
---|------|---------|--------------|--------------------------------------------
8  | r1   | p1      | yum          | http://omaciel.fedorapeople.org/fakerepo01/
9  | r2   | p1      | yum          | http://omaciel.fedorapeople.org/fakerepo01/
---|------|---------|--------------|--------------------------------------------
[root@dhcp201-167 ~]# hammer --username admin --password changeme repository info --id 9
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
ID:                 9
Name:               r2
Label:              r2
Organization:       testorg
Red Hat Repository: no
Content Type:       yum
Checksum Type:      sha256
URL:                http://omaciel.fedorapeople.org/fakerepo01/
Publish Via HTTP:   yes
Published At:       http://dhcp201-167.englab.pnq.redhat.com/pulp/repos/testorg/Library/custom/p1/r2
Product:            
    ID:   114
    Name: p1
GPG Key:            
    ID:   1
    Name: g_key
Sync:               
    Status:
Created:            2015/02/13 02:28:13
Updated:            2015/02/13 02:28:17
Content Counts:     
    Packages:       0
    Package Groups: 0
    Errata:         0

[root@dhcp201-167 ~]# 

I can see checksum type.

Comment 9 Bryan Kearney 2015-08-11 13:21:11 UTC

This bug is slated to be released with Satellite 6.1.

Comment 10 errata-xmlrpc 2015-08-12 05:18:22 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, 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-2015:1592