Project

General

Profile

Actions

Bug #24749

closed

hammer content-view version info does not require mandatory organization

Added by Chris Roberts over 5 years ago. Updated almost 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Category:
Hammer
Target version:
Difficulty:
easy
Triaged:
Yes
Fixed in Releases:
Found in Releases:

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

  1. Now (with luck) a version that is unique for all CVV returns the data:
    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 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

Actions #1

Updated by The Foreman Bot over 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/7661 added
Actions #2

Updated by Justin Sherrill over 5 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)
Actions #3

Updated by Zach Huntington-Meath over 5 years ago

  • Target version changed from Katello 3.9.0 to Katello 3.10.0
Actions #4

Updated by Christine Fouant over 5 years ago

  • Target version changed from Katello 3.10.0 to Katello 3.11.0
Actions #5

Updated by Jonathon Turel about 5 years ago

  • Target version changed from Katello 3.11.0 to Katello 3.12.0
Actions #6

Updated by Chris Roberts almost 5 years ago

  • Target version changed from Katello 3.12.0 to Katello Backlog
Actions #7

Updated by Chris Roberts over 4 years ago

  • Status changed from Assigned to Rejected

Closing since downstream BZ is closed

Actions #8

Updated by Jonathon Turel almost 4 years ago

  • Target version changed from Katello Backlog to Katello Recycle Bin
Actions

Also available in: Atom PDF