Issue #1471
closedDocumented yum importer option named 'skip' does not work
Description
The yum importer[0] documentation indicates that the yum importer takes an option named 'skip'. If you try to use this, it won't work. I believe the actual name should be type_skip_list.
When you use the --skip option to pulp-admin when creating a repo for instance:
pulp-admin -vv rpm repo create --repo-id rawhide --feed http://dl.fedoraproject.org/pub/fedora/linux/development/rawhide/x86_64/os/ --skip=rpm,drpm,erratum
pulp-admin will make the API call which shows the name as type_skip_list in the importer config:
2016-01-04 22:33:52,751 - INFO - POST request to /pulp/api/v2/repositories/ with parameters {"display_name": null, "description": null, "distributors": [{"distributor_id": "yum_distributor", "auto_publish": true, "distributor_config": {"skip": ["rpm", "drpm", "erratum"], "http": false, "relative_url": "/pub/fedora/linux/development/rawhide/x86_64/os/", "https": true}, "distributor_type_id": "yum_distributor"}, {"distributor_id": "export_distributor", "auto_publish": false, "distributor_config": {"skip": ["rpm", "drpm", "erratum"], "http": false, "https": true}, "distributor_type_id": "export_distributor"}], "notes": {"_repo-type": "rpm-repo"}, "importer_type_id": "yum_importer", "importer_config": {"feed": "http://dl.fedoraproject.org/pub/fedora/linux/development/rawhide/x86_64/os/", "type_skip_list": ["rpm", "drpm", "erratum"]}, "id": "rawhide"}
I think the right thing to do here is to have the yum importer docs updated to have 'skip' be renamed to the correct name 'type_skip_list'. If we want to keep the docs the same it will be more complicated with a importer cleanup migration, API validation changes, pulp-admin changes, tests, etc.
[0]: http://pulp-rpm.readthedocs.org/en/latest/tech-reference/yum-plugins.html#yum-importer
Related issues
Updated by bmbouter almost 9 years ago
- Related to Issue #1472: Yum import option 'skip' recommends several invalid values as options to use added
Updated by mhrivnak almost 9 years ago
- Triaged changed from No to Yes
- Tags Easy Fix added
Updated by bmbouter over 5 years ago
- Status changed from NEW to CLOSED - WONTFIX
Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.