From duke at openjdk.org Tue Sep 2 12:00:23 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Tue, 2 Sep 2025 12:00:23 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit Message-ID: Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue First Visit Second Visit ------------- Commit messages: - 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit Changes: https://git.openjdk.org/jmc/pull/669/files Webrev: https://webrevs.openjdk.org/?repo=jmc&pr=669&range=00 Issue: https://bugs.openjdk.org/browse/JMC-8436 Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jmc/pull/669.diff Fetch: git fetch https://git.openjdk.org/jmc.git pull/669/head:pull/669 PR: https://git.openjdk.org/jmc/pull/669 From schaturvedi at openjdk.org Wed Sep 3 08:36:52 2025 From: schaturvedi at openjdk.org (Suchita Chaturvedi) Date: Wed, 3 Sep 2025 08:36:52 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 14:34:14 GMT, Aymane Harmaz wrote: > Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue > > > First Visit > > Second Visit @aymane-harmaz The change looks good. But there is only one check which is performed. Please check why other 4 checks are not executed. ------------- PR Comment: https://git.openjdk.org/jmc/pull/669#issuecomment-3248233017 From aptmac at openjdk.org Wed Sep 3 13:27:48 2025 From: aptmac at openjdk.org (Alex Macdonald) Date: Wed, 3 Sep 2025 13:27:48 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 14:34:14 GMT, Aymane Harmaz wrote: > Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue > > > First Visit > > Second Visit You'll have to enable the workflows to run on your fork for the other checks to be performed, see: https://wiki.openjdk.org/display/SKARA/Testing#Testing-Configuringworkflowstorun ------------- PR Comment: https://git.openjdk.org/jmc/pull/669#issuecomment-3249251018 From hirt at openjdk.org Wed Sep 3 18:56:53 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Wed, 3 Sep 2025 18:56:53 GMT Subject: RFR: 6203: Add ZGC allocation stall rule [v3] In-Reply-To: References: Message-ID: On Wed, 27 Aug 2025 20:10:20 GMT, Marcus Hirt wrote: >> Suchita Chaturvedi has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits: >> >> - Resolving merge conflict >> - Updated the rule as per new metric allocation stall rate >> - 6203: Add ZGC allocation stall rule > > core/org.openjdk.jmc.flightrecorder.rules.jdk/src/main/java/org/openjdk/jmc/flightrecorder/rules/jdk/memory/ZGCAllocationStallRule.java line 139: > >> 137: >> 138: //Calculate time after JVM Start >> 139: IQuantity timeAfterJVMStart = RulesToolkit.getEarliestStartTime(items) > > Come to think of it - this will not work well for OldObjectSample events. They are a bit special and may be timestamped far earlier than the current recording period. I think we might want to add a RulesToolkit function that explicitly avoid the OldObjectSample events for this... Suchita - I checked the jdk.OldObjectSample event after your slack message, and it turns out that there is a separate allocationTime for that event, so this isn't a problem. The timestamp on the events seem to be from the time of emission, so it should be all good. ------------- PR Review Comment: https://git.openjdk.org/jmc/pull/664#discussion_r2319898491 From hirt at openjdk.org Wed Sep 3 19:16:50 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Wed, 3 Sep 2025 19:16:50 GMT Subject: RFR: 8425: Problem importing in Eclipse due to missing awatility 3rd party artefact In-Reply-To: References: Message-ID: On Sun, 27 Jul 2025 17:44:53 GMT, Martin Skarsaune wrote: > The Jolokia test plugin requires awaitility I think Virag just meant the target platform, where all third party dependencies are exported over a p2. ------------- PR Comment: https://git.openjdk.org/jmc/pull/668#issuecomment-3250464142 From schaturvedi at openjdk.org Thu Sep 4 07:33:27 2025 From: schaturvedi at openjdk.org (Suchita Chaturvedi) Date: Thu, 4 Sep 2025 07:33:27 GMT Subject: RFR: 6203: Add ZGC allocation stall rule [v4] In-Reply-To: References: Message-ID: > This enhancement is to add new rule for ZGC Allocation Stall events. > > The default configuration: > > image > > Here are few screenshots for reference: > > image > > image > > image > > image > > image > > Ignored > > image > > If we change default configuration as below: > > image > > image > > image > > image Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision: Implemented review comments ------------- Changes: - all: https://git.openjdk.org/jmc/pull/664/files - new: https://git.openjdk.org/jmc/pull/664/files/c1805581..4b543d9d Webrevs: - full: https://webrevs.openjdk.org/?repo=jmc&pr=664&range=03 - incr: https://webrevs.openjdk.org/?repo=jmc&pr=664&range=02-03 Stats: 6 lines in 3 files changed: 0 ins; 0 del; 6 mod Patch: https://git.openjdk.org/jmc/pull/664.diff Fetch: git fetch https://git.openjdk.org/jmc.git pull/664/head:pull/664 PR: https://git.openjdk.org/jmc/pull/664 From duke at openjdk.org Thu Sep 4 07:54:48 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Thu, 4 Sep 2025 07:54:48 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit In-Reply-To: References: Message-ID: On Wed, 3 Sep 2025 08:34:24 GMT, Suchita Chaturvedi wrote: > @aymane-harmaz The change looks good. But there is only one check which is performed. Please check why other 4 checks are not executed. @Suchitainf Now only the build and test on macOS-latest fails, I think this have to do with the migration of macOS-latest label from using macos-14 into using macos-15 I have looked at previous commits in the repository that have passed this job successfully and they all have been created before the migration (4th of august) ------------- PR Comment: https://git.openjdk.org/jmc/pull/669#issuecomment-3252366919 From schaturvedi at openjdk.org Thu Sep 4 07:54:49 2025 From: schaturvedi at openjdk.org (Suchita Chaturvedi) Date: Thu, 4 Sep 2025 07:54:49 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 07:49:48 GMT, Aymane Harmaz wrote: > > @aymane-harmaz The change looks good. But there is only one check which is performed. Please check why other 4 checks are not executed. > > @Suchitainf Now only the build and test on macOS-latest fails, I think this have to do with the migration of macOS-latest label from using macos-14 into using macos-15 > > I have looked at previous commits in the repository that have passed this job successfully and they all have been created before the migration (4th of august) @aymane-harmaz Yes, this failure is not related to your change. I have updated my ZGC PR and its failing for me also. ------------- PR Comment: https://git.openjdk.org/jmc/pull/669#issuecomment-3252375412 From duke at openjdk.org Thu Sep 4 15:22:36 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Thu, 4 Sep 2025 15:22:36 GMT Subject: RFR: 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions Message-ID: Skipped the execution of JDP tests requiring multicast which is not supported on macOS-15 runner This was the result of moving macOS-latest from 14 to 15 Similar issues are mentioned in the following links : https://github.com/orgs/community/discussions/170669 https://github.com/actions/runner-images/issues/10924 ------------- Commit messages: - 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions Changes: https://git.openjdk.org/jmc/pull/672/files Webrev: https://webrevs.openjdk.org/?repo=jmc&pr=672&range=00 Issue: https://bugs.openjdk.org/browse/JMC-8437 Stats: 20 lines in 1 file changed: 20 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jmc/pull/672.diff Fetch: git fetch https://git.openjdk.org/jmc.git pull/672/head:pull/672 PR: https://git.openjdk.org/jmc/pull/672 From schaturvedi at openjdk.org Thu Sep 4 17:48:48 2025 From: schaturvedi at openjdk.org (Suchita Chaturvedi) Date: Thu, 4 Sep 2025 17:48:48 GMT Subject: RFR: 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions In-Reply-To: References: Message-ID: <-OwcoOhPMRBdSFTWqkcJGoxFkW0MDHihurbQ-y4OZFo=.b78cb3ea-5ff4-412b-8184-e037547c068d@github.com> On Thu, 4 Sep 2025 15:16:46 GMT, Aymane Harmaz wrote: > Skipped the execution of JDP tests requiring multicast which is not supported on macOS-15 runner > > This was the result of moving macOS-latest from 14 to 15 > > Similar issues are mentioned in the following links : > https://github.com/orgs/community/discussions/170669 > https://github.com/actions/runner-images/issues/10924 Looks good to me. ------------- Marked as reviewed by schaturvedi (Committer). PR Review: https://git.openjdk.org/jmc/pull/672#pullrequestreview-3186424358 From hirt at openjdk.org Thu Sep 4 20:40:49 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Thu, 4 Sep 2025 20:40:49 GMT Subject: RFR: 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions In-Reply-To: References: Message-ID: <-TUfvBFiO0ChKV3u6Sc25lCID7meP_1Vt0zlKLS1Wcc=.e96a603a-56e0-49d2-959d-c459ba812fac@github.com> On Thu, 4 Sep 2025 15:16:46 GMT, Aymane Harmaz wrote: > Skipped the execution of JDP tests requiring multicast which is not supported on macOS-15 runner > > This was the result of moving macOS-latest from 14 to 15 > > Similar issues are mentioned in the following links : > https://github.com/orgs/community/discussions/170669 > https://github.com/actions/runner-images/issues/10924 Marked as reviewed by hirt (Lead). Hi @aymane-harmaz! Looks good for now. Let?s watch that thread to see if Apple comes up with a solution that lets us run these tests. Please /integrate. ------------- PR Review: https://git.openjdk.org/jmc/pull/672#pullrequestreview-3187176032 PR Comment: https://git.openjdk.org/jmc/pull/672#issuecomment-3255565286 From hirt at openjdk.org Thu Sep 4 20:42:53 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Thu, 4 Sep 2025 20:42:53 GMT Subject: RFR: 6203: Add ZGC allocation stall rule [v4] In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 07:33:27 GMT, Suchita Chaturvedi wrote: >> This enhancement is to add new rule for ZGC Allocation Stall events. >> >> The default configuration: >> >> image >> >> Here are few screenshots for reference: >> >> image >> >> image >> >> image >> >> image >> >> image >> >> Ignored >> >> image >> >> If we change default configuration as below: >> >> image >> >> image >> >> image >> >> image > > Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision: > > Implemented review comments The Mac JDP problem is unrelated, and is being fixed in #672. ------------- PR Comment: https://git.openjdk.org/jmc/pull/664#issuecomment-3255569926 From hirt at openjdk.org Thu Sep 4 21:45:04 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Thu, 4 Sep 2025 21:45:04 GMT Subject: RFR: 6203: Add ZGC allocation stall rule [v4] In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 07:33:27 GMT, Suchita Chaturvedi wrote: >> This enhancement is to add new rule for ZGC Allocation Stall events. >> >> The default configuration: >> >> image >> >> Here are few screenshots for reference: >> >> image >> >> image >> >> image >> >> image >> >> image >> >> Ignored >> >> image >> >> If we change default configuration as below: >> >> image >> >> image >> >> image >> >> image > > Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision: > > Implemented review comments Changes requested by hirt (Lead). core/org.openjdk.jmc.flightrecorder.rules.jdk/src/main/java/org/openjdk/jmc/flightrecorder/rules/jdk/memory/ZGCAllocationStallRule.java line 76: > 74: JdkAggregators.ZGC_ALLOCATION_STALL_COUNT, UnitLookup.NUMBER, IQuantity.class); > 75: > 76: public static final TypedResult ZGC_ALLOCATION_STALL_LONGEST_DURATION = new TypedResult<>( These are durations, so shouldn?t they use UnitLookup.MILLISECOND? core/org.openjdk.jmc.flightrecorder.rules.jdk/src/main/java/org/openjdk/jmc/flightrecorder/rules/jdk/messages/internal/Messages.java line 322: > 320: public static final String GcFreedRatioRule_RESULT_SHORT_DESCRIPTION = "GcFreedRatioRule_RESULT_SHORT_DESCRIPTION"; //$NON-NLS-1$ > 321: public static final String GarbageCollectionInfo_RULE_NAME = "GarbageCollectionInfo_RULE_NAME"; //$NON-NLS-1$ > 322: public static final String ZGCAllocationStall_RULE_NAME = "ZGCAllocationStall_RULE_NAME"; //$NON-NLS-1$ Consistent capitalization - you use Zgc elsewhere. core/org.openjdk.jmc.flightrecorder.rules.jdk/src/main/resources/org/openjdk/jmc/flightrecorder/rules/jdk/messages/internal/messages.properties line 763: > 761: ZGCAllocationStallRule_CONFIG_WARN_LIMIT_LONG=The ZGC allocation stall time per minute needed to trigger a warning > 762: ZGCAllocationStallRule_RATE=Allocation Stall time per minute > 763: ZGCAllocationStallRule_RATE_LONG=The rate of ZGC Allocation Stall duration based on the time from when JVM was started This is really not from when the JVM was started, but rather the average rate over the recording duration. Perhaps something like: ?Average ZGC allocation stalls per minute over the recording period? ------------- PR Review: https://git.openjdk.org/jmc/pull/664#pullrequestreview-3187220738 PR Review Comment: https://git.openjdk.org/jmc/pull/664#discussion_r2323542383 PR Review Comment: https://git.openjdk.org/jmc/pull/664#discussion_r2323533949 PR Review Comment: https://git.openjdk.org/jmc/pull/664#discussion_r2323528202 From hirt at openjdk.org Thu Sep 4 23:11:58 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Thu, 4 Sep 2025 23:11:58 GMT Subject: RFR: 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 15:16:46 GMT, Aymane Harmaz wrote: > Skipped the execution of JDP tests requiring multicast which is not supported on macOS-15 runner > > This was the result of moving macOS-latest from 14 to 15 > > Similar issues are mentioned in the following links : > https://github.com/orgs/community/discussions/170669 > https://github.com/actions/runner-images/issues/10924 Here's the ongoing discussion: https://developer.apple.com/forums/thread/770473 ------------- PR Comment: https://git.openjdk.org/jmc/pull/672#issuecomment-3256155650 From duke at openjdk.org Fri Sep 5 01:16:20 2025 From: duke at openjdk.org (Huang Xiao) Date: Fri, 5 Sep 2025 01:16:20 GMT Subject: RFR: 8406: Chinese Rule strings are out of date In-Reply-To: References: <6ScvFiQ7HRsIKkP2UvJZ4HwvzWDhQ3cWL8KCA_BS9sk=.ac2a4a50-ddc3-409d-b6ce-1f616e5665f1@github.com> <6dejvCr84JVe2s5bWYLKko0eBOuLI5k0lGN7dZTEfzU=.09f95282-f79d-427f-aa66-f2562e4a2ab7@github.com> Message-ID: On Tue, 26 Aug 2025 13:04:06 GMT, Alex Macdonald wrote: >> Hi @aptmac , just checking in to see if there's anything else needed for this PR. Thanks! > > @youngledo Great, looks like it's all working now. The bot added the instructions here: https://github.com/openjdk/jmc/pull/662#issuecomment-3047096405 > > You'll have to type `/integrate` and then the PR should be ready for sponsoring afterwards. @aptmac Hi, I discovered that there are other documents that urgently need translation, and I suggest that JMC introduce some AI tools to handle these tasks. ------------- PR Comment: https://git.openjdk.org/jmc/pull/662#issuecomment-3256704336 From duke at openjdk.org Fri Sep 5 08:22:21 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Fri, 5 Sep 2025 08:22:21 GMT Subject: RFR: 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 15:16:46 GMT, Aymane Harmaz wrote: > Skipped the execution of JDP tests requiring multicast which is not supported on macOS-15 runner > > This was the result of moving macOS-latest from 14 to 15 > > Similar issues are mentioned in the following links : > https://github.com/orgs/community/discussions/170669 > https://github.com/actions/runner-images/issues/10924 Thanks for the reviews ------------- PR Comment: https://git.openjdk.org/jmc/pull/672#issuecomment-3257488322 From duke at openjdk.org Fri Sep 5 08:22:22 2025 From: duke at openjdk.org (duke) Date: Fri, 5 Sep 2025 08:22:22 GMT Subject: RFR: 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 15:16:46 GMT, Aymane Harmaz wrote: > Skipped the execution of JDP tests requiring multicast which is not supported on macOS-15 runner > > This was the result of moving macOS-latest from 14 to 15 > > Similar issues are mentioned in the following links : > https://github.com/orgs/community/discussions/170669 > https://github.com/actions/runner-images/issues/10924 @aymane-harmaz Your change (at version 1cce2a742f10b59c10c7d9c54f34129b284c07e8) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jmc/pull/672#issuecomment-3257489544 From schaturvedi at openjdk.org Fri Sep 5 08:29:36 2025 From: schaturvedi at openjdk.org (Suchita Chaturvedi) Date: Fri, 5 Sep 2025 08:29:36 GMT Subject: RFR: 6203: Add ZGC allocation stall rule [v5] In-Reply-To: References: Message-ID: <9Wv2D5ls7WYKOkAYOtignug5EJYKtQcmv_4e3QBJ9Z0=.23c7861d-7c84-41e5-9f21-5d511908e855@github.com> > This enhancement is to add new rule for ZGC Allocation Stall events. > > The default configuration: > > image > > Here are few screenshots for reference: > > image > > image > > image > > image > > image > > Ignored > > image > > If we change default configuration as below: > > image > > image > > image > > image Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision: Implemented review comments ------------- Changes: - all: https://git.openjdk.org/jmc/pull/664/files - new: https://git.openjdk.org/jmc/pull/664/files/4b543d9d..1828c457 Webrevs: - full: https://webrevs.openjdk.org/?repo=jmc&pr=664&range=04 - incr: https://webrevs.openjdk.org/?repo=jmc&pr=664&range=03-04 Stats: 13 lines in 3 files changed: 0 ins; 0 del; 13 mod Patch: https://git.openjdk.org/jmc/pull/664.diff Fetch: git fetch https://git.openjdk.org/jmc.git pull/664/head:pull/664 PR: https://git.openjdk.org/jmc/pull/664 From duke at openjdk.org Fri Sep 5 08:31:22 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Fri, 5 Sep 2025 08:31:22 GMT Subject: Integrated: 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions In-Reply-To: References: Message-ID: <2RvEwxqvFgzoSTyG0WAlBL-otQnHEa0bC1EvSK703f4=.fa6d920c-f969-4295-8f9e-d1508685b99c@github.com> On Thu, 4 Sep 2025 15:16:46 GMT, Aymane Harmaz wrote: > Skipped the execution of JDP tests requiring multicast which is not supported on macOS-15 runner > > This was the result of moving macOS-latest from 14 to 15 > > Similar issues are mentioned in the following links : > https://github.com/orgs/community/discussions/170669 > https://github.com/actions/runner-images/issues/10924 This pull request has now been integrated. Changeset: 29a406f6 Author: Aymane Harmaz Committer: Suchita Chaturvedi URL: https://git.openjdk.org/jmc/commit/29a406f649fa30a9e81daadc26798fdb96dbaac1 Stats: 20 lines in 1 file changed: 20 ins; 0 del; 0 mod 8437: JDP Tests are failing on macOS-15 Github Actions runner due to local network privacy restrictions Reviewed-by: schaturvedi, hirt ------------- PR: https://git.openjdk.org/jmc/pull/672 From schaturvedi at openjdk.org Fri Sep 5 08:36:19 2025 From: schaturvedi at openjdk.org (Suchita Chaturvedi) Date: Fri, 5 Sep 2025 08:36:19 GMT Subject: RFR: 6203: Add ZGC allocation stall rule [v4] In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 21:02:14 GMT, Marcus Hirt wrote: >> Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision: >> >> Implemented review comments > > core/org.openjdk.jmc.flightrecorder.rules.jdk/src/main/java/org/openjdk/jmc/flightrecorder/rules/jdk/memory/ZGCAllocationStallRule.java line 76: > >> 74: JdkAggregators.ZGC_ALLOCATION_STALL_COUNT, UnitLookup.NUMBER, IQuantity.class); >> 75: >> 76: public static final TypedResult ZGC_ALLOCATION_STALL_LONGEST_DURATION = new TypedResult<>( > > These are durations, so shouldn?t they use UnitLookup.MILLISECOND? I have checked at other places. For Duration we have used UnitLookup.TIMESPAN. So I have used the same here. ------------- PR Review Comment: https://git.openjdk.org/jmc/pull/664#discussion_r2324463128 From hirt at openjdk.org Fri Sep 5 19:36:23 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Fri, 5 Sep 2025 19:36:23 GMT Subject: RFR: 6203: Add ZGC allocation stall rule [v5] In-Reply-To: <9Wv2D5ls7WYKOkAYOtignug5EJYKtQcmv_4e3QBJ9Z0=.23c7861d-7c84-41e5-9f21-5d511908e855@github.com> References: <9Wv2D5ls7WYKOkAYOtignug5EJYKtQcmv_4e3QBJ9Z0=.23c7861d-7c84-41e5-9f21-5d511908e855@github.com> Message-ID: On Fri, 5 Sep 2025 08:29:36 GMT, Suchita Chaturvedi wrote: >> This enhancement is to add new rule for ZGC Allocation Stall events. >> >> The default configuration: >> >> image >> >> Here are few screenshots for reference: >> >> image >> >> image >> >> image >> >> image >> >> image >> >> Ignored >> >> image >> >> If we change default configuration as below: >> >> image >> >> image >> >> image >> >> image > > Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision: > > Implemented review comments Marked as reviewed by hirt (Lead). ------------- PR Review: https://git.openjdk.org/jmc/pull/664#pullrequestreview-3190564429 From hirt at openjdk.org Fri Sep 5 19:58:18 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Fri, 5 Sep 2025 19:58:18 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 14:34:14 GMT, Aymane Harmaz wrote: > Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue > > > First Visit > > Second Visit Changes requested by hirt (Lead). core/org.openjdk.jmc.flightrecorder.rules/src/main/java/org/openjdk/jmc/flightrecorder/rules/report/html/internal/RulesHtmlToolkit.java line 255: > 253: description += (solution != null) ? "
" + solution + "
" : ""; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ > 254: Matcher matcher = LINK_PATTERN.matcher(description); > 255: description = description.replace("\n", "
") //$NON-NLS-1$//$NON-NLS-2$ Should this be <br/>? ------------- PR Review: https://git.openjdk.org/jmc/pull/669#pullrequestreview-3190613702 PR Review Comment: https://git.openjdk.org/jmc/pull/669#discussion_r2325926916 From schaturvedi at openjdk.org Sat Sep 6 18:16:16 2025 From: schaturvedi at openjdk.org (Suchita Chaturvedi) Date: Sat, 6 Sep 2025 18:16:16 GMT Subject: Integrated: 6203: Add ZGC allocation stall rule In-Reply-To: References: Message-ID: On Wed, 9 Jul 2025 14:17:10 GMT, Suchita Chaturvedi wrote: > This enhancement is to add new rule for ZGC Allocation Stall events. > > The default configuration: > > image > > Here are few screenshots for reference: > > image > > image > > image > > image > > image > > Ignored > > image > > If we change default configuration as below: > > image > > image > > image > > image This pull request has now been integrated. Changeset: 5c0e604a Author: Suchita Chaturvedi URL: https://git.openjdk.org/jmc/commit/5c0e604a4f11c2fb8523f20d4d83d3c2f60aa02b Stats: 326 lines in 11 files changed: 322 ins; 0 del; 4 mod 6203: Add ZGC allocation stall rule Reviewed-by: hirt ------------- PR: https://git.openjdk.org/jmc/pull/664 From duke at openjdk.org Mon Sep 8 08:51:51 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Mon, 8 Sep 2025 08:51:51 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit [v2] In-Reply-To: References: Message-ID: > Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue > > > First Visit > > Second Visit Aymane Harmaz has updated the pull request incrementally with one additional commit since the last revision: Fix line break tag syntax ------------- Changes: - all: https://git.openjdk.org/jmc/pull/669/files - new: https://git.openjdk.org/jmc/pull/669/files/a84f0cf0..f6ae0d48 Webrevs: - full: https://webrevs.openjdk.org/?repo=jmc&pr=669&range=01 - incr: https://webrevs.openjdk.org/?repo=jmc&pr=669&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jmc/pull/669.diff Fetch: git fetch https://git.openjdk.org/jmc.git pull/669/head:pull/669 PR: https://git.openjdk.org/jmc/pull/669 From duke at openjdk.org Mon Sep 8 12:22:17 2025 From: duke at openjdk.org (Martin Skarsaune) Date: Mon, 8 Sep 2025 12:22:17 GMT Subject: RFR: 8425: Problem importing in Eclipse due to missing awatility 3rd party artefact In-Reply-To: References: Message-ID: On Wed, 3 Sep 2025 19:14:31 GMT, Marcus Hirt wrote: > I think Virag just meant the target platform, where all third party dependencies are exported over a p2. Ok, I can try to include it ------------- PR Comment: https://git.openjdk.org/jmc/pull/668#issuecomment-3266031196 From duke at openjdk.org Mon Sep 8 15:14:56 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Mon, 8 Sep 2025 15:14:56 GMT Subject: RFR: 8438: Update the version of JMC to 10.0.0 Message-ID: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> Updated some versions in codebase that were referring to 9.1.0 toward 10.0.0 ------------- Commit messages: - 8438: Update the version of JMC to 10.0.0 Changes: https://git.openjdk.org/jmc/pull/673/files Webrev: https://webrevs.openjdk.org/?repo=jmc&pr=673&range=00 Issue: https://bugs.openjdk.org/browse/JMC-8438 Stats: 4 lines in 3 files changed: 0 ins; 0 del; 4 mod Patch: https://git.openjdk.org/jmc/pull/673.diff Fetch: git fetch https://git.openjdk.org/jmc.git pull/673/head:pull/673 PR: https://git.openjdk.org/jmc/pull/673 From hirt at openjdk.org Wed Sep 10 17:16:56 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Wed, 10 Sep 2025 17:16:56 GMT Subject: RFR: 8438: Update the version of JMC to 10.0.0 In-Reply-To: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> References: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> Message-ID: On Mon, 8 Sep 2025 15:08:42 GMT, Aymane Harmaz wrote: > Updated some versions in codebase that were referring to 9.1.0 toward 10.0.0 Marked as reviewed by hirt (Lead). ------------- PR Review: https://git.openjdk.org/jmc/pull/673#pullrequestreview-3207093322 From hirt at openjdk.org Wed Sep 10 17:19:18 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Wed, 10 Sep 2025 17:19:18 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit [v2] In-Reply-To: References: Message-ID: On Mon, 8 Sep 2025 08:51:51 GMT, Aymane Harmaz wrote: >> Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue >> >> >> First Visit >> >> Second Visit > > Aymane Harmaz has updated the pull request incrementally with one additional commit since the last revision: > > Fix line break tag syntax Marked as reviewed by hirt (Lead). ------------- PR Review: https://git.openjdk.org/jmc/pull/669#pullrequestreview-3207104762 From duke at openjdk.org Thu Sep 11 08:36:40 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Thu, 11 Sep 2025 08:36:40 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit [v2] In-Reply-To: References: Message-ID: On Mon, 8 Sep 2025 08:51:51 GMT, Aymane Harmaz wrote: >> Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue >> >> >> First Visit >> >> Second Visit > > Aymane Harmaz has updated the pull request incrementally with one additional commit since the last revision: > > Fix line break tag syntax Thanks for the review ------------- PR Comment: https://git.openjdk.org/jmc/pull/669#issuecomment-3279180742 From duke at openjdk.org Thu Sep 11 08:36:41 2025 From: duke at openjdk.org (duke) Date: Thu, 11 Sep 2025 08:36:41 GMT Subject: RFR: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit [v2] In-Reply-To: References: Message-ID: <6THFKp4BgZFW8XbFh1MwH_23H_FujzoyyY1wDMmWvWE=.fa21f5fe-a61b-4764-97e6-ca040c387445@github.com> On Mon, 8 Sep 2025 08:51:51 GMT, Aymane Harmaz wrote: >> Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue >> >> >> First Visit >> >> Second Visit > > Aymane Harmaz has updated the pull request incrementally with one additional commit since the last revision: > > Fix line break tag syntax @aymane-harmaz Your change (at version f6ae0d4835faba4002c29582d5fb117046dff8de) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jmc/pull/669#issuecomment-3279187437 From duke at openjdk.org Thu Sep 11 08:49:09 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Thu, 11 Sep 2025 08:49:09 GMT Subject: RFR: 8438: Update the version of JMC to 10.0.0 In-Reply-To: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> References: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> Message-ID: On Mon, 8 Sep 2025 15:08:42 GMT, Aymane Harmaz wrote: > Updated some versions in codebase that were referring to 9.1.0 toward 10.0.0 Thanks for the review ------------- PR Comment: https://git.openjdk.org/jmc/pull/673#issuecomment-3279242402 From duke at openjdk.org Thu Sep 11 08:52:24 2025 From: duke at openjdk.org (duke) Date: Thu, 11 Sep 2025 08:52:24 GMT Subject: RFR: 8438: Update the version of JMC to 10.0.0 In-Reply-To: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> References: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> Message-ID: <8xmc64C7SYIkkQ9hEBEOfgYEocS_MRloOh6zO94i1Ls=.c1d48837-3058-4173-886e-2a4e6d297e64@github.com> On Mon, 8 Sep 2025 15:08:42 GMT, Aymane Harmaz wrote: > Updated some versions in codebase that were referring to 9.1.0 toward 10.0.0 @aymane-harmaz Your change (at version f9f69f9c2ca532764b0c9205b9ff42c04fe987e3) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jmc/pull/673#issuecomment-3279254787 From duke at openjdk.org Mon Sep 15 09:38:00 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Mon, 15 Sep 2025 09:38:00 GMT Subject: Integrated: 8438: Update the version of JMC to 10.0.0 In-Reply-To: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> References: <6yCaQSeZmOuGUHGyxFbNXLlXoBP3nyp0-UGZSzdNpH8=.3a4e4789-9382-4e8e-96f6-966695b7b106@github.com> Message-ID: On Mon, 8 Sep 2025 15:08:42 GMT, Aymane Harmaz wrote: > Updated some versions in codebase that were referring to 9.1.0 toward 10.0.0 This pull request has now been integrated. Changeset: 84b51275 Author: Aymane Harmaz Committer: Suchita Chaturvedi URL: https://git.openjdk.org/jmc/commit/84b51275be1a20fd1fd192b7e57c58f79878c705 Stats: 4 lines in 3 files changed: 0 ins; 0 del; 4 mod 8438: Update the version of JMC to 10.0.0 Reviewed-by: hirt ------------- PR: https://git.openjdk.org/jmc/pull/673 From duke at openjdk.org Mon Sep 15 09:41:34 2025 From: duke at openjdk.org (Aymane Harmaz) Date: Mon, 15 Sep 2025 09:41:34 GMT Subject: Integrated: 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 14:34:14 GMT, Aymane Harmaz wrote: > Replaced manual bullet points and line breaks in rule descriptions with HTML entities and tags for improved rendering. the attached images show the issue > > > First Visit > > Second Visit This pull request has now been integrated. Changeset: 25b8e1d1 Author: Aymane Harmaz Committer: Suchita Chaturvedi URL: https://git.openjdk.org/jmc/commit/25b8e1d15d1f49f11ae021178e173b5346c4ed4b Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod 8436: Formatting of rule descriptions with manual bullets / line breaks is gone on revisit Reviewed-by: hirt ------------- PR: https://git.openjdk.org/jmc/pull/669