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

Add OpenTelemetry logging and export to app insights #1616

Closed
2 tasks done
Tracked by #1456
arealmaas opened this issue Dec 17, 2024 · 0 comments
Closed
2 tasks done
Tracked by #1456

Add OpenTelemetry logging and export to app insights #1616

arealmaas opened this issue Dec 17, 2024 · 0 comments
Assignees

Comments

@arealmaas
Copy link
Collaborator

arealmaas commented Dec 17, 2024

Tasks

Preview Give feedback
@arealmaas arealmaas self-assigned this Dec 17, 2024
@arealmaas arealmaas moved this from Doing to Code Review og PR in Dialogporten / Arbeidsflate - NY Jan 10, 2025
arealmaas added a commit that referenced this issue Jan 13, 2025
<!--- Provide a general summary of your changes in the Title above -->

## Description

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

Notes:
* Added loki as a way to view logs in OTLP format locally
* Confirmed logs are exported
* Continue using Serilog, using Serilog open telemetry sink if open
telemetry configuration is found, both for bootstrap logger and final
logger. Will use Console exporter if configuration is not found
* Moved configuration of open telemetry from the aspnet package to the
respective services that use them directly.
* Removed fusioncache tracing for `Service`

## Related Issue(s)

- #1616

## Verification

- [ ] **Your** code builds clean without any errors or warnings
- [ ] 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)

---------

Co-authored-by: Magnus Sandgren <[email protected]>
Co-authored-by: Knut Haug <[email protected]>
@arealmaas arealmaas moved this from Code Review og PR to Done in Dialogporten / Arbeidsflate - NY Jan 15, 2025
arealmaas added a commit that referenced this issue Jan 15, 2025
<!--- Provide a general summary of your changes in the Title above -->

## Description

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

## Related Issue(s)

- #1616 

## Verification

- [ ] **Your** code builds clean without any errors or warnings
- [ ] 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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant