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

Add auth password for user postgres #155

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Conversation

ybak
Copy link
Contributor

@ybak ybak commented Jul 7, 2024

without "postgresPassword “property, job "skywalking-skywalking-helm-oap-init" will fail. Cause the other auth properties didn't work for default account "postgres ", and postgresql generated a random password for the default account "postgres "。

without the postgresPassword ,  job skywalking-skywalking-helm-oap-init will fail , cause postgresql  generate a random password。
@wu-sheng wu-sheng requested a review from kezhenxu94 July 7, 2024 15:44
@wu-sheng wu-sheng added the bug Something isn't working label Jul 7, 2024
@wu-sheng wu-sheng added this to the 4.7.0 milestone Jul 7, 2024
@kezhenxu94
Copy link
Member

kezhenxu94 commented Jul 7, 2024

Does the field password not work anymore?

@ybak
Copy link
Contributor Author

ybak commented Jul 8, 2024

Does the field password not work anymore?

yes, check this out : https://artifacthub.io/packages/helm/bitnami/postgresql/

global.postgresql.auth.postgresPassword Password for the "postgres" admin user (overrides auth.postgresPassword) ""
global.postgresql.auth.username Name for a custom user to create (overrides auth.username) ""
global.postgresql.auth.password Password for the custom user to create (overrides auth.password) ""

@kezhenxu94 kezhenxu94 changed the title fix enabled postgresql but deploy failed case Add auth password for user postgres Jul 8, 2024
@kezhenxu94 kezhenxu94 merged commit 3effe7b into apache:master Jul 8, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants