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

[FEATURE] Operator should automatically update User password if it was changed in the Secret #908

Open
evheniyt opened this issue Nov 19, 2024 · 2 comments · May be fixed by #922
Open

[FEATURE] Operator should automatically update User password if it was changed in the Secret #908

evheniyt opened this issue Nov 19, 2024 · 2 comments · May be fixed by #922
Labels
enhancement New feature or request

Comments

@evheniyt
Copy link
Contributor

Right now, it is impossible to update a User password if it was changed in the corresponding Secret.

Operator already has a handleSecretEvent functionality, but in the current implementation, it is useless. Yes, it will trigger reconcile if smth was changed in the Secret, but because of the logic inside Users Reconcile function it will do nothing if password was updated.

The main problem is that Opensearch API is not returning a User password so you couldn't just compare it like Operator does with other properties.

Proposal: Operator should automatically update User password if it was changed in the Secret.

@evheniyt evheniyt added enhancement New feature or request untriaged Issues that have not yet been triaged labels Nov 19, 2024
@evheniyt
Copy link
Contributor Author

@swoehrl-mw I have few ideas on how we could improve and implement this, would you like me to share a PR?

@swoehrl-mw
Copy link
Collaborator

@evheniyt Please go ahead, PRs are always welcome. And thanks for reporting this.

@swoehrl-mw swoehrl-mw removed the untriaged Issues that have not yet been triaged label Nov 21, 2024
@evheniyt evheniyt linked a pull request Dec 27, 2024 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🆕 New
Development

Successfully merging a pull request may close this issue.

2 participants