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

Distribute cmd utilities as binary files #27856

Closed
theletterf opened this issue Oct 19, 2023 · 7 comments
Closed

Distribute cmd utilities as binary files #27856

theletterf opened this issue Oct 19, 2023 · 7 comments
Labels
cmd/configschema configschema command cmd/githubgen cmd/mdatagen mdatagen command cmd/opampsupervisor cmd/telemetrygen telemetrygen command enhancement New feature or request needs triage New item requiring triage Stale

Comments

@theletterf
Copy link
Member

Component(s)

cmd/configschema, cmd/githubgen, cmd/mdatagen, cmd/opampsupervisor, cmd/otelcontribcol, cmd/oteltestbedcol, cmd/telemetrygen

Is your feature request related to a problem? Please describe.

It'd be great if one could install the cmd utilities directly, using a package manager or a downloader, instead of having to clone the entire repo and compile each package. This is especially relevant for tools like telemetrygen, which are key to testing and learning.

Describe the solution you'd like

It'd be great to distribute the binary files of the cmd utilities.

Describe alternatives you've considered

No response

Additional context

No response

@theletterf theletterf added enhancement New feature or request needs triage New item requiring triage labels Oct 19, 2023
@github-actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@mx-psi
Copy link
Member

mx-psi commented Oct 19, 2023

This is especially relevant for tools like telemetrygen, which are key to testing and learning.

For telemetrygen we have a Docker image:
https://github.com/open-telemetry/opentelemetry-collector-contrib/pkgs/container/opentelemetry-collector-contrib%2Ftelemetrygen

But I agree it would be nice to also release binaries for it.

I am not sure we want to release binaries for the rest just yet; most are meant for internal usage and may heavily change from one version to the next, and the supervisor is under development yet.

Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@github-actions github-actions bot added the Stale label Dec 19, 2023
@theletterf
Copy link
Member Author

Any news on this?

@mx-psi
Copy link
Member

mx-psi commented Dec 19, 2023

I personally don't think we are at the point where we can guarantee any stability on anything other than telemetrygen. I would like to consider them case by case, and possibly move them to opentelemetry-go-build-tools instead of this repository if they are meant for public consumption

Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@atoulme
Copy link
Contributor

atoulme commented Mar 13, 2024

You can install telemetrygen with one line using:

go install github.com/open-telemetry/opentelemetry-collector-contrib/cmd/telemetrygen@latest

Unfortunately, I don't think we can otherwise distribute binaries at release time - it's easier to use go install on your local machine.

I'm going to close this as such, please reopen and vote up if you feel this can be done better.

@atoulme atoulme closed this as not planned Won't fix, can't repro, duplicate, stale Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmd/configschema configschema command cmd/githubgen cmd/mdatagen mdatagen command cmd/opampsupervisor cmd/telemetrygen telemetrygen command enhancement New feature or request needs triage New item requiring triage Stale
Projects
None yet
Development

No branches or pull requests

3 participants