Skip to content

Conversation

@paul-mesnilgrente
Copy link
Contributor

@paul-mesnilgrente paul-mesnilgrente commented Oct 23, 2025

Dashboard metrics showing correctly

As a content partner I want to see correct dashboard metrics so that I have the right data

Acceptance Criteria

  • api-metrics-crushing and api-request-metrics-summarize are running successfully
  • digitalnz.org/dashboard is not missing any data

Notes

  • We've done a lot of work recently on the api-request-metrics-summarize job sc-159778
  • But there still seems to be a problem:
    A date from May works:
    https://.../v3/metrics/?facets=AnyQuestions&start_date=2025-05-01&end_date=2025-05-05&metrics=view
    But June data is still missing:
    https://.../v3/metrics/?facets=AnyQuestions&start_date=2025-06-01&end_date=2025-06-05&metrics=view
  • We've been getting a lot of

Unexpected Container State
[production] 2025-08-26T20:00:57.243Z - api-metrics-crushing-29270250-87ptg (api-metrics-crushing) - ContainerStatusUnknown

  • Whats the difference between api-metrics-crushing  and the api-request-metrics-summarize ?
  • Could this problem go away now that Paul has bumped up the "max EC2 limit" in production?
  • Does this relate to a similar issue with digitalnz-popular-searches sc-161225

@github-actions
Copy link

github-actions bot commented Oct 23, 2025

Code quality score

Nice work!! The code quality has improved for this PR! ✨ 🌈 🎉 🌟

Ruby file count Similarity score (flay) ABC complexity (flog) Code smells (reek) TOTALS
base 145 3.35 8.08 22.05 33.48
this branch 146 3.34 8.0 21.6 32.94
difference 1 -0.01 -0.08 -0.45 -0.54

@paul-mesnilgrente paul-mesnilgrente force-pushed the pm/crushing branch 7 times, most recently from 27cd74e to 134ff08 Compare October 29, 2025 21:14
@paul-mesnilgrente paul-mesnilgrente force-pushed the pm/crushing branch 4 times, most recently from 4cc743a to 735a6f8 Compare November 12, 2025 20:15
@paul-mesnilgrente paul-mesnilgrente force-pushed the pm/crushing branch 8 times, most recently from 6737824 to 32fdd4c Compare November 13, 2025 15:03
@paul-mesnilgrente paul-mesnilgrente merged commit 79e0f66 into main Nov 18, 2025
5 checks passed
@paul-mesnilgrente paul-mesnilgrente deleted the pm/crushing branch November 18, 2025 21:43
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.

3 participants