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

Has the code been tested with grpcs? #385

Closed
symbiont-ji opened this issue Jan 1, 2021 · 2 comments
Closed

Has the code been tested with grpcs? #385

symbiont-ji opened this issue Jan 1, 2021 · 2 comments

Comments

@symbiont-ji
Copy link

Is there a worked out example for how to set up certs an keys on both the bazel-remote side and the bazel side? Doing the obvious (creating a self-signed CA, using it to sign a client and a server key, passing those as options in the config file of bazel-remote, and as options in bazel, results in the rather disappointing message ERROR: Failed to query remote execution capabilities: General OpenSslEngine problem.

It does not help that bazel's documentation is lacking; here is the corresponding issue I opened with them: bazelbuild/bazel#12764

@mostynb
Copy link
Collaborator

mostynb commented Jan 1, 2021

Hi.

I remember testing grpcs support when mTLS authentication was added- I think I either copied the test certs/keys from https://github.com/bazelbuild/bazel/tree/master/src/test/testdata/test_tls_certificate or generated my own by following that readme file. This comment in the mTLS PR mentions the relevant bazel and bazel-remote flags that I used: #360 (comment)

@mostynb
Copy link
Collaborator

mostynb commented Mar 19, 2021

I added a CI test for TLS and mTLS in #407. If anyone else is having trouble getting this working, here's an example:
6aa66a2

@mostynb mostynb closed this as completed Mar 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants
@mostynb @symbiont-ji and others