Skip to content

Commit f5ef1a5

Browse files
[8.19] [ML] Removing removeIfExists from sync task scheduler (#228783) (#229916)
# Backport This will backport the following commits from `main` to `8.19`: - [[ML] Removing removeIfExists from sync task scheduler (#228783)](#228783) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"James Gowdy","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-07-21T15:33:36Z","message":"[ML] Removing removeIfExists from sync task scheduler (#228783)\n\nThe call to `removeIfExists` is not necessary and can result in a\npossible race condition when the sync task is claimed.\nFixes https://github.com/elastic/kibana/issues/226554\nRelated to https://github.com/elastic/kibana/pull/226481","sha":"c346834223f93a62f377d3d245b60a0390c5ed3e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix",":ml","backport:version","v9.2.0","v9.0.5","v9.1.1","v8.18.5","v8.19.1"],"title":"[ML] Removing removeIfExists from sync task scheduler","number":228783,"url":"https://github.com/elastic/kibana/pull/228783","mergeCommit":{"message":"[ML] Removing removeIfExists from sync task scheduler (#228783)\n\nThe call to `removeIfExists` is not necessary and can result in a\npossible race condition when the sync task is claimed.\nFixes https://github.com/elastic/kibana/issues/226554\nRelated to https://github.com/elastic/kibana/pull/226481","sha":"c346834223f93a62f377d3d245b60a0390c5ed3e"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/228783","number":228783,"mergeCommit":{"message":"[ML] Removing removeIfExists from sync task scheduler (#228783)\n\nThe call to `removeIfExists` is not necessary and can result in a\npossible race condition when the sync task is claimed.\nFixes https://github.com/elastic/kibana/issues/226554\nRelated to https://github.com/elastic/kibana/pull/226481","sha":"c346834223f93a62f377d3d245b60a0390c5ed3e"}},{"branch":"9.0","label":"v9.0.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/229686","number":229686,"state":"MERGED","mergeCommit":{"sha":"cdd1d7dd1c9e8f0e816d6133b5bf62db164efecf","message":"[9.0] [ML] Removing removeIfExists from sync task scheduler (#228783) (#229686)\n\n# Backport\n\nThis will backport the following commits from `main` to `9.0`:\n- [[ML] Removing removeIfExists from sync task scheduler\n(#228783)](https://github.com/elastic/kibana/pull/228783)\n\n\n\n### Questions ?\nPlease refer to the [Backport tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by: James Gowdy <[email protected]>"}},{"branch":"9.1","label":"v9.1.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/229684","number":229684,"state":"MERGED","mergeCommit":{"sha":"e8dd72e7489f6b2f7dad4d3a6a93b7b0601bdb35","message":"[8.18] [ML] Removing removeIfExists from sync task scheduler (#228783) (#229684)\n\n# Backport\n\nThis will backport the following commits from `main` to `8.18`:\n- [[ML] Removing removeIfExists from sync task scheduler\n(#228783)](https://github.com/elastic/kibana/pull/228783)\n\n\n\n### Questions ?\nPlease refer to the [Backport tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by: James Gowdy <[email protected]>"}},{"branch":"8.19","label":"v8.19.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: James Gowdy <[email protected]>
1 parent 238c2e7 commit f5ef1a5

File tree

1 file changed

+0
-1
lines changed
  • x-pack/platform/plugins/shared/ml/server/saved_objects

1 file changed

+0
-1
lines changed

x-pack/platform/plugins/shared/ml/server/saved_objects/sync_task.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,6 @@ export class SavedObjectsSyncService {
135135
): Promise<TaskInstance | null> {
136136
this.core = core;
137137
try {
138-
await taskManager.removeIfExists(SAVED_OBJECTS_SYNC_TASK_ID);
139138
const state: LatestTaskStateSchema = {
140139
runs: 0,
141140
totalSavedObjectsSynced: 0,

0 commit comments

Comments
 (0)