forked from dagger/dagger
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Downgrade opentelemetry to match Buildkit's version.
Before this, when trying to use opentelemetry libs it was possible to get an error "cannot merge resource due to conflicting Schema URL". The error indicates that different versions of opentelemetry are being used at the same time. This seems to have ultimately resulted from the fact that we setup tracers and also provide those to some Buildkit libs, but buildkit is still on v1.4.1 while we recently upgraded to v1.7.0. This, for reasons that are not yet clear to me, only happened in the integration tests where the dagger client is running inside dagger. It may have something to do with a difference in how we build dagger inside dagger compared to when it is built directly in GHA, but not sure. Either way, this downgrade unblocks us for the moment until we have a better way of solving this. Signed-off-by: Erik Sipsma <[email protected]>
- Loading branch information
Showing
2 changed files
with
9 additions
and
14 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters