Bug #19927
closed
Hammer csv import package rules doesn't handle rules with the same name
Description
Steps to reproduce:
1. Create a content view with a package filter
2. Create two rules on this filter with the same name (e.g. exclude vim version > 3.0 and vim version = 3.0)
3. Export your filters
4. Remove the filters
5. Do an import
Expected results:
Your filter should have two rules (e.g. exclude vim version > 3.0 and vim version = 3.0)
Actual result:
Your filter only has one rule, the second one (e.g. vim version = 3.0).
The problem is that name isn’t unique across filter rules but hammer csv assumes it is:
Updated by David Davis almost 8 years ago
- Blocks Bug #19724: Support importing/exporting erratum filters added
Updated by David Davis almost 8 years ago
- Subject changed from Hammer package filter rules doesn't handle rules with the same name to Hammer csv import package rules doesn't handle rules with the same name
Updated by Thomas McKay almost 8 years ago
Just an FYI... Note that many/all of the csv files expect that names are unique and don't use labels typically. Perhaps an additional column should be added for label and then use that first, falling back to name column.
Updated by David Davis almost 8 years ago
Your idea is good but not really applicable. Content view rules don't have labels. This makes updating them hard.
Updated by David Davis almost 8 years ago
Talking this over with @jsherrill, I think when importing (or updating) existing filters, it might make sense to drop all its rules and then re-add the ones from the csv.
Updated by The Foreman Bot almost 8 years ago
- Pull request https://github.com/Katello/hammer-cli-csv/pull/157 added
Updated by