Bug 2228909

Summary: Improve displayed filter rules info in hammer
Product: Red Hat Satellite Reporter: Vladimír Sedmík <vsedmik>
Component: Content ViewsAssignee: satellite6-bugs <satellite6-bugs>
Status: NEW --- QA Contact: Satellite QE Team <sat-qe-bz-list>
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.14.0CC: rlavi, sajha
Target Milestone: UnspecifiedKeywords: EasyFix, Triaged, UserExperience
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: 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 Vladimír Sedmík 2023-08-03 15:05:36 UTC 
Description of problem:
`hammer content-view version info --include-applied-filters true` shows new details about the filter rules now. Here are some suggestions for further improvement:

1. Filters always show the `Original packages` and `Original module streams` fields regardless the filter type. `Original packages` should be displayed for RPM filters only and `Original module streams` should be displayed for Module stream filter only.
[root@sat ~]# hammer content-view version info --include-applied-filters true --id 18
...
Has Applied Filters:    yes
Applied Filters:        
...
 5) Id:                      8
    Name:                    Container image tag filter
    Type:                    docker
    Inclusion:               yes
    Original packages:       no    <========
    Original module streams: no    <========
    Rules:                   
     1) Id:                     1
        Name:                   latest
        Content view filter Id: 8

2. Package group filters display pulp api UUID for each rule, which is not very user friendly. Package group name (like in WebUI) might look better.
 2) Id:                      5
    Name:                    Package groups
    Type:                    package_group
    Inclusion:               yes
    Original packages:       no
    Original module streams: no
    Rules:                   
     1) Id:                     1
        UUID:                   /pulp/api/v3/content/rpm/packagegroups/ebf75f01-4ac3-4649-b818-055104a36f57/
        Content view filter Id: 5
     2) Id:                     2
        UUID:                   /pulp/api/v3/content/rpm/packagegroups/e1afd706-c213-4989-9bca-29ee6b1f9a6a/
        Content view filter Id: 5

3. Filter rule info does not provide much useful information for some filter types, like modulemd. It should provide more info.
[root@sat ~]# hammer content-view filter rule info --content-view-filter-id 9 --id 1
Rule ID:   1
Filter ID: 9
Created:   2023/08/03 13:26:48
Updated:   2023/08/03 13:26:48


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


How reproducible:
always


Steps to Reproduce:
1. Run `hammer content-view version info --include-applied-filters true --id <cvv_id>` for some CVV with appropriate filters and rules applied.
2. Run `hammer content-view filter rule info --content-view-filter-id <cvf_id> --id <rule_id>`