Skip to content

test(taskworker): Lower exec time #91907

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

Merged
merged 1 commit into from
May 19, 2025
Merged

test(taskworker): Lower exec time #91907

merged 1 commit into from
May 19, 2025

Conversation

john-z-yang
Copy link
Member

No description provided.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 19, 2025
@john-z-yang john-z-yang force-pushed the john/test-big-args-2 branch from bd453db to bfb67ed Compare May 19, 2025 22:16
@john-z-yang john-z-yang marked this pull request as ready for review May 19, 2025 22:16
@john-z-yang john-z-yang requested a review from a team as a code owner May 19, 2025 22:16
@john-z-yang john-z-yang enabled auto-merge (squash) May 19, 2025 22:16
Copy link

codecov bot commented May 19, 2025

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

⚠️ Parser warning

The parser emitted a warning. Please review your JUnit XML file:

Warning while parsing testcase attributes: Limit of string is 1000 chars, for name, we got 2083 at 1:157341 in /home/runner/work/sentry/sentry/.artifacts/pytest.junit.xml
Files with missing lines Patch % Lines
src/sentry/taskworker/tasks/examples.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master   #91907   +/-   ##
=======================================
  Coverage   87.63%   87.63%           
=======================================
  Files       10356    10356           
  Lines      587139   587133    -6     
  Branches    22585    22585           
=======================================
- Hits       514511   514506    -5     
+ Misses      72200    72199    -1     
  Partials      428      428           

@john-z-yang john-z-yang merged commit ea86049 into master May 19, 2025
59 checks passed
@john-z-yang john-z-yang deleted the john/test-big-args-2 branch May 19, 2025 22:46
jan-auer added a commit that referenced this pull request May 20, 2025
* master: (58 commits)
  link: cleanup link (#91687)
  ref: create project_id index for organizationonboardingtask (#91918)
  storybook: smaller last edited (#91875)
  issues: fix chonk stacktrace alignment (#91891)
  alert: drop custom alert (#91892)
  insights: fix bar height (#91895)
  ref(span-buffer): Move max-memory-percentage to right CLI (#91924)
  ref(js): Factor button functionality (#91763)
  tests(resolve_groups): Clean up the tests (#91779)
  ref(span-buffer): Add backpressure (#91707)
  fix(nextjs-insights): project id is not passed to explore link (#91920)
  fix(crons): Floor seconds / microsecond on recorded dateClock (#91890)
  fix(uptime): Fix bug with the uptime_checks dataset in the events endpoint (#91824)
  ref: add state-only migration to reflect existing indexes in prod (#91901)
  ref: remove unnecssary metaclass (#91906)
  fix(stats): use data category title name (#91913)
  feat(issues): Add success messages to some actions (#91899)
  test(taskworker): Lower exec time (#91907)
  chore(aci): manually add spans for delayed workflow processing (#91908)
  chore(aci): remove uses of WorkflowFireHistory rollout columns (#91904)
  ...
@github-actions github-actions bot locked and limited conversation to collaborators Jun 4, 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