- Notifications
You must be signed in to change notification settings - Fork19
Grafana Dashboard#116
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
thinkmorestupidless left a comment
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.
When i rundocker compose up it brings everything up but in Grafana I don't see any data source or dashboards, they should be automatically configured in Grafana but for some reason they're missing.
I removed all containers before running so it's not because of any pre-existing Grafana container.
| ``` | ||
| - Run theSodukosolver by executing the`runSolver` command from the`sbt` prompt. | ||
| - Run theSudokusolver by executing the`runSolver` command from the`sbt` prompt. |
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.
👍
| valsystem=ActorSystem[NotUsed](Main(),"sudoku-solver-system") | ||
| println(s"${GREEN}Hit RETURN to stop solver${RESET}") |
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.
run / fork := true is required for Kamon instrumentation to work (if the application doesn't run in a forked JVM then the instrumentation agent can't be attached so no metrics are sent anywhere)
BUT, this stops the application blocking onStdIn.readline() - hence the need to remove the two lines below (otherwise it just falls through tosystem.terminate()) - not sure why, seems to be a known thing.
We should remove them completely rather than just commenting out, and probably remove theprintln as well as it's not necessary either.
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.
We should remove them completely rather than just commenting out, and probably remove the println as well as it's not necessary either.
Yes, please remove it as needed
2af1ff0 to2cfc044Compare85309ba to22a913eCompare- Removed some graphs that are not relevant to the demo- Changed dashboard layout
No description provided.