Deprecating Defaults for to_all_distros Setting in repoman

Hello All. If you use repoman please be aware that it currently has hardcoded list of packages that defaults to being distroless. This is controlled by "to_all_distros" configuration setting, but if you do not set it than the defaults are used. For ovirt experimental we decided that we better have all packages to have distro specified in spec. This also inline with most distro packaging guidelines. But then with current defaults repoman will consider those package to be distroless even when they are not and specifying an empty list in configuration is a silly way to override this. This change provides an incompatibility, but you can still use this feature by specifying to_all_distros package list explicitly in the config. So it is just the change of default values at this point: https://gerrit.ovirt.org/#/c/67662/ If somebody is not ok with defaults change and need time to update configuration please let me know. Also if you are not sure we can go another way and have intermediate repoman version that does not do the change, but issue deprecation warning if defaults are being used, so everybody can test their setups. However I think that it is might not be actually needed in this case as it is a configuration thing. Please feel free to comment on change request. Anton. -- Anton Marchukov Senior Software Engineer - RHEV CI - Red Hat

Il 01/Dic/2016 08:07 PM, "Anton Marchukov" <amarchuk@redhat.com> ha scritto:
Hello All.
If you use repoman please be aware that it currently has hardcoded list
of packages that defaults to being distroless. This is controlled by "to_all_distros" configuration setting, but if you do not set it than the defaults are used.
For ovirt experimental we decided that we better have all packages to
have distro specified in spec. This also inline with most distro packaging guidelines. But then with current defaults repoman will consider those package to be distroless even when they are not and specifying an empty list in configuration is a silly way to override this.
This change provides an incompatibility, but you can still use this
feature by specifying to_all_distros package list explicitly in the config. So it is just the change of default values at this point:
https://gerrit.ovirt.org/#/c/67662/
If somebody is not ok with defaults change and need time to update
configuration please let me know. Also if you are not sure we can go another way and have intermediate repoman version that does not do the change, but issue deprecation warning if defaults are being used, so everybody can test their setups. However I think that it is might not be actually needed in this case as it is a configuration thing.
Please feel free to comment on change request.
I guess the most impacted will be ovirt release engineering while working on 4.0 repos but being 4.1 coming in 2 months we can survive. Just keep at hand an old el7 rpm, just in case.
Anton.
-- Anton Marchukov Senior Software Engineer - RHEV CI - Red Hat
_______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra

I guess the most impacted will be ovirt release engineering while working on 4.0 repos but being 4.1 coming in 2 months we can survive. Just keep at hand an old el7 rpm, just in case.
But it will be possible to have the previous behavior just by updating repoman configuration file with the list of packages. So it will not be hardcoded inside the source code, but instead stored in local configuration. Just let me know whom to contact so we can update and test the repoman configuration before the change is merged to make sure it won't cause any issues. Anton. -- Anton Marchukov Senior Software Engineer - RHEV CI - Red Hat
participants (2)
-
Anton Marchukov
-
Sandro Bonazzola