Skip to content

[DO NOT MERGE]feat: add instrumentation to forum#20

Open
jcapphelix wants to merge 3 commits into
release-ulmofrom
cjoshi/instrumentation
Open

[DO NOT MERGE]feat: add instrumentation to forum#20
jcapphelix wants to merge 3 commits into
release-ulmofrom
cjoshi/instrumentation

Conversation

@jcapphelix
Copy link
Copy Markdown
Collaborator

Description

This PR adds instrumentation around forum repo.

The need for this comes from us switching from v1 to v2 (Code wise, not backend wise), and the need for that comes from us bringing ulmo.1 changes.

Older implementation of forum cs_comments_service had it's own place in datadog, but that is no longer the case with edx/forum.

Thus this PR works in that direction, we won't have exactly same level of observability as we had cs_comments_service but this makes it more or less similar.

NOTE :- One need not merge this to check on staging, it can stay as is in a branch to check if it works or not in staging, the PR on how to add it in staging is available in "Related PR" section below.

DO NOT MERGE this in release-ulmo without testing and making required changes.

THIS IS POC only.

Useful links

Related PR

https://github.com/edx/edx-internal/pull/14147

@naincy128 naincy128 changed the title [DO NOT MERGE] feat: add instrumentation to forum feat: add instrumentation to forum Apr 13, 2026
@naincy128 naincy128 marked this pull request as ready for review April 13, 2026 07:28
@naincy128 naincy128 changed the title feat: add instrumentation to forum [DO NOT MERGE]feat: add instrumentation to forum Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants