Maintenance: Planio will be observing a scheduled maintenance window this Sunday, November 10, 2024 from 20:00 UTC until 21:00 UTC to perform important network maintenance in our primary data center. Your Planio account will be unavailable for a few minutes during this maintenance window.
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