Skip to content
This repository has been archived by the owner on Mar 22, 2021. It is now read-only.

PR testing doesn't happen if the message bus is down #33

Closed
cgwalters opened this issue Apr 13, 2017 · 7 comments
Closed

PR testing doesn't happen if the message bus is down #33

cgwalters opened this issue Apr 13, 2017 · 7 comments

Comments

@cgwalters
Copy link
Member

coreos/rpm-ostree#709 was pushed a while ago, but no PR status.

@cgwalters
Copy link
Member Author

Currently @jlebon thinks this is the internal Jenkins getting hung up, and we don't have access to debug/restart it.

@jlebon
Copy link
Collaborator

jlebon commented Apr 13, 2017

The internal message bus is currently down. I filed a ticket to have it investigated. There is only a single point through the event flow at which we still rely on the message bus rather than using it as an optimization with polling fallback. Will work to change that.

@jlebon
Copy link
Collaborator

jlebon commented Apr 13, 2017

Actually, auto branches should still be getting polled and tested. So we should see projectatomic/atomic#966 go through normally.

@jlebon
Copy link
Collaborator

jlebon commented Apr 13, 2017

(This is not specific to auto branches, just any branch test in general).

@jlebon
Copy link
Collaborator

jlebon commented Apr 13, 2017

Note that you should still be able to manually trigger a PR test by starting a build of the github-ci-testers-executor job and filling in the GitHub repo and pull id.

@jlebon
Copy link
Collaborator

jlebon commented Apr 18, 2017

This should be resolved now.

There is only a single point through the event flow at which we still rely on the message bus rather than using it as an optimization with polling fallback. Will work to change that.

Let's use this card to track this work.

@jlebon jlebon changed the title not getting PRs tested PR testing doesn't happen if the message bus is down Apr 18, 2017
@jlebon
Copy link
Collaborator

jlebon commented Apr 26, 2017

This should be fixed now. In the event of the message bus going down, PRs should still be tested within 5 mins.

@jlebon jlebon closed this as completed Apr 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants