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/etcd] Root password is logged at the startup of etcd pods #23910

Closed
jb-abbadie opened this issue Feb 26, 2024 · 4 comments
Closed

[bitnami/etcd] Root password is logged at the startup of etcd pods #23910

jb-abbadie opened this issue Feb 26, 2024 · 4 comments
Assignees
Labels
solved tech-issues The user has a technical issue about an application triage Triage is needed

Comments

@jb-abbadie
Copy link
Contributor

Name and Version

bitnami/etcd 9.14.2

What architecture are you using?

amd64

What steps will reproduce the bug?

  1. $ helm install etcd oci://registry-1.docker.io/bitnamicharts/etcd --version 9.14.2 --set auth.rbac.rootPassword=fakepassword
  2. Wait for pod etcd-0 to be created
  3. $ kubectl logs etcd-0 | grep fakepassword
  4. Observe the root password in the logs

Are you using any custom parameters or values?

Only parameter is auth.rbac.rootPassword is set.

This also happens when setting the root password through auth.rbac.existingSecret

What is the expected behavior?

Pod log do not contain the root password

What do you see instead?

The log contain the root password

Additional information

This issue was already fixed in 2020 #1909 but seems to have been reintroduced since.

The fix probably involves adding the command unset ETCD_ROOT_PASSWORD before the startup of the etcd process.
This script is in https://github.com/bitnami/containers/blob/main/bitnami/etcd/3.5/debian-12/rootfs/opt/bitnami/scripts/etcd/run.sh , I can file a bug in this repo instead if its easier.

@jb-abbadie jb-abbadie added the tech-issues The user has a technical issue about an application label Feb 26, 2024
@github-actions github-actions bot added the triage Triage is needed label Feb 26, 2024
@javsalgar
Copy link
Contributor

Hi!

Thank you so much for reporting. Would you like to submit a PR unsetting the password value before launching etcd?

@jb-abbadie
Copy link
Contributor Author

I created this PR to fix the issue

@javsalgar
Copy link
Contributor

Thank you so much for the PR! The team will take a look

@jb-abbadie
Copy link
Contributor Author

Closing since it was fixed and a new version of the chart is released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved tech-issues The user has a technical issue about an application triage Triage is needed
Projects
None yet
Development

No branches or pull requests

2 participants