-
Notifications
You must be signed in to change notification settings - Fork 12
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
[LocalNet] Add observability stack #475
Conversation
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 didn't see the data locally (yet) but did manage to access the dashboard so it's a great first step and good enough to merge in!
@@ -0,0 +1,2 @@ | |||
loki: | |||
isDefault: false |
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.
Shouldn't this be true?
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.
@Olshansk no, - Prometheus is configured to be the default data source. Can't have two default data sources.
Do you have this turned on or off in your config?
If it is enabled, you'd need to pull in the latest changes in the chart repo. You usually don't need to turn on that feature unless you actively develop a helm chart. If it is disabled, and you still experience that issue, please let me know - we can take a look. I just double-checked and it worked on my machine. |
Co-authored-by: Daniel Olshansky <[email protected]>
* pokt/main: [TODOs] Add todos for future smt optimizations (#423) [Metrics] feat: Initial on-chain metrics implementation (#449) [Tooling] Add application stake/unstake/delegate logs (#482) [SMT] Updated smt release from v0.9.2 -> v0.10.2 (#472) [Docs] update supplier actor CLI section (#471) [Testing] chore: add make targets & update feature path logging (#470) [Metrics] Additional off-chain metrics (#468) Update test instructions pull_request_template.md (#473) [Tooling] Makefile checkers when starting localnet (#466) [LocalNet] Switch validator logs to json (#480) [Documentation] Refactor poktroll documentation so its easier to navigate (#474) [LocalNet] Add observability stack (#475)
Co-authored-by: Daniel Olshansky <[email protected]>
Summary
Brings observability stack to LocalNet.
Issue
Type of change
Select one or more:
Testing
make go_develop_and_test
make test_e2e
devnet-test-e2e
label to the PR. THIS IS VERY EXPENSIVE, so only do it after all the reviews are complete.make docusaurus_start
Sanity Checklist