Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Modify the updates fedmsg-client to check/set the CI status for new updates #1514

Closed
pypingou opened this issue May 12, 2017 · 7 comments · Fixed by #2086
Closed

Modify the updates fedmsg-client to check/set the CI status for new updates #1514

pypingou opened this issue May 12, 2017 · 7 comments · Fixed by #2086
Assignees
@bowlofeggs
Labels
High priority These issues are higher priority than normal RFE Requests for Enhancement

Comments

@pypingou
Copy link
Member

We need a fedmsg-client listening to messages from bodhi about new updates and edits to updates and checks the CI status of the builds associated with this update and adjust the CI status accordingly
@bowlofeggs bowlofeggs changed the title Create a fedmsg-client to check/set the CI status for new updates Modify the updates fedmsg-client to check/set the CI status for new updates May 12, 2017
@bowlofeggs
Copy link
Contributor

There is an existing "updates" consumer that listens for new updates to do background processing - I think it makes sense to tack this feature onto it:

https://github.com/fedora-infra/bodhi/blob/2.6.2/bodhi/server/consumers/updates.py

@bowlofeggs bowlofeggs added High priority These issues are higher priority than normal RFE Requests for Enhancement labels May 12, 2017
@pypingou pypingou self-assigned this May 12, 2017
@bowlofeggs
Copy link
Contributor

This is blocked on https://pagure.io/greenwave/issue/26.

@ralphbean
Copy link
Contributor

Are you unblocked now with https://pagure.io/greenwave/pull-request/27 ?

@bowlofeggs
Copy link
Contributor

@ralphbean Indeed!

@bowlofeggs
Copy link
Contributor

@pypingou You had assigned this to yourself, but I can probably do it if you don't still intend to. What say you?

@pypingou
Copy link
Member Author

@bowlofeggs go for it, thanks :)

@bowlofeggs bowlofeggs assigned bowlofeggs and unassigned pypingou Dec 15, 2017
@bowlofeggs bowlofeggs mentioned this issue Dec 15, 2017
Closed
bowlofeggs added a commit to bowlofeggs/bodhi that referenced this issue Dec 15, 2017
@bowlofeggs
Check Greenwave status when updates are created or edited.
5d89092 
fixes fedora-infra#1514

Signed-off-by: Randy Barlow <randy@electronsweatshop.com>
@bowlofeggs bowlofeggs mentioned this issue Dec 15, 2017
Merged
bowlofeggs added a commit to bowlofeggs/bodhi that referenced this issue Dec 16, 2017
@bowlofeggs
Check Greenwave status when updates are created or edited.
3a1ce5f 
fixes fedora-infra#1514

Signed-off-by: Randy Barlow <randy@electronsweatshop.com>
@bowlofeggs bowlofeggs mentioned this issue Dec 16, 2017
Merged
bowlofeggs added a commit to bowlofeggs/bodhi that referenced this issue Dec 20, 2017
@bowlofeggs
Check Greenwave status when updates are created or edited.
b837515 
fixes fedora-infra#1514

Signed-off-by: Randy Barlow <randy@electronsweatshop.com>
bowlofeggs added a commit to bowlofeggs/bodhi that referenced this issue Jan 12, 2018
@bowlofeggs
Check Greenwave status when updates are created or edited.
6704066 
fixes fedora-infra#1514

Signed-off-by: Randy Barlow <randy@electronsweatshop.com>
@bowlofeggs
Copy link
Contributor

The patch for this feature has been included in the 3.3.0b0 build, which is currently deployed to staging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bowlofeggs bowlofeggs

Labels
High priority These issues are higher priority than normal RFE Requests for Enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check Greenwave status when updates are created or edited. bowlofeggs/bodhi
3 participants
@ralphbean @bowlofeggs @pypingou