-
Notifications
You must be signed in to change notification settings - Fork 192
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do not force critpath packages to stay in testing for long pre-beta #4108
Conversation
If a release is not composed by bodhi, we should not force packages in critpath to stay 14 days in testing. Otherwise, between branching and beta, when bodhi is still in "rawhide" mode, critpath packages get stuck for 14 days. Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
This should do, though a better fix maybe to not update the critpath field pre-beta, but I didn't find where this is done in bodhi (and I'm running out of time to search it right now). Feel free to accept or reject if you find a better solution. Here is an example update that was stuck until I edited its critpath value in the DB: https://bodhi.fedoraproject.org/updates/FEDORA-2020-bac5ae64cf |
I think we must override the karma critpath setting also to make this work, but I'm not completely sure. Anyway, do we want to perform any check for rawhide releases? There's really no testing repository, so I don't think there's a reason to have any update stuck in testing. If gating is successful, I think any new update should be pushed directly to stable, shouldn't it? |
@@ -1993,7 +1993,7 @@ def mandatory_days_in_testing(self): | |||
:return: The number of mandatory days in testing. | |||
:rtype: int | |||
""" | |||
if self.critpath: | |||
if self.critpath and self.release.composed_by_bodhi: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should probably add a test for that new condition. What do you think ?
So, I would close this one in favor of #4135 We will need to adjust the configuration template in ansible for pre-beta: |
If a release is not composed by bodhi, we should not force packages in
critpath to stay 14 days in testing. Otherwise, between branching and
beta, when bodhi is still in "rawhide" mode, critpath packages get stuck
for 14 days.
Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr