Bug #24749
closedhammer content-view version info does not require mandatory organization
Description
Description of problem:
The API of content-view version info (called from hammer) does not require the mandatory organization.
This results in wrong behaviour:
- Specifying incorrect version is specified then an incorrect error because the filter is missing the organization.
- Specifying a version that is also in an other content view version available it returns error that more than one found
Without organization
-----------------------------------------------
[crash/LI] root@li-lc-1578:~# sudo -u hoici hammer -c /opt/hoici/etc/sat6/hammer-hoici.yaml content-view version info --content-view sat-6.3-ci --version 999
Error: one of content_view_versions not found
[crash/LI] root@li-lc-1578:~# sudo -u hoici hammer -c /opt/hoici/etc/sat6/hammer-hoici.yaml content-view version info --content-view sat-6.3-ci --version 2
Error: found more than one content_view_version
- Now (with luck) a version that is unique for all CVV returns the data:
crash/LI] root@li-lc-1578:~# sudou hoici hammer -c /opt/hoici/etc/sat6/hammer-hoici.yaml content-view version info --content-view sat-6.3-ci --version 19 | grep '^[^ ]'
ID: 1244
Name: hgp-crash:sat-6.3-ci 19.0
Version: 19.0
Description:
Content View ID: 126
Content View Name: hgp-crash:sat-6.3-ci
Content View Label: hgp-crash_sat-6_3-ci
Lifecycle Environments:
Repositories:
Puppet Modules:
----------------------------------------------
Now adding the correct Organization:
-----------------------------------------------
[crash/LI] root@li-lc-1578:~# sudo -u hoici hammer -c /opt/hoici/etc/sat6/hammer-hoici.yaml content-view version info --content-view sat-6.3-ci --version 999 --organization Hilti
Error: content_view not found
[crash/LI] root@li-lc-1578:~# sudo u hoici hammer -c /opt/hoici/etc/sat6/hammer-hoici.yaml content-view version info --content-view sat-6.3-ci --version 2 --organization Hilti
Error: content_view not found
----------------------------------------------
Version-Release number of selected component (if applicable):
How reproducible:
Steps to Reproduce:
1. See calls in description
2.
3.
Actual results:
Unexpected results or incorrect errors
Expected results:
Correct results and errors
Updated by The Foreman Bot over 6 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/7661 added
Updated by Justin Sherrill over 6 years ago
- Subject changed from API content-view version info does not require mandatory organization to hammer content-view version info does not require mandatory organization
- Category changed from API to Hammer
- Status changed from Ready For Testing to Assigned
- Target version set to Katello 3.9.0
- Triaged changed from No to Yes
- Pull request deleted (
https://github.com/Katello/katello/pull/7661)
Updated by Zach Huntington-Meath over 6 years ago
- Target version changed from Katello 3.9.0 to Katello 3.10.0
Updated by Christine Fouant over 6 years ago
- Target version changed from Katello 3.10.0 to Katello 3.11.0
Updated by Jonathon Turel about 6 years ago
- Target version changed from Katello 3.11.0 to Katello 3.12.0
Updated by Chris Roberts almost 6 years ago
- Target version changed from Katello 3.12.0 to Katello Backlog
Updated by Chris Roberts over 5 years ago
- Status changed from Assigned to Rejected
Closing since downstream BZ is closed
Updated by Jonathon Turel almost 5 years ago
- Target version changed from Katello Backlog to Katello Recycle Bin