fedora-infra / bodhi

Bodhi is a web-system that facilitates the process of publishing updates for a Fedora-based software distribution.
https://bodhi.fedoraproject.org
GNU General Public License v2.0
151 stars 189 forks source link

bodhi's repo checks don't allow disabling drpms #5616

Closed nirik closed 4 months ago

nirik commented 4 months ago

We are attempting to disable/not make drpms in f40 updates/updates-testing.

The initial f40 updates-testing compose fails with:

celery-3[2046089]: [2024-02-28 02:03:29,353: 
ERROR/ForkPoolWorker-16] ComposerThread failed. Transaction rolled back.                            
celery-3[2046089]: Traceback (most recent call last): 
celery-3[2046089]:   File "/usr/lib/python3.11/site-packages/bodhi/server/tasks/composer.py", line 340, in run                                   celery-3[2046089]:     self.work()           
celery-3[2046089]:   File "/usr/lib/python3.11/site-packages/bodhi/server/tasks/composer.py", line 411, in work                                  
celery-3[2046089]:     self._compose_updates()                                                                                                   
celery-3[2046089]:   File "/usr/lib/python3.11/site-packages/bodhi/server/tasks/composer.py", line 967, in _compose_updates                      celery-3[2046089]:     self._sanity_check_repo()                                                                                                 
celery-3[2046089]:   File "/usr/lib/python3.11/site-packages/bodhi/server/tasks/composer.py", line 1172, in _sanity_check_repo                   
celery-3[2046089]:     sanity_check_repodata(repodata, repo_type=repo_type)  
celery-3[2046089]:   File "/usr/lib/python3.11/site-packages/bodhi/server/util.py", line 324, in sanity_check_repodata                           celery-3[2046089]:     raise RepodataException(f'Required parts not in repomd.xml: {", ".join(missing)}') 
celery-3[2046089]: bodhi.server.exceptions.RepodataException: Required parts not in repomd.xml: prestodelta   

So, we need a way to disable this check when they are disabled. :(