Bug 1370108 - User can set checksum-type for ostree repos via hammer but that option is not available from UI
Summary: User can set checksum-type for ostree repos via hammer but that option is not...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Repositories
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: Unspecified
Assignee: Tom McKay
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-25 10:52 UTC by Sachin Ghai
Modified: 2018-09-04 17:47 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-09-04 17:47:09 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 17775 0 None None None 2016-12-19 20:37:12 UTC

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.


Note You need to log in before you can comment on or make changes to this bug.