opal icon indicating copy to clipboard operation
opal copied to clipboard

OPA executable path

Open daveads opened this issue 1 year ago • 6 comments

/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

daveads avatar Oct 11 '24 09:10 daveads

Deploy Preview for opal-docs canceled.

Name Link
Latest commit 06d74ab6badb75052fe040c9ba469311534f92cd
Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/673a22df2a0691000892b7b5

netlify[bot] avatar Oct 11 '24 09:10 netlify[bot]

@danyi1212 review when free....

daveads avatar Oct 11 '24 09:10 daveads

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

gemanor avatar Oct 13 '24 20:10 gemanor

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

sure

daveads avatar Oct 13 '24 21:10 daveads

 Cloning into 'opal-tests-policy-repo'...
Load key "/home/runner/work/opal/opal/app-tests/opal-tests-policy-repo-key": error in libcrypto
[email protected]: 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 avatar Oct 14 '24 07:10 daveads

pre-commit / pre-commit (pull_request) FIX

https://github.com/permitio/opal/pull/683

daveads avatar Oct 14 '24 22:10 daveads

@danyi1212 @gemanor

Good to go ?

daveads avatar Nov 17 '24 17:11 daveads