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

Legacy reference to opendistro in internal_users.yml #2058

Closed
cwperks opened this issue Aug 30, 2022 · 1 comment
Closed

Legacy reference to opendistro in internal_users.yml #2058

cwperks opened this issue Aug 30, 2022 · 1 comment
Labels
refactoring code/test refactoring untriaged Require the attention of the repository maintainers and may need to be prioritized

Comments

@cwperks
Copy link
Member

cwperks commented Aug 30, 2022

There is a reference to opendistro in internal_users.yml as an attribute name opendistro_security_roles. This was discovered in (#1339) where a user had copied the example configuration on the documentation website which referenced opensearch_security_roles and received an Unrecognized field error when running security admin.

The documentation website has since been reverted to display opendistro_security_roles in this PR: opensearch-project/documentation-website#892.

We should revisit this area to replace the reference to the legacy opendistro.

@cwperks cwperks added bug Something isn't working untriaged Require the attention of the repository maintainers and may need to be prioritized labels Aug 30, 2022
@cwperks cwperks added refactoring code/test refactoring and removed bug Something isn't working labels Aug 30, 2022
@peternied
Copy link
Member

Thanks for filing this, I think that this would be a good fix, but this might not be a good time to make this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring code/test refactoring untriaged Require the attention of the repository maintainers and may need to be prioritized
Projects
None yet
Development

No branches or pull requests

2 participants