edx / edx-arch-experiments

A plugin to include applications under development by the architecture team at edx
GNU Affero General Public License v3.0
0 stars 3 forks source link

Enable log correlation for edxapp #715

Open robrap opened 4 months ago

robrap commented 4 months ago

AC:

Notes:

robrap commented 4 months ago

[question] If we decide edxapp observability is the next in line for priority, we may want to move to the existing Datadog Migration epic, or create a new one.

robrap commented 3 months ago

The following screenshot is from the Logs tab of a Trace. It makes it seem like we just need to inject trace_id and span_id, event though the docs include much more.

Image

robrap commented 3 months ago

I wrote my findings of the challenges in https://2u-internal.atlassian.net/wiki/spaces/ENG/pages/1277394947/Challenges+with+edx-platform+log+formatting.

Here is a copy of the most relevant information from that page:

The log formatting in edx-platform is not easily configurable. You can find some of the configuration in code the edx-platform logsettings.py.

The following are past discussions about making it more easily configurable, but this work was not completed at the time:

@timmc-edx: I'm going to punt this to your containerization epic as a P5. Once that initial work is complete, which should affect some of the log file usage, we can determine whether to P4 or close this issue.