Skip to content

feat(preprod): add e2e timing metrics#101575

Merged
trevor-e merged 8 commits into
masterfrom
telkins/e2e-metrics
Oct 16, 2025
Merged

feat(preprod): add e2e timing metrics#101575
trevor-e merged 8 commits into
masterfrom
telkins/e2e-metrics

Conversation

@trevor-e

Copy link
Copy Markdown
Member

So we can track whether any particular org/project is taking longer to process builds.

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 15, 2025
@trevor-e

Copy link
Copy Markdown
Member Author

cursor review

cursor[bot]

This comment was marked as outdated.

@trevor-e

Copy link
Copy Markdown
Member Author

cursor review

cursor[bot]

This comment was marked as outdated.

@codecov

codecov Bot commented Oct 15, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #101575      +/-   ##
===========================================
+ Coverage   80.34%    80.97%   +0.62%     
===========================================
  Files        8703      8705       +2     
  Lines      386942    387107     +165     
  Branches    24540     24540              
===========================================
+ Hits       310878    313447    +2569     
+ Misses      75717     73313    -2404     
  Partials      347       347              

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@trevor-e trevor-e marked this pull request as ready for review October 16, 2025 14:51
@trevor-e trevor-e requested a review from a team as a code owner October 16, 2025 14:51
}
)
# Always trigger status check update (success or failure)
create_preprod_status_check_task.apply_async(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also moved this into the finally block so the status check is always updated.

@chromy chromy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@trevor-e trevor-e merged commit 9ffe38e into master Oct 16, 2025
68 checks passed
@trevor-e trevor-e deleted the telkins/e2e-metrics branch October 16, 2025 15:29
@sentry

sentry Bot commented Oct 16, 2025

Copy link
Copy Markdown
Contributor

Issues attributed to commits in this pull request

This pull request was merged and Sentry observed the following issues:

@github-actions github-actions Bot locked and limited conversation to collaborators Nov 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants