RFR: 1199: Enforce maintainer approval in JBS before allowing to integrate backports into updates projects [v6]
Zhao Song
zsong at openjdk.org
Fri Aug 11 18:26:08 UTC 2023
On Fri, 11 Aug 2023 16:31:32 GMT, Zhao Song <zsong at openjdk.org> wrote:
>> 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
> - 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>
The GHA test failure is not related to this change. See: https://github.com/openjdk/skara/pull/1543
-------------
PR Comment: https://git.openjdk.org/skara/pull/1537#issuecomment-1675195912
More information about the skara-dev
mailing list