Skip to content

[Backport 7.72.x] [SYNTH-22405]: Fix Synthetics Scheduler Flush by locking it#41978

Merged
AliDatadog merged 1 commit into
7.72.xfrom
backport-41954-to-7.72.x
Oct 15, 2025
Merged

[Backport 7.72.x] [SYNTH-22405]: Fix Synthetics Scheduler Flush by locking it#41978
AliDatadog merged 1 commit into
7.72.xfrom
backport-41954-to-7.72.x

Conversation

@dd-octo-sts
Copy link
Copy Markdown
Contributor

@dd-octo-sts dd-octo-sts Bot commented Oct 15, 2025

Backport 75bc012 from #41954.


What does this PR do?

This PR fixes an issue in the Synthetics Scheduler where we weren't locking the flush function and therefore we had race conditions when updating the nextRun value for tests.

Motivation

Describe how you validated your changes

Local execution of the Agent.

Additional Notes

### What does this PR do?

This PR fixes an issue in the Synthetics Scheduler where we weren't locking the flush function and therefore we had race conditions when updating the nextRun value for tests.

### Motivation

### Describe how you validated your changes

Local execution of the Agent.

### Additional Notes

(cherry picked from commit 75bc012)
@dd-octo-sts dd-octo-sts Bot requested a review from a team as a code owner October 15, 2025 08:52
@dd-octo-sts dd-octo-sts Bot added changelog/no-changelog No changelog entry needed qa/done QA done before merge and regressions are covered by tests backport bot short review PR is simple enough to be reviewed quickly labels Oct 15, 2025
@AliDatadog AliDatadog merged commit a9be70f into 7.72.x Oct 15, 2025
72 of 73 checks passed
@AliDatadog AliDatadog deleted the backport-41954-to-7.72.x branch October 15, 2025 10:21
@github-actions github-actions Bot added this to the 7.72.0 milestone Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport bot changelog/no-changelog No changelog entry needed qa/done QA done before merge and regressions are covered by tests short review PR is simple enough to be reviewed quickly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants