Skip to content

Conversation

@LantaoJin
Copy link
Member

@LantaoJin LantaoJin commented Jul 21, 2025

Description

Fix 3 flaky tests in RestHandlerClientYamlTestSuiteIT

Related Issues

Resolves #3900

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • New functionality has javadoc added.
  • New functionality has a user manual doc added.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

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.

@LantaoJin LantaoJin added bug Something isn't working flaky-test Flaky build or test issue testing Related to improving software testing labels Jul 21, 2025
@LantaoJin LantaoJin marked this pull request as ready for review July 21, 2025 09:19
@penghuo penghuo merged commit 3c9766a into opensearch-project:main Jul 21, 2025
30 of 32 checks passed
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.19-dev 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/sql/backport-2.19-dev 2.19-dev
# Navigate to the new working tree
pushd ../.worktrees/sql/backport-2.19-dev
# Create a new branch
git switch --create backport/backport-3901-to-2.19-dev
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 3c9766a04101be048611fd401730bd1a5aa45fe1
# Push it to GitHub
git push --set-upstream origin backport/backport-3901-to-2.19-dev
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/sql/backport-2.19-dev

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

@LantaoJin
Copy link
Member Author

LantaoJin commented Jul 22, 2025

The auto-backport failure was caused by missing relevant backporting of #3882

LantaoJin added a commit to LantaoJin/search-plugins-sql that referenced this pull request Jul 31, 2025
@LantaoJin LantaoJin added the backport-manually Filed a PR to backport manually. label Jul 31, 2025
penghuo pushed a commit that referenced this pull request Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 2.19-dev backport-failed backport-manually Filed a PR to backport manually. bug Something isn't working flaky-test Flaky build or test issue testing Related to improving software testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Flaky RestHandlerClientYamlTestSuiteIT

3 participants