8247362: HeapDumpCompressedTest.java#id0 fails due to "Multiple garbage collectors selected"

[satyenme]

Backporting JDK-8247362: HeapDumpCompressedTest.java#id0 fails due to "Multiple garbage collectors selected". Adjusted test to add @ requires for GCs, fixing failure when run with GC directly. Ran GHA Sanity Checks, local Tier 1 and 2, and adjusted test directly. Patch is clean.

---

Progress

• JDK-8247362 needs maintainer approval
• Change must not contain extraneous whitespace
• Commit message must refer to an issue

Issue

• JDK-8247362: HeapDumpCompressedTest.java#id0 fails due to "Multiple garbage collectors selected" (Bug - P3 - Requested)

Reviewers

• Ralf Schmelter (@schmelter-sap - Reviewer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk11u-dev.git pull/3061/head:pull/3061
$ git checkout pull/3061

Update a local copy of the PR:
$ git checkout pull/3061
$ git pull https://git.openjdk.org/jdk11u-dev.git pull/3061/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 3061

View PR using the GUI difftool:
$ git pr show -t 3061

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk11u-dev/pull/3061.diff

Using Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back ssubramaniam! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

[openjdk]

This backport pull request has now been updated with issue from the original commit.

[openjdk]

⚠️ @satyenme This change is now ready for you to apply for maintainer approval. This can be done directly in each associated issue or by using the /approval command.

[mlbridge]

Webrevs

• 00: Full (55421786)

[satyenme]

/approval request for backport of JDK-8247362: HeapDumpCompressedTest.java#id0 fails due to "Multiple garbage collectors selected"

Motivation: Without this change the test/hotspot/jtreg/serviceability/dcmd/gc/HeapDumpCompressedTest.java test will fail when run with jtreg and a GC explicitly set. With this backport, the test coverage will be preserved, and will no longer fail in such scenarios.

Risk: Low. Ran GHA Sanity Checks, local Tier 1 and 2, and adjusted test directly. Patch is clean. Change has been present in tip since June, 2020. Only affects test code for a single test.

[openjdk]

@satyenme
8247362: The approval request has been created successfully.

[schmelter-sap]

Looks good.