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

[bitnami/mongodb-sharded] Fix "Authentication Failed" error if the mongodb-sharded Helm chart is deployed with custom root username #43526

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

jacky96623
Copy link
Contributor

Fix "Authentication Failed" error if the mongodb-sharded Helm chart is deployed with custom root username

Description of the change

This change replaces the hardcoded root username root by the dynamically injected environment variable MONGODB_ROOT_USERNAME.

Benefits

Due to the hardcode, "Authentication Failed" error occurs during the initialisation of a sharded MongoDB if a custom root username is supplied by the user (since the creation of admin user uses the variable MONGODB_ROOT_USERNAME instead of the hardcoded root username root). After the change, this error can be fixed.

Possible drawbacks

I believe this change has no drawback.

Applicable issues

N/A

Additional information

  • Hope that my PR is appropriate enough.
  • If it passes review, I hope that this can be released as soon as possible since I found this when I encountered the stated error during my work to set up a sharded MongoDB cluster.

@github-actions github-actions bot added the triage Triage is needed label Aug 4, 2023
@javsalgar javsalgar added the verify Execute verification workflow for these changes label Aug 7, 2023
@javsalgar
Copy link
Contributor

Hi!

Thank you so much for the PR! Could you fix the DCO issue?

…s deployed with custom root username

Signed-off-by: Jacky Ng <37379550+jacky96623@users.noreply.github.com>
@jacky96623
Copy link
Contributor Author

@javsalgar Sure! Please if it works now.

@github-actions github-actions bot added in-progress and removed triage Triage is needed labels Aug 8, 2023
@bitnami-bot bitnami-bot removed the request for review from javsalgar August 8, 2023 07:50
Copy link
Contributor

@aoterolorenzo aoterolorenzo left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@aoterolorenzo aoterolorenzo merged commit 7f831b1 into bitnami:main Aug 8, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mongodb-sharded solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants