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

Issue using dynamic build and requirement on GLIBC #15

Closed
klauern opened this issue Mar 31, 2022 · 4 comments · Fixed by #24
Closed

Issue using dynamic build and requirement on GLIBC #15

klauern opened this issue Mar 31, 2022 · 4 comments · Fixed by #24
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@klauern
Copy link

klauern commented Mar 31, 2022

Hello!

I don't have a way to share this, but we are trying to use this action on our self-hosted runners, and ran into an issue with opa requiring GLIBC:

Run opa test configuration_rules/ -v
  opa test configuration_rules/ -v
  shell: /bin/bash -e {0}
opa: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.29' not found (required by opa)
opa: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by opa)
Error: Process completed with exit code 1.

I ended up switching this action out by curl'ing the static binary and adding it to the path, but it would be nice if this action did that as well. Here's my substitution for the setup-opa task:

        run: |
          curl -L -o opa https://openpolicyagent.org/downloads/v0.39.0/opa_linux_amd64_static
          chmod 755 opa
          mkdir -p ~/.local/bin
          mv opa ~/.local/bin/opa
          echo "$HOME/.local/bin" >> $GITHUB_PATH
@tim775
Copy link
Member

tim775 commented Mar 31, 2022

Thanks for the report! I think it would solve your problem if we added a new input calledstatic to select which binary you need, right?

Something like:

 ...
  static:
    description: Install the statically linked version of the OPA binary.  Usefull when GLIBC is not installed on self-hosted runners.
    required: false
    default: false

(PS let me know if you want to contribute the code too)

@tim775 tim775 added enhancement New feature or request good first issue Good for newcomers labels Mar 31, 2022
@humbertoc-silva
Copy link

@tim775,
I have the same problem and I am interested to contribute the code.

@tim775
Copy link
Member

tim775 commented Oct 12, 2022

@humbertoc-silva great do you need any help getting started?

@humbertoc-silva
Copy link

Hi @tim775,
Maybe I need some help, but first I will look at the project and try something based on your suggestion to include a new input parameter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants