Bug 1370108

Summary: User can set checksum-type for ostree repos via hammer but that option is not available from UI
Product: Red Hat Satellite Reporter: Sachin Ghai <sghai>
Component: RepositoriesAssignee: Tom McKay <tomckay>
Status: CLOSED WONTFIX QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.2.0CC: bbuckingham, bkearney, paji
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-09-04 17:47:09 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 Sachin Ghai 2016-08-25 10:52:54 UTC
Description of problem:


~]# hammer -u admin -p changeme repository create --name test2 --content-type 'ostree' --product=p1 --url=https://kojipkgs.fedoraproject.org/atomic/23/ --publish-via-http=no --checksum-type=sha1 --organization-id=1
Repository created

Here I created a ostree type repo with checksum-type=`sha1` and repo is created and info shows the checksum-type. But 

1) is it a valid use case ? can user set checksum type for ostree repos ?
If not then we should raise validation error

If yes, then same functionality should be exposed on UI

[root@qe-sat6-rhel7 ~]# hammer -u admin -p changeme repository info --name test2 --product=p1 --organization-id 1
ID:                 211
Name:               test2
Label:              test2
Organization:       Default Organization
Red Hat Repository: no
Content Type:       ostree
Checksum Type:      sha1
URL:                https://kojipkgs.fedoraproject.org/atomic/23/
Publish Via HTTP:   no
Published At:       https://qe-sat6-rhel7.satqe.lab.eng.rdu2.redhat.com/pulp/ostree/web/Default_Organization/Library/custom/p1/test2
Relative Path:      Default_Organization/Library/custom/p1/test2
Product:            
    ID:   516
    Name: p1
GPG Key:            

Sync:               
    Status: Not Synced
Created:            2016/08/25 10:43:13
Updated:            2016/08/25 10:43:15
Content Counts:     
    OSTree Branches: 0


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:
Here I created a ostree type repo with checksum-type=`sha1` and repo is created and info shows the checksum-type. But 

1) is it a valid use case ? can user set checksum type for ostree repos ?
If not, then we should raise validation error

If yes, then same functionality should be exposed on UI

Additional info:

Comment 2 Partha Aji 2016-08-25 13:12:49 UTC

So setting checksum for ostree repos doesn't make sense. We should have a validation error of sorts if you are setting checksum type for sore type.

Comment 4 Bryan Kearney 2016-08-26 15:42:48 UTC
moving to backlog until fixed.

Comment 6 Brad Buckingham 2016-12-19 20:37:09 UTC
Created redmine issue http://projects.theforeman.org/issues/17775 from this bug

Comment 7 Bryan Kearney 2018-09-04 17:47:09 UTC
Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in the product in the foreseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.