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

Binary is not PIE (Position-independent code) #693

Closed
DanTulovsky opened this issue Oct 12, 2024 · 3 comments
Closed

Binary is not PIE (Position-independent code) #693

DanTulovsky opened this issue Oct 12, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@DanTulovsky
Copy link

Component(s)

cmd/otelcontribcol

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

It looks like the binary for linux produced from this repo is not PIE (Position-independent code). I was wondering if this was a conscious decision. If not, would it be possible to start publishing PIE binaries for otel-collector-contrib?

My specific use case right now is that I am trying to run this on Unikraft Cloud (https://unikraft.org/docs/concepts/compatibility) and they require that binaries be PIE.

My workaround is to build from source, but I'd prefer to not have to do that.

Describe the solution you'd like

Linux binaries published by this project are PIE.

Describe alternatives you've considered

Building from source is a viable alternative.

Additional context

This is the error returned by the platform (confirmed by the people who run it).

Powered by Unikraft Telesto (0.16.2~83244528)
[    0.004302] ERR:  [appelfloader] otelcol-contrib: ELF executable is not position-independent!
[    0.005081] ERR:  [appelfloader] otelcol-contrib: Parsing of ELF image failed: Exec format error (-8)
@DanTulovsky
Copy link
Author

fyi, it looks like this is requested here: #618

@dosubot dosubot bot added the enhancement New feature or request label Oct 12, 2024
@mx-psi
Copy link
Member

mx-psi commented Oct 14, 2024

Yes, this looks like a duplicate of #618. @DanTulovsky, would you be willing to file a PR for this?

@mx-psi
Copy link
Member

mx-psi commented Oct 30, 2024

Closing as duplicate of #618

@mx-psi mx-psi closed this as not planned Won't fix, can't repro, duplicate, stale Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants