-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Backport] [2.x] Add a system property to configure YamlParser codepoint limits (#12301) #12305
Conversation
Compatibility status:Checks if related components are compatible with change f0de4f2 Incompatible componentsSkipped componentsCompatible componentsCompatible 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] |
❕ Gradle check result for d52d89c: UNSTABLE
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
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. |
…earch-project#12301) Signed-off-by: Andriy Redko <[email protected]> (cherry picked from commit bff8eb7)
d52d89c
to
f0de4f2
Compare
❌ 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? |
❌ 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? |
@kotwanikunal @andrross folks mind please approving this backport? thank you! |
Backport of #12301 to
2.x