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

Report mobile report row count during restore to datadog #34795

Merged
merged 2 commits into from
Jun 20, 2024

Conversation

gherceg
Copy link
Contributor

@gherceg gherceg commented Jun 19, 2024

Product Description

Technical Summary

https://dimagi.atlassian.net/browse/SAAS-15614

We currently don't have insight into the size of mobile reports during restores. This will enable us to monitor the sizes of reports generated during restores, and correlate restore issues with large mobile reports.

Feature Flag

Safety Assurance

Safety story

Automated test coverage

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 orangejenny as a code owner June 19, 2024 21:27
@gherceg gherceg requested a review from millerdev June 19, 2024 21:27
Copy link

sentry-io bot commented Jun 19, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: corehq/apps/app_manager/fixtures/mobile_ucr.py

Function Unhandled Issue
report_config_to_fixture MobileUCRTooLargeException /a/{domain}/phone/rest...
Event Count: 12

Did you find this useful? React with a 👍 or 👎

@gherceg gherceg force-pushed the gh/ucrs/datadog-metrics branch from ffdbdcc to dacb887 Compare June 19, 2024 21:34
@millerdev millerdev merged commit 1d0a7d0 into master Jun 20, 2024
13 checks passed
@millerdev millerdev deleted the gh/ucrs/datadog-metrics branch June 20, 2024 01:44
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