RFR: 8320358: GHA: ignore jdk* branches

Aleksey Shipilev shade at openjdk.org
Tue Nov 21 15:13:06 UTC 2023


On Tue, 21 Nov 2023 14:42:07 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> At some point we are likely to use stabilization branches in the mainline jdk repo rather than a separate repo. In preparation, this PR excludes branches matching `jdk*`, like we currently do for `master` and `pr/*`.
> 
> A potential drawback of doing this is that it will exclude developer branches named `jdk-8888888` or similar, using a lower-case `jdk`. Developers who want a GHA run will need to use `JDK` (uppercase) or some other prefix.
> 
> This is unlikely to be a problem in practice. I checked the most recent 100 open pull requests in the `jdk` repo at the time I created this fix, and while many of them use "JDK" (upper case) as a prefix, I found none that use "jdk" (lower case).
> 
> 
> #### Testing
> 
> I pushed the following branch that was even with `jdk:master` at the time I pushed it (thus without this fix). GHA was run as expected:
> 
> * [jdk-8000000](https://github.com/kevinrushforth/jdk/tree/jdk-8000000) : [GHA run](https://github.com/kevinrushforth/jdk/actions/runs/6910232226) (NOTE: once this fix is integrated, such a branch would not get a GHA run)
> 
> 
> I pushed the following branches that all include this fix. GHA runs were skipped on the branches that start with `jdk` and run on the others:
> 
> * [JDK-8320358](https://github.com/kevinrushforth/jdk/tree/JDK-8320358) : [GHA run](https://github.com/kevinrushforth/jdk/actions/runs/6910192534), (you can also see this from this PR's test results)
> * [gha-exclude-jdk](https://github.com/kevinrushforth/jdk/tree/gha-exclude-jdk) : [GHA run](https://github.com/kevinrushforth/jdk/actions/runs/6910204060)
> * [jdk12345](https://github.com/kevinrushforth/jdk/tree/jdk12345) : No GHA run
> * [jdk-8320358-gha](https://github.com/kevinrushforth/jdk/tree/jdk-8320358-gha) : No GHA run

Looks to me it just asks for corner case problems. For example, bots regularly create backport branches like `openjdk-bots:${username}-backport-${hash}`, which means if anyone has a username starting from `jdk` (/me stares at general vicinity of @jdksjolen), then GHA would suddenly stop for them?

I _think_ we can avoid this with more precise glob pattern, e.g. `jdk[0-9]+*`, which would match `jdk21`, `jdk21u-dev`, etc.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16763#issuecomment-1821109102


More information about the build-dev mailing list