You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ticket #2705 introduced a change that made made our logs more easily digestible by kibana (json format). However, it was meant to just log in a json format on "Error" but is currently logging for all log lines. The json format is harder for human eyes to digest, especially when running scripts or other commands, and while helpful to keep tracebacks (errors) together, it gets less helpful for things that are warning, info and debug. Ideally, we should display logs as we do on localhost.
Acceptance criteria
Logs match the format present on localhost for DEBUG, INFO, and WARNING
Log level ERROR is still in JSON format
Additional context
Note: logging for localhost should not be modified as part of this ticket.
unit testing isn't required for this one. Just manually test locally and on sandbox. For the PR review, it would be very nice to add some temporary logger lines set to error to an easy to get to page (like the home page), as well as some lines that are at the warning level. It would speed up the review. Just if you do this, be sure to delete those logs prior to merging.
Issue description
Ticket #2705 introduced a change that made made our logs more easily digestible by kibana (json format). However, it was meant to just log in a json format on "Error" but is currently logging for all log lines. The json format is harder for human eyes to digest, especially when running scripts or other commands, and while helpful to keep tracebacks (errors) together, it gets less helpful for things that are warning, info and debug. Ideally, we should display logs as we do on localhost.
Acceptance criteria
Additional context
Note: logging for localhost should not be modified as part of this ticket.
unit testing isn't required for this one. Just manually test locally and on sandbox. For the PR review, it would be very nice to add some temporary logger lines set to error to an easy to get to page (like the home page), as well as some lines that are at the warning level. It would speed up the review. Just if you do this, be sure to delete those logs prior to merging.
Links to other issues
The text was updated successfully, but these errors were encountered: