Closes #1193 - Update the documentation to include the changes to the demo #1198
Conversation
There was a problem hiding this comment.
Reviewed 1 of 2 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: 2 of 3 files reviewed, 3 unresolved discussions (waiting on @Mahir-Isikli and @Patrick-Eichhorn)
inspectit-ocelot-documentation/docs/getting-started/docker-examples.md, line 33 at r2 (raw file):
* File: `docker-compose-influxdb-jaeger.yml` * [OpenAPM Landscape](https://openapm.io/landscape?agent=inspectit-ocelot-agent&instrumentation-lib=opencensus&collector=jaeger-collector&storage=influx-db&visualization=jaeger-query&dashboarding=grafana&alerting=grafana&showCommercial=false&showFormats=false)
In this link, the Jaeger UI is connected to Grafana, but this connection does not exist.
inspectit-ocelot-documentation/docs/getting-started/docker-examples.md, line 134 at r2 (raw file):
flux
Flux is the query language for influxDB version > 2.0, we still use the 1.x version and therefore the query language InfluxQL. Please also verify this for the influx-zipkin setup
inspectit-ocelot-documentation/docs/getting-started/docker-examples.md, line 178 at r2 (raw file):
- *visits-service* - *vets-service* - *api-gateway*
Can you add a chapter Load Test after ## Changing Agent Configurations? (for each demo setup a load test runs, the user behavior is simulated / artillery is used and briefly describe what is called how and how often).
There was a problem hiding this comment.
Reviewable status: 2 of 3 files reviewed, 3 unresolved discussions (waiting on @Patrick-Eichhorn)
inspectit-ocelot-documentation/docs/getting-started/docker-examples.md, line 33 at r2 (raw file):
Previously, Patrick-Eichhorn wrote…
In this link, the Jaeger UI is connected to Grafana, but this connection does not exist.
Done.
inspectit-ocelot-documentation/docs/getting-started/docker-examples.md, line 134 at r2 (raw file):
Previously, Patrick-Eichhorn wrote…
fluxFlux is the query language for influxDB version > 2.0, we still use the 1.x version and therefore the query language InfluxQL. Please also verify this for the influx-zipkin setup
Done.
inspectit-ocelot-documentation/docs/getting-started/docker-examples.md, line 178 at r2 (raw file):
Previously, Patrick-Eichhorn wrote…
Can you add a chapter Load Test after ## Changing Agent Configurations? (for each demo setup a load test runs, the user behavior is simulated / artillery is used and briefly describe what is called how and how often).
Done.
There was a problem hiding this comment.
Reviewed 3 of 3 files at r4, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @Mahir-Isikli)
closes #1193
This change is