Skip to content

[Backport 3.6] Fix flaky org.opensearch.http.netty4.Netty4HttpRequestSizeLimitIT.testDoesNotLimitExcludedRequests test case#21571

Merged
reta merged 1 commit into
3.6from
backport/backport-21565-to-3.6
May 9, 2026
Merged

[Backport 3.6] Fix flaky org.opensearch.http.netty4.Netty4HttpRequestSizeLimitIT.testDoesNotLimitExcludedRequests test case#21571
reta merged 1 commit into
3.6from
backport/backport-21565-to-3.6

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 0d31e28 from #21565.

…tDoesNotLimitExcludedRequests test case (#21565)

Signed-off-by: Andriy Redko <drreta@gmail.com>
(cherry picked from commit 0d31e28)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot Bot requested review from a team and peternied as code owners May 8, 2026 21:09
@reta reta added flaky-test Random test failure that succeeds on second run skip-changelog skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. labels May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

❌ Gradle check result for 0996045: 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?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

✅ Gradle check result for 0996045: SUCCESS

@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.11%. Comparing base (e2270fc) to head (0996045).
⚠️ Report is 12 commits behind head on 3.6.

Additional details and impacted files
@@             Coverage Diff              @@
##                3.6   #21571      +/-   ##
============================================
- Coverage     73.15%   73.11%   -0.05%     
+ Complexity    72554    72526      -28     
============================================
  Files          5848     5837      -11     
  Lines        331985   331901      -84     
  Branches      47949    47934      -15     
============================================
- Hits         242873   242678     -195     
- Misses        69595    69712     +117     
+ Partials      19517    19511       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@reta reta merged commit 278de8f into 3.6 May 9, 2026
30 of 33 checks passed
@github-actions github-actions Bot deleted the backport/backport-21565-to-3.6 branch May 9, 2026 02:03
@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor Author

The backport to 2.19 failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/OpenSearch/backport-2.19 2.19
# Navigate to the new working tree
pushd ../.worktrees/OpenSearch/backport-2.19
# Create a new branch
git switch --create backport/backport-21571-to-2.19
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 278de8f75ae2efa917dcfcbd8354b2a01bebf352
# Push it to GitHub
git push --set-upstream origin backport/backport-21571-to-2.19
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/OpenSearch/backport-2.19

Then, create a pull request where the base branch is 2.19 and the compare/head branch is backport/backport-21571-to-2.19.

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

Labels

backport 2.19 backport-failed flaky-test Random test failure that succeeds on second run skip-changelog 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.

1 participant