#3684 PR code updated (new commits) Web Hook Event not sent (until I refresh the PR page)
Closed: Fixed 2 years ago Opened 2 years ago by fbo.

I'm working on an integration between Zuul and Pagure. The driver heavily relies on the Web Hook system of pagure.

I figured out than when I update a PR by adding new commits to the PR the event is not sent until I refresh the related PR page. Note that the pagure instance I'm using is pagure.io.

It looks like a bug, I expect to receive the event a soon as the PR is updated.


Metadata Update from @pingou:
- Issue set to the milestone: 5.1

2 years ago

Metadata Update from @pingou:
- Issue assigned to pingou

2 years ago

Metadata Update from @pingou:
- Issue tagged with: bug

2 years ago

Metadata Update from @fbo:
- Issue status updated to: Open (was: Closed)

2 years ago

I'm receiving notification when a PR commented but not when a new commit is added. So it looks like it is not fixed then I reopen the issue.

I'm receiving notification when a PR commented but not when a new commit is added. So it looks like it is not fixed then I reopen the issue.

Also receiving when the PR is created, but not the PR is updated (new commit)

Metadata Update from @pingou:
- Issue set to the milestone: 5.2 (was: 5.1)

2 years ago

Just to be sure, you know that notifications about a PR being updated/rebased will appear in the form a comment, right?

Yes I known that. The problem is I don't receive the comment notification since the 5.1.

Yes I known that. The problem is I don't receive the comment notification since the 5.1.

Could you give me more details about the project? Locally things seems to work.

It is https://pagure.io/test-zuul. I'll re-try again asap (tomorrow). I keep you updated there.

I believe this has been fixed in #4012 and #4013, for some reasons the change got undone so https://pagure.io/pagure/pull-request/4113 fixes this again :)

First tests show it is fixed. I think we can close the issue.

Login to comment on this ticket.

Metadata
Related Pull Requests
  • #3832 Merged 2 years ago