When merging https://src.fedoraproject.org/rpms/zsh-syntax-highlighting/pull-request/1, I enabled the Pagure hook and added "Merges #1" to the commit message.
This resulted in the following error messages (sorry, I do not have the full log anymore): - No handlers could be found for logger "pagure.hooks.pagure_hook" - DetachedInstanceError: Parent instance <Project at 0x7f29ebd6b790> is not bound to a Session; lazy load operation of attribute 'issue_keys' cannot proceed - ProgrammingError('(ProgrammingError) permission denied for relation pull_requests
No handlers could be found for logger "pagure.hooks.pagure_hook"
DetachedInstanceError: Parent instance <Project at 0x7f29ebd6b790> is not bound to a Session; lazy load operation of attribute 'issue_keys' cannot proceed
ProgrammingError('(ProgrammingError) permission denied for relation pull_requests
Moreover, the commit did not close the PR automatically and no comment was added to the PR.
Seems to be related to #7530.
This is known in dist-git because of the many ways dist-git is special. The gist of it is that the git hook only have a read-only access to the database and are thus not able to edit the pull_requests table to mark the PR as merged :(
Metadata Update from @smooge: - Issue priority set to: Waiting on Assignee (was: Needs Review) - Issue tagged with: src.fp.o
Got it, thanks! Maybe a small warning can be added to the hook to make people aware of this?
Metadata Update from @pingou: - Issue assigned to pingou
Open a ticket upstream https://pagure.io/pagure-dist-git/issue/96
Metadata Update from @cverna: - Issue close_status updated to: Upstream - Issue status updated to: Closed (was: Open)
Log in to comment on this ticket.