Skip to content

Propagate cancellation in DataTiersUsageTransportAction #100253

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

Conversation

DaveCTurner
Copy link
Contributor

This action invokes a subsidiary action but does not set up the proper
parent/child relationship, so cancellations of the parent task do not
propagate to the child.

Relates #100230

This action invokes a subsidiary action but does not set up the proper
parent/child relationship, so cancellations of the parent task do not
propagate to the child.

Relates elastic#100230
@DaveCTurner DaveCTurner added >bug :Data Management/Data streams Data streams and their lifecycles v8.11.0 labels Oct 4, 2023
@DaveCTurner DaveCTurner requested a review from andreidan October 4, 2023 09:11
@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label Oct 4, 2023
@elasticsearchmachine
Copy link
Collaborator

Hi @DaveCTurner, I've created a changelog YAML for you.

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

Copy link
Contributor

@andreidan andreidan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this David.

LGTM (assuming CI is happy)

@@ -74,7 +75,7 @@ protected void masterOperation(
ClusterState state,
ActionListener<XPackUsageFeatureResponse> listener
) {
client.admin()
new ParentTaskAssigningClient(client, clusterService.localNode(), task).admin()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL about ParentTaskAssigningClient

@DaveCTurner DaveCTurner added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Oct 4, 2023
@elasticsearchmachine elasticsearchmachine merged commit c99896e into elastic:main Oct 4, 2023
@DaveCTurner DaveCTurner deleted the 2023/10/04/data-tiers-usage-cancellation branch October 4, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >bug :Data Management/Data streams Data streams and their lifecycles Team:Data Management Meta label for data/management team v8.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants