-
Notifications
You must be signed in to change notification settings - Fork 1.6k
go install
fails for mdatagen
#9281
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
Comments
This fixes open-telemetry#9281 Signed-off-by: Alex Boten <[email protected]>
Fixes #9281 --------- Signed-off-by: Alex Boten <[email protected]>
…ry#9310) Fixes open-telemetry#9281 --------- Signed-off-by: Alex Boten <[email protected]>
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 |
Why was this closed? This is still an issue. Currently have to manually build mdatagen to get this to work. go install go.opentelemetry.io/collector/cmd/mdatagen@latest go: downloading go.opentelemetry.io/collector/cmd/mdatagen v0.106.1 |
@fredyfredburger It's not closed. |
Derp; I saw "open-telemetry/opentelemetry-collector-contrib#33603 Closed" and thought this ticket was closed. That's my bad! |
Since I did the dumb and commented something useless on this ticket, I'll try to redeem myself by adding a temporary solution for those finding this ticket like I did looking for an answer:
|
$ go install go.opentelemetry.io/collector/cmd/mdatagen@latest Still issue persists. |
Expected:
should install mdatagen under $GOBIN.
Actual:
The text was updated successfully, but these errors were encountered: