From duke at openjdk.org Tue Oct 1 04:06:22 2024 From: duke at openjdk.org (Johnny Lim) Date: Tue, 1 Oct 2024 04:06:22 GMT Subject: RFR: Upgrade GitHub Action Runner image for macOS to macos-12 Message-ID: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> This PR upgrades GitHub Action Runner image for macOS to `macos-12` as `macos-11` seems to have been removed. See https://github.com/actions/runner-images?tab=readme-ov-file#available-images and https://github.com/actions/runner-images/issues/9255 ------------- Commit messages: - Upgrade GitHub Action Runner image for macOS to macos-12 Changes: https://git.openjdk.org/jcstress/pull/152/files Webrev: https://webrevs.openjdk.org/?repo=jcstress&pr=152&range=00 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jcstress/pull/152.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/152/head:pull/152 PR: https://git.openjdk.org/jcstress/pull/152 From duke at openjdk.org Tue Oct 1 04:26:47 2024 From: duke at openjdk.org (Johnny Lim) Date: Tue, 1 Oct 2024 04:26:47 GMT Subject: RFR: 7903848: JCStress: Use "out" field consistently In-Reply-To: <7qGacUHcT8tfvBD8PXC1ame7HkuzU9RDqRXfb8ivdLE=.6040070d-b3af-43c3-9d30-79c305c7559f@github.com> References: <3Vvd0I7Giez2y7MZIFUsLwvUH2VrmjWV3DvgX0lj_oc=.6bbd2919-b1ce-4a28-9dc1-ed6797d244ae@github.com> <7qGacUHcT8tfvBD8PXC1ame7HkuzU9RDqRXfb8ivdLE=.6040070d-b3af-43c3-9d30-79c305c7559f@github.com> Message-ID: On Mon, 30 Sep 2024 08:20:38 GMT, Aleksey Shipilev wrote: > GHA tests on Mac are failing, I assume this is environmental? @shipilev `macos-11` seems to have been removed. I created https://github.com/openjdk/jcstress/pull/152 to try to resolve it. ------------- PR Comment: https://git.openjdk.org/jcstress/pull/151#issuecomment-2384757660 From shade at openjdk.org Tue Oct 1 08:21:52 2024 From: shade at openjdk.org (Aleksey Shipilev) Date: Tue, 1 Oct 2024 08:21:52 GMT Subject: RFR: 7903850: JCStress: Upgrade GitHub Action Runner image for macOS In-Reply-To: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> References: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> Message-ID: <_dpa35DjEOX1lBrEpiEhh4H-WR-rBkUx82dYWfds51Y=.4b9c4148-e38f-47fe-bb60-2dd32c9ad61d@github.com> On Tue, 1 Oct 2024 04:01:03 GMT, Johnny Lim wrote: > This PR upgrades GitHub Action Runner image for macOS to `macos-12` as `macos-11` seems to have been removed. > > See https://github.com/actions/runner-images?tab=readme-ov-file#available-images and https://github.com/actions/runner-images/issues/9255 Changes requested by shade (Committer). .github/workflows/pre-integration.yml line 21: > 19: build-java: [17, 21] > 20: run-java: [8, 11, 17, 21] > 21: os: [ubuntu-22.04, windows-2022, macos-12] Cool, thanks! Let's go straight to `macos-13`. ------------- PR Review: https://git.openjdk.org/jcstress/pull/152#pullrequestreview-2339454623 PR Review Comment: https://git.openjdk.org/jcstress/pull/152#discussion_r1782319399 From duke at openjdk.org Tue Oct 1 10:55:03 2024 From: duke at openjdk.org (Johnny Lim) Date: Tue, 1 Oct 2024 10:55:03 GMT Subject: RFR: 7903850: JCStress: Upgrade GitHub Action Runner image for macOS [v2] In-Reply-To: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> References: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> Message-ID: > This PR upgrades GitHub Action Runner image for macOS to `macos-12` as `macos-11` seems to have been removed. > > See https://github.com/actions/runner-images?tab=readme-ov-file#available-images and https://github.com/actions/runner-images/issues/9255 Johnny Lim has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: Upgrade GitHub Action Runner image for macOS to macos-13 ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/152/files - new: https://git.openjdk.org/jcstress/pull/152/files/207580f9..b33edc67 Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=152&range=01 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=152&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jcstress/pull/152.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/152/head:pull/152 PR: https://git.openjdk.org/jcstress/pull/152 From duke at openjdk.org Tue Oct 1 11:29:50 2024 From: duke at openjdk.org (Johnny Lim) Date: Tue, 1 Oct 2024 11:29:50 GMT Subject: RFR: 7903850: JCStress: Upgrade GitHub Action Runner image for macOS [v2] In-Reply-To: <_dpa35DjEOX1lBrEpiEhh4H-WR-rBkUx82dYWfds51Y=.4b9c4148-e38f-47fe-bb60-2dd32c9ad61d@github.com> References: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> <_dpa35DjEOX1lBrEpiEhh4H-WR-rBkUx82dYWfds51Y=.4b9c4148-e38f-47fe-bb60-2dd32c9ad61d@github.com> Message-ID: On Tue, 1 Oct 2024 08:19:26 GMT, Aleksey Shipilev wrote: >> Johnny Lim has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: >> >> Upgrade GitHub Action Runner image for macOS to macos-13 > > .github/workflows/pre-integration.yml line 21: > >> 19: build-java: [17, 21] >> 20: run-java: [8, 11, 17, 21] >> 21: os: [ubuntu-22.04, windows-2022, macos-12] > > Cool, thanks! Let's go straight to `macos-13`. @shipilev Sure, done! ------------- PR Review Comment: https://git.openjdk.org/jcstress/pull/152#discussion_r1782597865 From shade at openjdk.org Tue Oct 1 11:52:48 2024 From: shade at openjdk.org (Aleksey Shipilev) Date: Tue, 1 Oct 2024 11:52:48 GMT Subject: RFR: 7903850: JCStress: Upgrade GitHub Action Runner image for macOS [v2] In-Reply-To: References: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> Message-ID: On Tue, 1 Oct 2024 10:55:03 GMT, Johnny Lim wrote: >> This PR upgrades GitHub Action Runner image for macOS to `macos-12` as `macos-11` seems to have been removed. >> >> See https://github.com/actions/runner-images?tab=readme-ov-file#available-images and https://github.com/actions/runner-images/issues/9255 > > Johnny Lim has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: > > Upgrade GitHub Action Runner image for macOS to macos-13 Marked as reviewed by shade (Committer). ------------- PR Review: https://git.openjdk.org/jcstress/pull/152#pullrequestreview-2339972591 From duke at openjdk.org Tue Oct 1 12:00:52 2024 From: duke at openjdk.org (duke) Date: Tue, 1 Oct 2024 12:00:52 GMT Subject: RFR: 7903850: JCStress: Upgrade GitHub Action Runner image for macOS [v2] In-Reply-To: References: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> Message-ID: On Tue, 1 Oct 2024 10:55:03 GMT, Johnny Lim wrote: >> This PR upgrades GitHub Action Runner image for macOS to `macos-12` as `macos-11` seems to have been removed. >> >> See https://github.com/actions/runner-images?tab=readme-ov-file#available-images and https://github.com/actions/runner-images/issues/9255 > > Johnny Lim has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: > > Upgrade GitHub Action Runner image for macOS to macos-13 @izeye Your change (at version b33edc676980de4796f041bb0765846fa6e10b8c) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jcstress/pull/152#issuecomment-2385585626 From duke at openjdk.org Tue Oct 1 12:57:50 2024 From: duke at openjdk.org (Johnny Lim) Date: Tue, 1 Oct 2024 12:57:50 GMT Subject: Integrated: 7903850: JCStress: Upgrade GitHub Action Runner image for macOS In-Reply-To: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> References: <_pb9fC2Rp0gKUVwG6B0blT2wffzChMlAxp2ACvkvNgY=.79c868dc-d442-4b98-a9a9-4b766f151d34@github.com> Message-ID: On Tue, 1 Oct 2024 04:01:03 GMT, Johnny Lim wrote: > This PR upgrades GitHub Action Runner image for macOS to `macos-12` as `macos-11` seems to have been removed. > > See https://github.com/actions/runner-images?tab=readme-ov-file#available-images and https://github.com/actions/runner-images/issues/9255 This pull request has now been integrated. Changeset: eb1a23c8 Author: Johnny Lim Committer: Aleksey Shipilev URL: https://git.openjdk.org/jcstress/commit/eb1a23c8e7fbc0e1157d36f22ea4b5e2413ca788 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod 7903850: JCStress: Upgrade GitHub Action Runner image for macOS Reviewed-by: shade ------------- PR: https://git.openjdk.org/jcstress/pull/152 From duke at openjdk.org Tue Oct 1 13:02:24 2024 From: duke at openjdk.org (Johnny Lim) Date: Tue, 1 Oct 2024 13:02:24 GMT Subject: RFR: 7903848: JCStress: Use "out" field consistently [v2] In-Reply-To: <3Vvd0I7Giez2y7MZIFUsLwvUH2VrmjWV3DvgX0lj_oc=.6bbd2919-b1ce-4a28-9dc1-ed6797d244ae@github.com> References: <3Vvd0I7Giez2y7MZIFUsLwvUH2VrmjWV3DvgX0lj_oc=.6bbd2919-b1ce-4a28-9dc1-ed6797d244ae@github.com> Message-ID: > This PR changes to use `out` field consistently in the `JCStress` as it would be better for internal consistency although it's effectively same as before. Johnny Lim has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - Merge branch 'master' into polish-JCStress - Use "out" field consistently in JCStress ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/151/files - new: https://git.openjdk.org/jcstress/pull/151/files/3e54942b..403cc88e Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=151&range=01 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=151&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jcstress/pull/151.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/151/head:pull/151 PR: https://git.openjdk.org/jcstress/pull/151 From duke at openjdk.org Tue Oct 1 13:27:50 2024 From: duke at openjdk.org (duke) Date: Tue, 1 Oct 2024 13:27:50 GMT Subject: RFR: 7903848: JCStress: Use "out" field consistently [v2] In-Reply-To: References: <3Vvd0I7Giez2y7MZIFUsLwvUH2VrmjWV3DvgX0lj_oc=.6bbd2919-b1ce-4a28-9dc1-ed6797d244ae@github.com> Message-ID: <9AUeJ6WAiswVgSC4Bi55MdLjeEJJNOxPlBixktsh2dI=.bbe94924-97f1-4c78-a51a-4b23b72e4e54@github.com> On Tue, 1 Oct 2024 13:02:24 GMT, Johnny Lim wrote: >> This PR changes to use `out` field consistently in the `JCStress` as it would be better for internal consistency although it's effectively same as before. > > Johnny Lim has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: > > - Merge branch 'master' into polish-JCStress > - Use "out" field consistently in JCStress @izeye Your change (at version 403cc88e16b0c195b90a608660ae894b8137282a) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jcstress/pull/151#issuecomment-2385851000 From duke at openjdk.org Tue Oct 1 13:32:55 2024 From: duke at openjdk.org (Johnny Lim) Date: Tue, 1 Oct 2024 13:32:55 GMT Subject: Integrated: 7903848: JCStress: Use "out" field consistently In-Reply-To: <3Vvd0I7Giez2y7MZIFUsLwvUH2VrmjWV3DvgX0lj_oc=.6bbd2919-b1ce-4a28-9dc1-ed6797d244ae@github.com> References: <3Vvd0I7Giez2y7MZIFUsLwvUH2VrmjWV3DvgX0lj_oc=.6bbd2919-b1ce-4a28-9dc1-ed6797d244ae@github.com> Message-ID: <093-PlBv4KnnPO0p1OjQ47MgjjDgU-KENma3Wgk78yg=.f80b362a-7b3f-48a6-bcd6-c2c47c2ed190@github.com> On Sat, 28 Sep 2024 13:41:31 GMT, Johnny Lim wrote: > This PR changes to use `out` field consistently in the `JCStress` as it would be better for internal consistency although it's effectively same as before. This pull request has now been integrated. Changeset: 95384bab Author: Johnny Lim Committer: Aleksey Shipilev URL: https://git.openjdk.org/jcstress/commit/95384babf87d2dffc4b0d81b0558cd9fc2a6fea1 Stats: 5 lines in 1 file changed: 0 ins; 0 del; 5 mod 7903848: JCStress: Use "out" field consistently Reviewed-by: shade ------------- PR: https://git.openjdk.org/jcstress/pull/151