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.

Sign up for GitHub

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

Jump to bottom

Enable comprehensive telemetry #1101

Closed
3 tasks done
Tracked by #74
arealmaas opened this issue Sep 9, 2024 · 1 comment
Closed
3 tasks done
Tracked by #74

Enable comprehensive telemetry #1101

arealmaas opened this issue Sep 9, 2024 · 1 comment
Assignees
@arealmaas @knuhau
Labels
monitoring Issue related to logging and monitoring

Comments

@arealmaas
Copy link
Collaborator

arealmaas commented Sep 9, 2024 •
edited
Loading

We should convert to using OpenTelemetry in BE and FE as it has quickly become the new standard of instrumenting applications. It makes instrumentation platform agnostic and we can utilize the huge eco system to enrich our telemetry data.

TODO:

Tasks
Beta Give feedback
  2. Convert to opentelemetry in backend #1262
    0 of 1
    knuhau
  3. Fix issue where telemetry is disabled in FE #1265
    arealmaas
@arealmaas arealmaas self-assigned this Sep 9, 2024
@arealmaas arealmaas added the monitoring Issue related to logging and monitoring label Sep 9, 2024
@arealmaas arealmaas mentioned this issue Sep 9, 2024
Merged
3 tasks
@arealmaas arealmaas moved this from New issues to Doing in Dialogporten / Arbeidsflate Sep 10, 2024
@knuhau knuhau self-assigned this Sep 25, 2024
@knuhau knuhau mentioned this issue Sep 30, 2024
Merged
4 tasks
@arealmaas arealmaas mentioned this issue Oct 10, 2024
Open
@arealmaas arealmaas changed the title Convert to OpenTelemetry Enable comprehensive telemetry Oct 10, 2024
knuhau added a commit that referenced this issue Oct 21, 2024
@knuhau
chore(webapi): Add Opentelemetry tracing and metrics to webapi (#1202)
ec7e049 
<!--- Provide a general summary of your changes in the Title above -->

## Description

Replace Application Insights tracing with Opentelemetry in the webapi

<!--- Describe your changes in detail -->

## Related Issue(s)

- #1101 

## Verification

- [x] **Your** code builds clean without any errors or warnings
- [x] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)
@knuhau knuhau mentioned this issue Oct 25, 2024
Merged
4 tasks
oskogstad added a commit that referenced this issue Nov 5, 2024
@knuhau @oskogstad
chore(graphql): Removed Application Insights SDK, added OpenTelemetry (…
16f3b38 
…#1349)

<!--- Provide a general summary of your changes in the Title above -->

## Description
Removed Application Insights SDK, added OpenTelemetry

<!--- Describe your changes in detail -->

## Related Issue(s)

- #1101 

## Verification

- [x] **Your** code builds clean without any errors or warnings
- [x] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **New Features**
- Introduced `OpenTelemetryEventListener` for enhanced tracing and
monitoring of GraphQL requests.
- Updated telemetry configuration to streamline logging and ensure
consistent telemetry setup.

- **Bug Fixes**
- Removed the deprecated `ApplicationInsightEventListener`, improving
the overall telemetry approach.

- **Refactor**
- Adjusted service collection to replace the event listener for GraphQL
with the new OpenTelemetry implementation.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Co-authored-by: Ole Jørgen Skogstad <skogstad@softis.net>
@arealmaas
Copy link
Collaborator Author

Lukker denne til fordel for: #1456

enig @knuhau ?

@arealmaas arealmaas closed this as completed by moving to Done in Dialogporten / Arbeidsflate Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arealmaas arealmaas

@knuhau knuhau

Labels
monitoring Issue related to logging and monitoring
Projects
Dialogporten / Arbeidsflate
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arealmaas @knuhau