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

OPA executable path #680

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

OPA executable path #680

wants to merge 3 commits into from

Conversation

daveads
Copy link
Contributor

@daveads daveads commented Oct 11, 2024

/claim #676

feat(opal-client): Add configurable OPA executable path

  • Introduce INLINE_OPA_EXECUTABLE_PATH configuration option
  • Update OpaRunner to use configurable path for OPA executable
  • Default to 'opa' if no custom path is specified
  • Update setup_opa_runner to handle the new configuration

Copy link

netlify bot commented Oct 11, 2024

Deploy Preview for opal-docs ready!

Name Link
🔨 Latest commit 08199fa
🔍 Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/670e769b13c4cd00080efbd3
😎 Deploy Preview https://deploy-preview-680--opal-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@daveads
Copy link
Contributor Author

daveads commented Oct 11, 2024

@danyi1212 review when free....

@gemanor
Copy link
Collaborator

gemanor commented Oct 13, 2024

Hey @daveads can you please check why the test isn't pass before asking @danyi1212 for a review?

@daveads
Copy link
Contributor Author

daveads commented Oct 13, 2024

Hey @daveads can you please check why the test isn't pass before asking @danyi1212 for a review?

sure

@daveads
Copy link
Contributor Author

daveads commented Oct 14, 2024

 Cloning into 'opal-tests-policy-repo'...
Load key "/home/runner/work/opal/opal/app-tests/opal-tests-policy-repo-key": error in libcrypto
git@github.com: Permission denied (publickey).
fatal: Could not read from remote repository.

This suggests that the SSH key is not being recognized or doesn't have the correct permissions.

@daveads
Copy link
Contributor Author

daveads commented Oct 14, 2024

pre-commit / pre-commit (pull_request) FIX

#683

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants