Skip to content

Conversation

@MarcusRisanger
Copy link
Contributor

This PR is unrelated to a user story

Description

@MarcusRisanger
Copy link
Contributor Author

I also see that azure-monitor-opentelemetry is now in v1.7.0, so the version pinning in #144 could also perhaps be undone.

@bjorn-einar-bjartnes-4ss
Copy link
Contributor

I also see that azure-monitor-opentelemetry is now in v1.7.0, so the version pinning in #144 could also perhaps be undone.

We will take a look and test it with the latest version. There were some runtime errors thrown due, and they were fixed and reintroduced, so likely we will pin this package anyway, the dependency management with auto-instrumentation of different databases hasn't worked very well with python dependencies in that package.

@branislav-jenco-4ss
Copy link
Contributor

LGTM!

@branislav-jenco-4ss branislav-jenco-4ss merged commit b474e39 into 4Subsea:master Sep 2, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants