Skip to content

feat(cp): estimate dropped continuous profiling stats page #88034

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Mar 28, 2025
Merged

feat(cp): estimate dropped continuous profiling stats page #88034

merged 10 commits into from
Mar 28, 2025

Conversation

@brendanhsentry
Copy link
Member

@brendanhsentry brendanhsentry commented Mar 26, 2025 •
edited
Loading

@brendanhsentry brendanhsentry changed the title Brendan/estimate chunks stats page feat(cp): estimate dropped continuous profiling stats page Mar 26, 2025
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 26, 2025
@codecov
Copy link

codecov bot commented Mar 28, 2025 •
edited
Loading

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
10045 1 10044 11
View the top 1 failed test(s) by shortest run time 
OrganizationStats shows only profile duration category when both profiling features are enabled
Stack Traces | 0.518s run time 
TestingLibraryElementError: Unable to find an element by: [data-test-id="estimation-text"]

Ignored nodes: comments, script, style
...
    at Object.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19
    at Object.<anonymous> (.../views/organizationStats/index.spec.tsx:494:40)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@brendanhsentry brendanhsentry requested a review from a team March 28, 2025 20:40
@brendanhsentry brendanhsentry marked this pull request as ready for review March 28, 2025 20:40
@brendanhsentry brendanhsentry requested a review from a team as a code owner March 28, 2025 20:40
@brendanhsentry brendanhsentry merged commit 426ef74 into master Mar 28, 2025
41 checks passed
@brendanhsentry brendanhsentry deleted the brendan/estimate-chunks-stats-page branch March 28, 2025 20:50
andrewshie-sentry pushed a commit that referenced this pull request Mar 31, 2025
@brendanhsentry @andrewshie-sentry
feat(cp): estimate dropped continuous profiling stats page (#88034)
6f5f544 
stats page part of getsentry/getsentry#16904

<img width="1587" alt="Screenshot 2025-03-28 at 1 27 30 PM"
src="https://github.com/user-attachments/assets/313b23d1-8a67-4747-bbec-6fdf2791b290"
/>
<img width="1587" alt="Screenshot 2025-03-28 at 1 27 55 PM"
src="https://github.com/user-attachments/assets/708404d8-dfff-411c-bda6-9b66376a91fb"
/>
<img width="1523" alt="Screenshot 2025-03-28 at 12 00 31 PM"
src="https://github.com/user-attachments/assets/c73f1012-1409-45b4-ab6b-c631d9bd572c"
/>
@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@dashed dashed dashed approved these changes

+1 more reviewer

@jarrettscott jarrettscott jarrettscott approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@brendanhsentry @dashed @jarrettscott