Skip to content

Update Netty to 4.2.16.Final#22403

Open
reta wants to merge 2 commits into
opensearch-project:mainfrom
reta:update.netty.4.2.16
Open

Update Netty to 4.2.16.Final#22403
reta wants to merge 2 commits into
opensearch-project:mainfrom
reta:update.netty.4.2.16

Conversation

@reta

@reta reta commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

Update Netty to 4.2.16.Final

Related Issues

N/A

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Andriy Redko <drreta@gmail.com>
@reta reta added the dependencies Pull requests that update a dependency file label Jul 7, 2026
@reta reta requested review from a team and peternied as code owners July 7, 2026 13:50
@reta reta added the skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. label Jul 7, 2026
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit c37ff7d)

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

❌ Gradle check result for f6a9566: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@reta

reta commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

❌ Gradle check result for f6a9566: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

The org.opensearch.http.netty4.Netty4HttpRequestSizeLimitIT.testDoesNotLimitExcludedRequests test case failure is legit, sent a pull request to netty (see please netty/netty#17068). I do have mitigation to apply but would like to close the loop first.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit c37ff7d

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

❌ Gradle check result for c37ff7d: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

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

Labels

dependencies Pull requests that update a dependency file skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants