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

otelpyroscope link in README.md needs to be updated #3

Open
vasigorc opened this issue Dec 20, 2022 · 0 comments
Open

otelpyroscope link in README.md needs to be updated #3

vasigorc opened this issue Dec 20, 2022 · 0 comments

Comments

@vasigorc
Copy link

The otelpyroscope link from the README file currently yields a 404 page.

I wanted to create a PR to fix this:

diff --git a/README.md b/README.md
index 7708ab0..a7a96f0 100644
--- a/README.md
+++ b/README.md
@@ -8,7 +8,7 @@ link traces with the profiling data, and find specific lines of code related to
 * Because of how sampling profilers work, spans shorter than the sample interval may not be captured. By default pyroscope CPU profiler probes stack traces 100 times per second, meaning that spans shorter than 10ms may not be captured.
 
 
-Java code can be easily instrumented with [otelpyroscope](https://github.com/pyroscope-io/pyroscope-java/tree/main/otelpyroscope) package -
+Java code can be easily instrumented with [otelpyroscope](https://github.com/pyroscope-io/otel-profiling-java/tree/main/src/main/java/io/otel/pyroscope) package -
 a `OpenTelemetry` implementation, that annotates profiling data with span IDs which makes it possible to filter
 out profile of a particular trace span in Pyroscope:
 
diff --git a/gradle.properties b/gradle.properties
index c260f33..4d5c931 100644
--- a/gradle.properties
+++ b/gradle.properties
@@ -1,2 +1,2 @@
 pyroscope_version=0.10.1
-otel_profiling_version=0.10.1.3
+otel_profiling_version=0.10.1.4

but I don't have permissions to read from this repository:

ERROR: Permission to pyroscope-io/otel-profiling-java.git denied to vasigorc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant