release-engineering / ubi-population-tool

A tool for populating UBI repositories.
GNU General Public License v3.0
3 stars 14 forks source link

Make publishing of repos optional [RHELDST-21951] #221

Closed drepelov closed 8 months ago

drepelov commented 8 months ago

After the rhel-ubi repos sync, this tool also took care of publishing the affected ubi repos to the CDN. However, this did not work for Exodus. It was decided that all publishes should be done via Pub, which is already configured and works for Exodus.

This commit prepares this tool for removal of publishing by making it possible to switch the publishing on/off as needed.

It also removes the intended deduplication of srpms in _get_pulp_actions_src_pkgs() which was a bit confusing and also unnecessary, since deduplication is done during manifest creation. Finally, it removes some forgotten unused variables and renames some variables so it's more understandable.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4a0dbf0) 96.89% compared to head (c0dbf3a) 96.96%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #221 +/- ## ========================================== + Coverage 96.89% 96.96% +0.07% ========================================== Files 8 8 Lines 998 990 -8 ========================================== - Hits 967 960 -7 + Misses 31 30 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.