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

Fix default password for minio. #338

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Conversation

ewelinagr
Copy link
Collaborator

@ewelinagr ewelinagr commented Dec 11, 2024

MinIO won't start if the password length is less than 9 characters.

In the development setup there was a default password for MinIO, which does not meet the requirements (at least 8 characters). With that setup Minio hangs on "Adding local minio host to 'mc' configuration. " and won't start.

Note added by @pvannierop
Being able to use unmodified passwords is only relevant for development deployments that run wit non-randomized passwords. Hence, this change does not affect production envrionments.

@ewelinagr ewelinagr requested a review from pvannierop December 11, 2024 16:07
Copy link
Contributor

@pvannierop pvannierop left a comment

Choose a reason for hiding this comment

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

I would recommend some change to comments.

etc/base-secrets.yaml Show resolved Hide resolved
etc/base-secrets.yaml Outdated Show resolved Hide resolved
@keyvaann
Copy link
Collaborator

Minio shouldn't have a default password and it should be auto-generated. How does this issue happen?

Minio won't start if the password length is less than 9 characters.
@ewelinagr ewelinagr force-pushed the fix-default-minio-pass branch from 4414e84 to c0c8a57 Compare December 13, 2024 09:07
@ewelinagr ewelinagr requested a review from pvannierop December 13, 2024 09:19
Copy link
Contributor

@pvannierop pvannierop left a comment

Choose a reason for hiding this comment

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

The failing e2e-tests are not caused by the changes in this PR. I will approve and merge therefore.

@pvannierop pvannierop merged commit 4c7f0a4 into release-1.2 Dec 13, 2024
2 of 3 checks passed
@keyvaann
Copy link
Collaborator

keyvaann commented Dec 13, 2024

@pvannierop the CI test of last commit on the release branch was passing
a0e27d9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants