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

Revert "Make NodeEnvironment.nodeDataPaths singular (#72432)" #78861

Merged
merged 1 commit into from
Oct 11, 2021

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Oct 7, 2021

This reverts commit 6a7298e.

The revert was not clean. There were two adjustements necessary.
First, this effectively must revert #73136. Second, some tests
were written after MDP removal so needed to be made multi path aware.

relates #78525
relates #71205

This reverts commit 6a7298e.

The revert was not clean. There were two adjustements necessary.
First, this effectively must revert elastic#73136. Second, some tests
were written after MDP removal so needed to be made multi path aware.

relates elastic#78525
relates elastic#71205
@rjernst rjernst added >non-issue :Core/Infra/Core Core issues without another label v8.0.0 labels Oct 7, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Oct 7, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@rjernst rjernst mentioned this pull request Oct 7, 2021
17 tasks
Copy link
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

LGTM.

@rjernst rjernst merged commit 5744ed5 into elastic:master Oct 11, 2021
@rjernst rjernst deleted the mdp/revert9 branch October 11, 2021 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team v8.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants