Skip to content
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

Deprecate the xpack.task_manager.claim_strategy setting #202492

Closed
4 tasks
mikecote opened this issue Dec 2, 2024 · 1 comment · Fixed by #203341
Closed
4 tasks

Deprecate the xpack.task_manager.claim_strategy setting #202492

mikecote opened this issue Dec 2, 2024 · 1 comment · Fixed by #203341
Assignees
Labels
Feature:Task Manager Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@mikecote
Copy link
Contributor

mikecote commented Dec 2, 2024

The xpack.task_manager.claim_strategy setting was introduced to feature flag the new mget claim strategy and eventually turn into a fallback mechanism if ever a cluster was experiencing issues with mget. Now that we're rolling out mget as the default to everyone in 8.17, we should deprecate in 9.0 the setting to discourage its usage, given we'll be fixing issues, if any, as they arise.

In a future 9.x version, we plan to make this setting a no-op and remove the update_by_query task claimer.

Definition of Done

  • Change applied to 9.0+ only
  • Configuration setting deprecated using the deprecations object
  • Message to ask users to remove this setting as a fix
  • Tests
@mikecote mikecote added Feature:Task Manager Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Dec 2, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Task Manager Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants