Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable log correlation for edxapp #715

Open
2 tasks
robrap opened this issue Jul 9, 2024 · 3 comments
Open
2 tasks

Enable log correlation for edxapp #715

robrap opened this issue Jul 9, 2024 · 3 comments
Assignees

Comments

@robrap
Copy link
Contributor

robrap commented Jul 9, 2024

AC:

  • Enable log correlation for edxapp.
  • Update docs and communicate.

Notes:

@robrap robrap added this to Arch-BOM Jul 9, 2024
@robrap robrap converted this from a draft issue Jul 9, 2024
@robrap robrap moved this to Backlog in Arch-BOM Jul 9, 2024
@robrap
Copy link
Contributor Author

robrap commented Jul 9, 2024

[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 robrap moved this from Backlog to Ready For Development in Arch-BOM Aug 15, 2024
@robrap robrap self-assigned this Aug 20, 2024
@robrap
Copy link
Contributor Author

robrap commented Aug 20, 2024

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
Copy link
Contributor Author

robrap commented Aug 21, 2024

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

  • This private page includes some log examples, which I wanted to keep private.

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.

@robrap robrap moved this from In Progress to Backlog in Arch-BOM Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

No branches or pull requests

1 participant