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.

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

GitHub integration #119

Open
bonzini opened this issue Apr 16, 2019 · 6 comments
Open

GitHub integration #119

bonzini opened this issue Apr 16, 2019 · 6 comments

Comments

@bonzini
Copy link
Contributor

bonzini commented Apr 16, 2019

Write a new plugin that creates a pull request for every complete series, and closes it when the series is merged. The plugin could also use the GitHub webhook to mark series as merged/obsolete when a pull request is closed, and to report the state of GitHub checks as a status box.

@famz
Copy link
Collaborator

famz commented Apr 17, 2019

Out of curiosity: why do we want this?

@bonzini
Copy link
Contributor Author

bonzini commented Apr 17, 2019

Mostly to integrate GitHub checks with mailing list workflows, where patch application would be done with GitHub to ensure that checks gate the application. I am not planning to do it immediately, but I am opening the issue as a "reminder that it can be done".

@famz
Copy link
Collaborator

famz commented Apr 17, 2019

Okay.

We already pushes tags to github so checks can happen there, I'm not sure what more pull requests can provide.

We do want some webhook callback mechanism to get the results, though. For now we'll have to "poll" from the testing script, to get results.

@bonzini
Copy link
Contributor Author

bonzini commented Apr 17, 2019

Can you see the results of checks that are run on a tag? Pull requests would provide the UI for checks and for merging, basically.

(Laszlo asked what can be done to integrate GitHub CI with mailing list workflow).

@famz
Copy link
Collaborator

famz commented Apr 17, 2019

That is a good point. It's not easy afaict.

@bonzini
Copy link
Contributor Author

bonzini commented Apr 17, 2019

Also, since Patchew can parse the MAINTAINERS file, it could use that to assign reviewers on GitHub (e.g. M: Paolo Bonzini <https://github.com/bonzini>).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants