-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Docs: Local install edits #2220
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2220 +/- ##
=======================================
Coverage 62.10% 62.10%
=======================================
Files 154 154
Lines 12457 12457
=======================================
Hits 7737 7737
Misses 4111 4111
Partials 609 609 |
Co-authored-by: Cyril Tovena <[email protected]>
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.
Looking great, thanks for the help. I've added some comments/suggestions and tried to fill in a few blanks.
Co-authored-by: Owen Diehl <[email protected]>
Co-authored-by: Owen Diehl <[email protected]>
…/loki into docs-local-install-edits
Co-authored-by: Ed Welch <[email protected]>
…/loki into docs-local-install-edits
Co-authored-by: Ed Welch <[email protected]>
Co-authored-by: Ed Welch <[email protected]>
Co-authored-by: Ed Welch <[email protected]>
Applied edits and renamed the file as requested. |
docs/installation/local.md
Outdated
``` | ||
wget https://raw.githubusercontent.com/grafana/loki/master/cmd/loki/loki-local-config.yaml | ||
wget https://raw.githubusercontent.com/grafana/loki/master/cmd/promtail/promtail-local-config.yaml | ||
wget NEED DOWNLOAD COMMAND |
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.
I think this third wget NEED DOWNLOAD COMMAND
can be removed?
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.
Just one more comment but LGTM thanks @oddlittlebird !
Fixed, @slim-bean |
* gRPC Storage Service This PR contains gRPC service client i.e 1. Index Client 2. Storage Client 3. Table Client Signed-off-by: vineeth <[email protected]> * Fixed the spaces in imports. Signed-off-by: Vineeth <[email protected]> * Made changes around review comments Signed-off-by: Vineeth Pothulapati <[email protected]> * Address the rpc message change from TableDesc to CreateTableRequest & added the logic to terminate connection on Stop() invocation. Signed-off-by: Vineeth Pothulapati <[email protected]> * Added the chnages in CHANGELOG.md & add information around rpc calls in grpc.proto Signed-off-by: Vineeth Pothulapati <[email protected]> Co-authored-by: Marco Pracucci <[email protected]>
What this PR does / why we need it: Adds information from the meeting where I interviewed Owen and Ed. Gives users information for installing, running, and getting logs into Loki.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer: Please add information to download config files and run programs where indicated.
Checklist