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] Adds support for tracing runnable scenarios #9187

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

Gaganjuneja
Copy link
Contributor

2.x backport of PR - #8831.

Description

[Describe what this change achieves]

Related Issues

Resolves #8428

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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: Gagan Juneja <gjjuneja@amazon.com>
@opensearch-trigger-bot
Copy link
Contributor

Compatibility status:

> Task :checkCompatibility
Checking compatibility for: https://github.com/opensearch-project/opensearch-oci-object-storage.git with ref: 2.x
Skipping compatibility check for https://github.com/opensearch-project/opensearch-oci-object-storage.git
Incompatible components: [https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/performance-analyzer.git]
Components skipped due to git failures: [https://github.com/opensearch-project/opensearch-oci-object-storage.git]
Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/ml-commons.git]

BUILD SUCCESSFUL in 22m 31s

@github-actions
Copy link
Contributor

github-actions bot commented Aug 9, 2023

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Merging #9187 (adabaee) into 2.x (cc9d907) will increase coverage by 0.03%.
Report is 3 commits behind head on 2.x.
The diff coverage is 71.66%.

@@             Coverage Diff              @@
##                2.x    #9187      +/-   ##
============================================
+ Coverage     70.83%   70.87%   +0.03%     
- Complexity    57454    57561     +107     
============================================
  Files          4754     4756       +2     
  Lines        271758   271788      +30     
  Branches      40082    40086       +4     
============================================
+ Hits         192504   192618     +114     
- Misses        62713    62718       +5     
+ Partials      16541    16452      -89     
Files Changed Coverage Δ
.../opensearch/telemetry/tracing/noop/NoopTracer.java 80.00% <50.00%> (-20.00%) ⬇️
...rg/opensearch/telemetry/tracing/WrappedTracer.java 83.33% <50.00%> (-16.67%) ⬇️
...ter/snapshots/get/TransportGetSnapshotsAction.java 56.07% <61.76%> (+14.49%) ⬆️
...rg/opensearch/telemetry/tracing/DefaultTracer.java 92.30% <85.71%> (-2.70%) ⬇️
.../org/opensearch/telemetry/tracing/SpanContext.java 100.00% <100.00%> (ø)
.../telemetry/tracing/runnable/TraceableRunnable.java 100.00% <100.00%> (ø)

... and 472 files with indirect coverage changes

@reta reta changed the title Adds support for tracing runnable scenarios [Backport] [2.x] Adds support for tracing runnable scenarios Aug 9, 2023
@reta reta merged commit c3a0784 into opensearch-project:2.x Aug 9, 2023
35 checks passed
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.

2 participants