Skip to content

fix(crons): Temporarily increase frequency of the broken monitor task #68632

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 1 commit into from
Apr 10, 2024
Merged

fix(crons): Temporarily increase frequency of the broken monitor task #68632

merged 1 commit into from
Apr 10, 2024

Conversation

wedamija
Copy link
Member

We want to capture profiles for this task to see what's up with performance.

@wedamija
fix(crons): Temporarily increase frequency of the broken monitor task
a81947e 
We want to capture profiles for this task to see what's up with performance.
@wedamija wedamija requested a review from a team April 10, 2024 17:18
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 10, 2024
@wedamija wedamija enabled auto-merge (squash) April 10, 2024 17:20
@codecov Codecov
Copy link

codecov bot commented Apr 10, 2024

Bundle Report

Changes will increase total bundle size by 8.08kB ⬆️

Bundle name Size Change
sentry-webpack-bundle-array-push 26.27MB 8.08kB ⬆️

@wedamija wedamija merged commit 768f4e2 into master Apr 10, 2024
51 checks passed
@wedamija wedamija deleted the danf/crons-broken-monitor-freq branch April 10, 2024 17:47
@beezz beezz added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Apr 11, 2024
@getsentry-bot
Copy link
Contributor

PR reverted: 1c7d2b4

getsentry-bot added a commit that referenced this pull request Apr 11, 2024
@getsentry-bot @beezz
Revert "fix(crons): Temporarily increase frequency of the broken moni…
1c7d2b4 
…tor task (#68632)"

This reverts commit 768f4e2.

Co-authored-by: beezz <1500654+beezz@users.noreply.github.com>
wedamija added a commit that referenced this pull request Apr 11, 2024
@wedamija
Revert "Revert "fix(crons): Temporarily increase frequency of the bro…
ba201b7 
…ken monitor task (#68632)""

This reverts commit 1c7d2b4.

We've disabled this feature flag for now. I want to get this running frequently again so that I can perform some tests.
wedamija added a commit that referenced this pull request Apr 11, 2024
@wedamija
Revert "Revert "fix(crons): Temporarily increase frequency of the bro…
d7df4b4 
…ken monitor task (#68632)"" (#68746)

This reverts commit 1c7d2b4.

We've disabled this feature flag for now. I want to get this running
frequently again so that I can perform some tests.
c298lee pushed a commit that referenced this pull request Apr 12, 2024
@wedamija @c298lee
fix(crons): Temporarily increase frequency of the broken monitor task (…
3c0ebe1 
…#68632)

We want to capture profiles for this task to see what's up with
performance.
c298lee pushed a commit that referenced this pull request Apr 12, 2024
@getsentry-bot @beezz @c298lee
Revert "fix(crons): Temporarily increase frequency of the broken moni…
7f30e59 
…tor task (#68632)"

This reverts commit 768f4e2.

Co-authored-by: beezz <1500654+beezz@users.noreply.github.com>
c298lee pushed a commit that referenced this pull request Apr 12, 2024
@wedamija @c298lee
Revert "Revert "fix(crons): Temporarily increase frequency of the bro…
ea13255 
…ken monitor task (#68632)"" (#68746)

This reverts commit 1c7d2b4.

We've disabled this feature flag for now. I want to get this running
frequently again so that I can perform some tests.
getsentry-bot added a commit that referenced this pull request Apr 12, 2024
@getsentry-bot @wedamija
Revert "Revert "Revert "fix(crons): Temporarily increase frequency of…
01b280a 
… the broken monitor task (#68632)"" (#68746)"

This reverts commit d7df4b4.

Co-authored-by: wedamija <6288560+wedamija@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators Apr 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@evanpurkhiser evanpurkhiser evanpurkhiser approved these changes

Assignees
No one assigned
Labels
Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert Add to a merged PR to revert it (skips CI)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@wedamija @getsentry-bot @evanpurkhiser @beezz