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

[Bug] Core changes to ImmutableOpenMap causing various failures #2714

Closed
stephen-crawford opened this issue Apr 21, 2023 · 3 comments
Closed
Labels
triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.

Comments

@stephen-crawford
Copy link
Contributor

stephen-crawford commented Apr 21, 2023

Builds against main (and potentially other versions) are encountering issues related to the valuesIt() method used by the clusterService when it tries to look at the list of indices.

/home/runner/work/security/security/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluator.java:652: error: cannot find symbol
                   return clusterService.state().getMetadata().getIndices().valuesIt();

These changes are the result of the Core refactor to no longer use ImmutableOpenMap.

Originally posted by @willyborankin in #2557 (comment)

@github-actions github-actions bot added the untriaged Require the attention of the repository maintainers and may need to be prioritized label Apr 21, 2023
@stephen-crawford
Copy link
Contributor Author

I expect this change caused the issue: opensearch-project/OpenSearch@1ab22e1

Working on swapping over now.

@stephen-crawford stephen-crawford added triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable. and removed untriaged Require the attention of the repository maintainers and may need to be prioritized labels Apr 24, 2023
@stephen-crawford stephen-crawford changed the title [Bug] ClusterService.state()...valuesIt() failing due to unknown symbol valuesIt() [Bug] Core changes to ImmutableOpenMap causing various failures Apr 24, 2023
@stephen-crawford
Copy link
Contributor Author

[Triage] @scrawfor99 is working on a fix for this issue at #2715.

@DarshitChanpura
Copy link
Member

Closing this as related PRs have been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.
Projects
None yet
Development

No branches or pull requests

2 participants