Skip to content

8280158: New test from JDK-8274736 failed with/without patch in JDK11u#802

Open
vieiro wants to merge 1 commit into
openjdk:pr/799from
vieiro:backports/JDK-8280158
Open

8280158: New test from JDK-8274736 failed with/without patch in JDK11u#802
vieiro wants to merge 1 commit into
openjdk:pr/799from
vieiro:backports/JDK-8280158

Conversation

@vieiro
Copy link
Copy Markdown
Contributor

@vieiro vieiro commented May 20, 2026

Backport of JDK-8280158 from 11 as a follow up of JDK-8274736 (#799) .

Backport is not clean since JDK8 has different directory structure. Also changes to ProblemList.txt were dropped from the original commit since JDK8 is missing a backport for JDK-8277970.

Build on RHEL-8/gcc 8.5.0, and tested with:

  • /jdk/test/sun/security/ssl/ (same results as in previous PR)
--------------------------------------------------
Test results: passed: 110; failed: 1; error: 4
  • jdk/test/sun/security/ssl/SSLSessionImpl/NoInvalidateSocketException.java

Passed correctly, couldn't reproduce the race problem reported in JDK-8280158



Progress

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

Integration blocker

 ⚠️ Dependency #799 must be integrated first

Issue

  • JDK-8280158: New test from JDK-8274736 failed with/without patch in JDK11u (Bug - P3)

Reviewers

Reviewing

Using git

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

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

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 802

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

Using diff file

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

Using Webrev

Link to Webrev Comment

@bridgekeeper
Copy link
Copy Markdown

bridgekeeper Bot commented May 20, 2026

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

@openjdk
Copy link
Copy Markdown

openjdk Bot commented May 20, 2026

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

@openjdk openjdk Bot changed the title Backport 18631ef9928fe781ad5a6459642062d5f40d04cb 8280158: New test from JDK-8274736 failed with/without patch in JDK11u May 20, 2026
@openjdk
Copy link
Copy Markdown

openjdk Bot commented May 20, 2026

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

@openjdk openjdk Bot added backport Port of a pull request already in a different code base rfr Pull request is ready for review labels May 20, 2026
@mlbridge
Copy link
Copy Markdown

mlbridge Bot commented May 20, 2026

Webrevs

Copy link
Copy Markdown
Contributor

@jerboaa jerboaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me.

@jerboaa
Copy link
Copy Markdown
Contributor

jerboaa commented May 21, 2026

Looking at the bug we might want to consider https://bugs.openjdk.org/browse/JDK-8277970 as well, since that was part of 11u all along and is also related to potential issues with NoInvalidateSocketException.java. Thoughts?

@vieiro
Copy link
Copy Markdown
Contributor Author

vieiro commented May 21, 2026

Looking at the bug we might want to consider https://bugs.openjdk.org/browse/JDK-8277970 as well, since that was part of 11u all along and is also related to potential issues with NoInvalidateSocketException.java. Thoughts?

It may be a good idea, yes. I'll investigate if this requires a PR on top of this one or below this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Port of a pull request already in a different code base rfr Pull request is ready for review

Development

Successfully merging this pull request may close these issues.

2 participants