Skip to content

[8.19] Validate interval when a task is updated (#222486)#224717

Merged
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-222486
Jun 22, 2025
Merged

[8.19] Validate interval when a task is updated (#222486)#224717
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-222486

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

Resolves: elastic#213391

This PR adds validation for task interval where it is updated.

There are 5 places in the TaskStore that we update the interval:
`schedule`, `bulkSchedule`, `update`, `bulkUpdate` and
`bulkPartialUpdate`

For the bulk operations we just skip the task with invalid interval, for
schedule and update throw an error and stop the execution.

## To verify:
Create a rule with connector.
Let it run for a while.
Then update the rule.

There shouldn't be any error.

(cherry picked from commit 723fcc3)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 20, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) June 20, 2025 14:51
@elasticmachine
Copy link
Contributor

elasticmachine commented Jun 20, 2025

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #20 / getUseCellActionsHook should render cell actions correctly for gridView view
  • [job] [logs] FTR Configs #43 / Security Solution - Telemetry Security Telemetry - Indices metadata task telemetry @ess indices metadata should publish ilm policy events
  • [job] [logs] x-pack/test/api_integration/deployment_agnostic/configs/stateful/oblt.stateful.config.ts / Stateful Observability - Deployment-agnostic API integration tests Observability Alerting Synthetics Alerting SyntheticsCustomStatusRule NumberOfChecks - Location threshold > 1 - ungrouped - 2 down locations should be down again
  • [job] [logs] x-pack/test/api_integration/deployment_agnostic/configs/stateful/oblt.stateful.config.ts / Stateful Observability - Deployment-agnostic API integration tests Observability Alerting Synthetics Alerting SyntheticsCustomStatusRule NumberOfChecks - Location threshold > 1 - ungrouped - 2 down locations should be down again

Metrics [docs]

Unknown metric groups

References to deprecated APIs

id before after diff
taskManager 12 14 +2

History

cc @ersin-erdal

@kibanamachine kibanamachine merged commit 454d5d1 into elastic:8.19 Jun 22, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants