RFR: 1199: Enforce maintainer approval in JBS before allowing to integrate backports into updates projects [v2]
Zhao Song
zsong at openjdk.org
Thu Jun 29 16:30:12 UTC 2023
> As Erik said in the description of this issue, currently, this issue only cares about tracking approval labels in the related bugs.
>
> If a repository is set up with the "approval" configuration, pull requests in that repository will require the maintainer's approval in JBS. Otherwise, the pull request will not be considered ready.
>
> Erik has also provided a design outlining how to configure the "approval" for a repository.
>
>
> The simple case, where the labels are the same for every branch in a repository:
>
> "approval": {
> "request": "jdk17u-fix-request",
> "approved": "jdk17u-fix-yes",
> "rejected": "jdk17u-fix-no",
> }
>
> To reduce the need for changing multiple strings when copying a configuration for a new repository, there is an optional "prefix" field:
>
> "approval": {
> "prefix": "jdk17u-fix-",
> "request": "request",
> "approved": "yes",
> "rejected": "no",
> }
>
> When there are multiple branches with different labels, having the prefix set per branch can help reduce the size of the configuration significantly:
>
> "approval": {
> "request": "-critical-request",
> "approved": "-critical-approved",
> "rejected": "-critical-rejected",
> "branches": [
> "jdk20\.0\.1": { "prefix": "CPU23_04" }
> ]
> }
Zhao Song has updated the pull request incrementally with two additional commits since the last revision:
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
-------------
Changes:
- all: https://git.openjdk.org/skara/pull/1537/files
- new: https://git.openjdk.org/skara/pull/1537/files/a5eb7f35..83196d1a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=skara&pr=1537&range=01
- incr: https://webrevs.openjdk.org/?repo=skara&pr=1537&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/skara/pull/1537.diff
Fetch: git fetch https://git.openjdk.org/skara.git pull/1537/head:pull/1537
PR: https://git.openjdk.org/skara/pull/1537
More information about the skara-dev
mailing list