-
Notifications
You must be signed in to change notification settings - Fork 195
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
feat: support --redis_uri command line option #1488
feat: support --redis_uri command line option #1488
Conversation
Documenting this could take the form of another column under https://github.com/bazelbuild/bazel-buildfarm/blob/main/_site/docs/configuration/configuration.md that lists cmdline arg aliases for a config. |
40e88ed
to
0a50941
Compare
@jasonschroeder-sfdc did you want to circle back on the docs here, or are you punting for later changes? |
@werkt sorry - the ol' day job is killing me with deadlines. I'll squeeze these out today. |
Also, is URI or URL the correct term for this? 🤔 |
It is whatever the mechanism we use to interpret it says it is: in this case we're using java.net.URI. As long as nobody changes that, or looks too closely at whether it's the correct usage, itsa URI. |
0a50941
to
377a6c8
Compare
Support a `--redis_uri` command line option for start-up.
also fixed some spelling typos.
377a6c8
to
5f3550e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for the typo cleanups
Support a
--redis_uri
command line option for start-up.TODO: I wasn't sure where to document it under
_site
? Please let me know if it needs to be in the doc site. Thanks!Partially addresses #1445