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

Remove all remote dependencies on the semconv repo (i.e. Git URL or Git Archive URL) for stability purposes. #267

Open
lquerel opened this issue Jul 29, 2024 · 0 comments
Open

Remove all remote dependencies on the semconv repo (i.e. Git URL or Git Archive URL) for stability purposes. #267

lquerel opened this issue Jul 29, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@lquerel
Copy link
Contributor

lquerel commented Jul 29, 2024

We should move away from relying on remote APIs like git url on the semconv repo and instead stand up a local server that hosts the file, for stability purposes.

See: crates/weaver_cache/src/lib.rs test test_semconv_registry_git_repo. Other instances of that exist in the repo.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
OTel Weaver Project
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Mini http server to serve static files in the context of unit tests lquerel/otel-weaver
1 participant
@lquerel