Skip to content

Temporarily track xform ids when collecting dd metrics for reprocessing forms #36102

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gherceg
Copy link
Contributor

@gherceg gherceg commented Mar 29, 2025

Product Description

Technical Summary

This is to help with the investigation into celerybeat issues: https://dimagi.atlassian.net/browse/SAAS-16482

Feature Flag

Safety Assurance

Safety story

Straightforward change on. This will have a minimal effect on additional custom metrics as this queue isn't very busy, typically processing tens of forms per hour.

Automated test coverage

No automated test coverage that I know of.

QA Plan

N/A

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

@gherceg gherceg requested a review from snopoke as a code owner March 29, 2025 14:45
@dimagimon dimagimon added the Risk: High Change affects files that have been flagged as high risk. label Mar 29, 2025
@gherceg
Copy link
Contributor Author

gherceg commented Mar 31, 2025

I have one ID to look into, so will hold off on merging in case I don't actually need to

@gherceg gherceg requested a review from millerdev March 31, 2025 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Risk: High Change affects files that have been flagged as high risk.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants