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

[CHK-1566] feat(tracing): use OpenTelemtry agent instead of Elastic APM #60

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

giovanniberti
Copy link
Contributor

@giovanniberti giovanniberti commented Jul 12, 2023

List of Changes

  • use OpenTelemtry agent instead of Elastic APM

Motivation and Context

This PR replaces Elastic APM monitoring agent with its OpenTelemetry counterpart.
This is done as Azure SDK tracing is not currently supported by Elastic APM.

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as
    expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@pietro-tota pietro-tota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done, just add a brief PR description

@giovanniberti giovanniberti merged commit aa2d764 into main Jul 13, 2023
@giovanniberti giovanniberti deleted the otel-integration branch July 13, 2023 13:19
@github-actions
Copy link

Pull request label validation error!
Required label for app version: patch,minor,major,ignore-for-release
Required label for chart version: chart-patch,chart-minor,chart-major.

Found labels on PR...

app version:
chart version:

If you want to skip release for this PR add skip-release label

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

Successfully merging this pull request may close these issues.

2 participants