Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport] [2.x] Add a system property to configure YamlParser codepoint limits (#12301) #12305

Merged

Conversation

reta
Copy link
Collaborator

@reta reta commented Feb 13, 2024

Backport of #12301 to 2.x

Copy link
Contributor

github-actions bot commented Feb 13, 2024

Compatibility status:

Checks if related components are compatible with change f0de4f2

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/flow-framework.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/performance-analyzer.git]

Copy link
Contributor

❕ Gradle check result for d52d89c: UNSTABLE

  • TEST FAILURES:
      1 org.opensearch.index.shard.RemoteIndexShardTests.testNoFailuresOnFileReads

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Copy link

codecov bot commented Feb 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (74bd8f8) 71.21% compared to head (f0de4f2) 71.19%.
Report is 1 commits behind head on 2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #12305      +/-   ##
============================================
- Coverage     71.21%   71.19%   -0.02%     
- Complexity    60004    60020      +16     
============================================
  Files          4944     4944              
  Lines        282197   282200       +3     
  Branches      41335    41335              
============================================
- Hits         200969   200917      -52     
- Misses        64241    64309      +68     
+ Partials      16987    16974      -13     

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

…earch-project#12301)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
(cherry picked from commit bff8eb7)
@reta reta force-pushed the backport/backport-12301-to-2.x branch from d52d89c to f0de4f2 Compare February 14, 2024 00:23
Copy link
Contributor

❌ Gradle check result for f0de4f2: 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
Copy link
Collaborator Author

reta commented Feb 14, 2024

❌ Gradle check result for f0de4f2: FAILURE

java.lang.Exception: Test abandoned because suite timeout was reached.

Copy link
Contributor

❌ Gradle check result for f0de4f2: 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
Copy link
Collaborator Author

reta commented Feb 14, 2024

❌ Gradle check result for f0de4f2: FAILURE

#11113
#10755

Copy link
Contributor

✅ Gradle check result for f0de4f2: SUCCESS

@reta
Copy link
Collaborator Author

reta commented Feb 14, 2024

@kotwanikunal @andrross folks mind please approving this backport? thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants