8276139: TestJpsHostName.java not reliable, better to expand HostIdentifierCreate.java test

[yaqsun]

It is not reliable for a to modify this problem, so JDK-8276139 is submitted on the basis of JDK-8251155. JDK-8251155 and JDK-8276139 have been in 11u and later. so I backport JDK-8251155 and JDK-8276139 into one PR here.

Apply clean except for test path. And test have been locally verified to pass.

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• JDK-8251155 needs maintainer approval
• Change must not contain extraneous whitespace
• Commit message must refer to an issue
• JDK-8276139 needs maintainer approval

Issues

• JDK-8276139: TestJpsHostName.java not reliable, better to expand HostIdentifierCreate.java test (Bug - P4 - Approved)
• JDK-8251155: HostIdentifier fails to canonicalize hostnames starting with digits (Bug - P4 - Approved)

Reviewers

• Paul Hohensee (@phohensee - Reviewer)

Reviewing

Using git

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

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

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 411

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

Using diff file

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

Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back yaqsun! 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 backport pull request has now been updated with issue from the original commit.

[mlbridge]

Webrevs

• 01: Full - Incremental (d2090799)
• 00: Full (1f481e24)

[phohensee]

/issue add 8251155

[openjdk]

@phohensee Only the author (@yaqsun) is allowed to issue the /issue command.

[yaqsun]

/issue add 8251155

[openjdk]

@yaqsun
Adding additional issue to issue list: 8251155: HostIdentifier fails to canonicalize hostnames starting with digits.

[phohensee]

Lgtm. Please check GHA failure, looks unrelated to me, but you may differ.

[openjdk]

⚠️ @yaqsun 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.

[yaqsun]

/approval 8251155 request

[yaqsun]

/approval 8276139 request

[openjdk]

@yaqsun
8251155: The approval request has been created successfully.

[openjdk]

@yaqsun
8276139: The approval request has been created successfully.

[bridgekeeper]

@yaqsun This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply add a new comment to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

[jerboaa]

/approve yes

[openjdk]

@jerboaa
8276139: The approval request has been approved.
8251155: The approval request has been approved.

[openjdk]

@yaqsun This change now passes all automated pre-integration checks.

After integration, the commit message for the final commit will be:

8276139: TestJpsHostName.java not reliable, better to expand HostIdentifierCreate.java test
8251155: HostIdentifier fails to canonicalize hostnames starting with digits

Reviewed-by: phh

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been 20 new commits pushed to the master branch:

• 9aabf22: 8270517: Add Zero support for LoongArch
• dce5300: 8168518: rcache interop with krb5-1.15
• 3dc011b: 8323202: [8u] Remove get_source.sh and hgforest.sh
• abb4975: 8011180: Delete obsolete scripts
• cfed45d: 8155590: Dubious collection management in sun.net.www.http.KeepAliveCache
• 818c53e: 8315042: NPE in PKCS7.parseOldSignedData
• f4f45e3: 8321060: [8u] hotspot needs to recognise VS2022
• 15e2c0b: 8322725: (tz) Update Timezone Data to 2023d
• 474c8c1: 8192931: Regression test java/awt/font/TextLayout/CombiningPerf.java fails
• 4a69ad2: 8324184: Windows VS2010 build failed with "error C2275: 'int64_t'"
• ... and 10 more: https://git.openjdk.org/jdk8u-dev/compare/d898d4018e0a16f2115f4b1cbf93eb5e653b789b...master

As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

As you do not have Committer status in this project an existing Committer must agree to sponsor your change. Possible candidates are the reviewers of this PR (@phohensee) but any other Committer may sponsor as well.

➡️ To flag this PR as ready for integration with the above commit message, type /integrate in a new comment. (Afterwards, your sponsor types /sponsor in a new comment to perform the integration).

[yaqsun]

/integrate

[openjdk]

@yaqsun
Your change (at version d209079) is now ready to be sponsored by a Committer.

[phohensee]

/sponsor

[openjdk]

Going to push as commit e926edc.
Since your change was applied there have been 20 commits pushed to the master branch:

• 9aabf22: 8270517: Add Zero support for LoongArch
• dce5300: 8168518: rcache interop with krb5-1.15
• 3dc011b: 8323202: [8u] Remove get_source.sh and hgforest.sh
• abb4975: 8011180: Delete obsolete scripts
• cfed45d: 8155590: Dubious collection management in sun.net.www.http.KeepAliveCache
• 818c53e: 8315042: NPE in PKCS7.parseOldSignedData
• f4f45e3: 8321060: [8u] hotspot needs to recognise VS2022
• 15e2c0b: 8322725: (tz) Update Timezone Data to 2023d
• 474c8c1: 8192931: Regression test java/awt/font/TextLayout/CombiningPerf.java fails
• 4a69ad2: 8324184: Windows VS2010 build failed with "error C2275: 'int64_t'"
• ... and 10 more: https://git.openjdk.org/jdk8u-dev/compare/d898d4018e0a16f2115f4b1cbf93eb5e653b789b...master

Your commit was automatically rebased without conflicts.

[openjdk]

@phohensee @yaqsun Pushed as commit e926edc.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.

[gnu-andrew]

Why have two backports been bundled into one PR here? That doesn't seem to be the case in 11u and later.

[gnu-andrew]

/approval 8251155 request

A reason should be given as to why the request should be approved. See https://wiki.openjdk.org/display/SKARA/Pull+Request+Commands#PullRequestCommands-/approval

[yaqsun]

Why have two backports been bundled into one PR here? That doesn't seem to be the case in 11u and later.

It is not reliable for a to modify this problem, so JDK-8276139 is submitted on the basis of JDK-8251155. JDK-8251155 and JDK-8276139 have been in 11u and later.