Skip to content

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)

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