Closed csomh closed 2 years ago
The problem was in the database. It's probably related to the fact that you created src#2 with the same title & description as src#1, packit refused to create a different dist-git MR and hardly couldn't add another relation into db, because the rpms#1 was already there in a relation with the src#1.
I fixed it in the database, so it works in this case. But it's a bug and has to be investigated and fixed.
Might be related to packit/packit-service#1504
(At least) two things need to be done here:
sync_release
will no longer return None
(but the PR itself) in case of already existing PRcheck_existing_pr: bool
argument to sync_release
which would allow us to skip the checking for already existing PRwe have to change the condition here because
sync_release
will no longer returnNone
(but the PR itself) in case of already existing PR
We need to handle the case when someone creates another source-git PR with the same title and description. Because now, packit refuses to create another PR. We either have to:
* add some more info into the dist-git PR description, which distinguishes it from the already existing PR
Actually, the dist-git MR description contains a link to the src-git MR so "when someone creates another source-git PR with the same title and description" the dist-git MR description would be different and hence this ("packit refuses to create another PR") could not happen.
I no longer understand how the problem was triggered (as I describe in https://github.com/packit/hardly/issues/70#issuecomment-1137086094) nor how to reproduce it.
Closing
Example:
This should work since #62, but for the MRs above, nothing happened.
Looking at the worker logs, the event was received and handled (the source-git repo was cloned and the dist-git repo was updated), but handling ended with a
PR already exists: https://gitlab.com/packit-service/rpms/systemd/-/merge_requests/1
message and nothing else was done.Re-opening the source-git MR produces the same behaviour, that is: there is no update in the dist-git MR.