Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project overview
  • Repository
  • Issues 261
  • Merge Requests 27
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !428

Open
Opened 2 weeks ago by Emiliano Balbuena@edgebal
Report abuse

PSR-3 Error logging (Monolog)

  • Overview 16
  • Commits 9
  • Pipelines 6
  • Changes 10
5/5 threads resolved

Summary

Standarizes the way error are being logged.

Closes #555.

Steps to test

  • Run kubectl get pods | grep goal-error-log-minds-app and take not of the Pod Name.
  • Run kubectl logs -f <Pod Name>
  • Check for uncaught errors (those that triggers a 500) output in the logs, they should be displayed correctly and with its full trace in a single line.

Note: Xander might not be able to test this, so maybe we should defer QA verification to Mark or Brian.

Estimated Regression Scope

Error logging might be skewed or Sentry reporting for uncaught error might not be triggering.

Edited 1 day ago by Emiliano Balbuena
Request to merge goal/error-log into master
The source branch is 14 commits behind the target branch
Open in Web IDE
Pipeline #104730190 passed for 95700722 on goal/error-log
          Requires approval from QA. Approved by
          Brian Hatchet
          Brian Hatchet
          Ben Hayward
          Ben Hayward
          Guy Thouret
          Guy Thouret
          Rami Albatal
          Rami Albatal