Project

Profile

Help

Issue #1458

Updated by jcline@redhat.com over 8 years ago

Currently, the Errata's pushcount field is a StringField, although pushcount should always be an integer. As part of https://pulp.plan.io/issues/1427 it was changed to an IntField, but this introduced problems with syncing repositories that have errata that do not have a pushcount field. It was difficult to decide on the correct default value (possibly 1, 0, or None). In addition to deciding on a default value, it may be wise to address the code in the distributor that handles publishing the errata.  

 Particularly, https://github.com/pulp/pulp_rpm/blob/28c520aadffb4f6256681e218477ec4a648fc310/plugins/pulp_rpm/plugins/distributors/yum/metadata/updateinfo.py#L52 https://github.com/pulp/pulp_rpm/commit/944a05f7c131893a3fd46f1a67a53e4a618d95d6#L52 is concerning since a default value of 0 causes no pushcount to be published, and a default value of 1 does cause a pushcount to be published.

Back