-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
go install
fails for mdatagen
#9281
Comments
This fixes open-telemetry#9281 Signed-off-by: Alex Boten <aboten@lightstep.com>
Fixes #9281 --------- Signed-off-by: Alex Boten <aboten@lightstep.com>
…ry#9310) Fixes open-telemetry#9281 --------- Signed-off-by: Alex Boten <aboten@lightstep.com>
Not sure if this is a real problem, because this seem to work in CI, see https://github.com/open-telemetry/opentelemetry-collector/blob/main/.github/workflows/build-and-test.yml#L120 and https://github.com/open-telemetry/opentelemetry-collector/blob/main/Makefile#L81 |
It works in this repository, but not under a different folder. This means we can't use it for contrib. |
We are also experiencing this when trying to install the mdatagen tool via |
Is there a fix for this yet? Trying to update some generated files in vcenterreceiver. |
Weird, please make sure you are on latest main and run make generate. |
Issue persists, tried from inside a folder and from repository as well. Make generate fails. Error displayed after go install github.com/open-telemetry/opentelemetry-collector/cmd/mdatagen@latest: |
Please check that GOBIN is in your PATH. This might be the problem. |
Also seeing this issue:
|
I also can still reproduce. The way contrib works around this is by importing mdatagen into a go.mod first, and then install from there: |
Some traction on this would be appreciated. Trying to understand why the |
still experiencing this when doing |
Expected:
should install mdatagen under $GOBIN.
Actual:
The text was updated successfully, but these errors were encountered: