Bug 2228909 - Improve displayed filter rules info in hammer
Summary: Improve displayed filter rules info in hammer
Keywords:
Status: NEW
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Content Views
Version: 6.14.0
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: Unspecified
Assignee: satellite6-bugs
QA Contact: Satellite QE Team
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-08-03 15:05 UTC by Vladimír Sedmík
Modified: 2023-08-11 15:34 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker SAT-19511 0 None None None 2023-08-11 15:34:06 UTC

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>`


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