Skip to content

Improve influx logging

Matthias Larisch requested to merge improve-influx-logging into master

What does this MR do?

  • Fix logging email count: Now also count properly for emails with multiple recipients
  • Add SQL query metrics: query count, execution time and execution time per query
  • Add Eventlistener to generate these page statistics also for symfony controllers (e.g. REST API)
  • use hrtime instead of microtime for improved accuracy and performance

How confident are you it won't break things if deployed?

tested locally, doesn't really touch many things, so quite sure.

I didn't add any tests :-( No user facing changes (hopefully :-)

Links to related issues

Any relevant links (issues, documentation, slack discussions).

How to test

Steps a reviewer can take to verify that this MR does what it says it does e.g.

  1. Checkout branch locally
  2. Setup grafana (using credentials admin:admin) on http://localhost:4000
  3. Add an influxdb source via browser access to localhost:8089
  4. you can then add a dashboard looking into the influx data...

Screenshots (if applicable)

Bildschirmfoto_von_2020-05-03_18-17-21

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings

Merge request reports