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.

Sign up for GitHub

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

Jump to bottom

Remove LegacyCTRAL from MetadataIndexTemplateService #86459

Merged
merged 1 commit into from
May 5, 2022
Merged

Remove LegacyCTRAL from MetadataIndexTemplateService #86459

merged 1 commit into from
May 5, 2022

Conversation

DaveCTurner
Copy link
Contributor

Relates #83784, #86017

@DaveCTurner DaveCTurner added >non-issue :Data Management/Indices APIs APIs to create and manage indices and templates v8.3.0 labels May 5, 2022
@DaveCTurner DaveCTurner requested a review from dakrone May 5, 2022 10:39
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label May 5, 2022
@elasticmachine
Copy link
Collaborator

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

@DaveCTurner
Copy link
Contributor Author

Not quite as elegant as #86446 - these tasks involve various components of a Metadata and in several places we need to rebuild a Metadata for validation. It's probably possible to push this down to a lower level but it'd be a much bigger change so I'm just sticking with iterating on the ClusterState here.

@DaveCTurner DaveCTurner mentioned this pull request May 5, 2022
Closed
16 tasks
dakrone
dakrone approved these changes May 5, 2022
View reviewed changes
Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner DaveCTurner merged commit 3ef46b0 into elastic:master May 5, 2022
@DaveCTurner DaveCTurner deleted the 2022-05-05-LegacyCTRAL-MetadataIndexTemplateService branch May 5, 2022 14:52
@DaveCTurner
Copy link
Contributor Author

Thanks Lee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dakrone dakrone dakrone approved these changes

Assignees
No one assigned
Labels
:Data Management/Indices APIs APIs to create and manage indices and templates >non-issue Team:Data Management Meta label for data/management team v8.3.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DaveCTurner @elasticmachine @dakrone