Skip to content

[8.x] [9.0] Remove duplicate paths (including exclusive) in FileAccessTree (#123776 and #124023) (#123924) #124332

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

Merged
merged 6 commits into from
Mar 9, 2025

Conversation

prdoyle
Copy link
Contributor

@prdoyle prdoyle commented Mar 7, 2025

…lastic#123776 and elastic#124023) (elastic#123924)

* remove duplicate paths in FileAccessTree (elastic#123776)

Co-authored-by: Lorenzo Dematté <[email protected]>
Co-authored-by: Patrick Doyle <[email protected]>

* Remove duplicate exclusive paths (elastic#124023)

* Remove duplicate exclusive paths

* Normalize paths in tests to support Windows

* Remove withMode

---------

Co-authored-by: Jack Conradson <[email protected]>
Co-authored-by: Lorenzo Dematté <[email protected]>
@prdoyle prdoyle added :Core/Infra/Entitlements Entitlements infrastructure >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Core/Infra Meta label for core/infra team test-entitlements test-windows Trigger CI checks on Windows labels Mar 7, 2025
@prdoyle
Copy link
Contributor Author

prdoyle commented Mar 7, 2025

🤔

You are running with unsupported Java 24. To make full use of the Vector API, please update Elasticsearch

@ldematte
Copy link
Contributor

ldematte commented Mar 8, 2025

Windows failures are known issue: #124103 and #124104. They have been muted in 8.18 and 9 but not on 8.x

@rjernst
Copy link
Member

rjernst commented Mar 8, 2025

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine merged commit db48a0f into elastic:8.x Mar 9, 2025
20 checks passed
@prdoyle prdoyle deleted the backport/8.x/pr-123924 branch March 9, 2025 11:11
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!) backport :Core/Infra/Entitlements Entitlements infrastructure >non-issue Team:Core/Infra Meta label for core/infra team test-windows Trigger CI checks on Windows v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants