Bug 1379991

Summary: Inconsistency in Content View Filter sub-commands
Product: Red Hat Satellite Reporter: Oleksandr Shtaier <oshtaier>
Component: HammerAssignee: Andrew Kofink <akofink>
Status: CLOSED DUPLICATE QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.3.0CC: akofink, bbuckingham, jcallaha
Target Milestone: UnspecifiedKeywords: Regression, Triaged
Target Release: Unused   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-10-31 19:21:05 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 Oleksandr Shtaier 2016-09-28 11:39:22 UTC
Description of problem:
It seems that 'organization*' parameters are removed for 'update' and 'delete' sub-commands for content-view filter command, but not for 'create'. It is not obvious what scenario correct, so it necessary to make a decision and choose only one.

hammer content-view filter create [OPTIONS]

Options:
 --content-view CONTENT_VIEW_NAME        Content view name to search by
 --content-view-id CONTENT_VIEW_ID       content view numeric identifier
 --description DESCRIPTION               description of the filter
 --inclusion INCLUSION                   specifies if content should be included or excluded, default:
                                         inclusion=false
                                         One of true/false, yes/no, 1/0.
 --name NAME                             name of the filter
 --organization ORGANIZATION_NAME        Organization name to search by
 --organization-id ORGANIZATION_ID       Organization ID to search by
 --organization-label ORGANIZATION_LABEL Organization label to search by
 --original-packages ORIGINAL_PACKAGES   add all packages without errata to the included/excluded list. (package
                                         filter only)
                                         One of true/false, yes/no, 1/0.
 --repositories REPOSITORY_NAMES         Comma separated list of values.
 --repository-ids REPOSITORY_IDS         list of repository ids
                                         Comma separated list of values.
 --type TYPE                             type of filter (e.g. rpm, package_group, erratum)


hammer content-view filter delete [OPTIONS]

Options:
 --content-view CONTENT_VIEW_NAME  Content view name to search by
 --content-view-id CONTENT_VIEW_ID content view numeric identifier
 --id ID                           filter identifier
 --name NAME                       Name to search by

hammer content-view filter update [OPTIONS]

Options:
 --content-view CONTENT_VIEW_NAME      Content view name to search by
 --content-view-id CONTENT_VIEW_ID     content view numeric identifier
 --id ID                               filter identifier
 --inclusion INCLUSION                 specifies if content should be included or excluded, default:
                                       inclusion=false
                                       One of true/false, yes/no, 1/0.
 --name NAME                           Name to search by
 --new-name NEW_NAME                   new name for the filter
 --original-packages ORIGINAL_PACKAGES add all packages without errata to the included/excluded list. (package
                                       filter only)
                                       One of true/false, yes/no, 1/0.
 --repositories REPOSITORY_NAMES       Comma separated list of values.
 --repository-ids REPOSITORY_IDS       list of repository ids

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

How reproducible:
Always

Steps to Reproduce:
1.
2.
3.

Actual results:
There are no same approach in entity sub-commands

Expected results:
One approach should be chosen

Additional info:

Comment 3 Andrew Kofink 2016-10-31 19:21:05 UTC

*** This bug has been marked as a duplicate of bug 1378018 ***