-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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: fix minion default bucket #37693
Conversation
Signed-off-by: yang.yu <yang.yu@chaitin.com>
717f248
to
2a21692
Compare
Signed-off-by: yang.yu <yang.yu@chaitin.com>
How is it going? |
We are going to review this logic internally as we want to further investigate the behaviour of the field that you propose to modify with a variable. We will notify you in this PR when there is any news. |
still open? |
ping |
bitnami/minio/2023/debian-11/rootfs/opt/bitnami/scripts/libminio.sh
Outdated
Show resolved
Hide resolved
# Start MinIO server in background | ||
minio_start_bg | ||
# Ensure MinIO Client is stopped when this script ends. | ||
trap "minio_stop" EXIT | ||
|
||
# set client env |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you explain the reason for moving this logic after minio starts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because minio will compare all MINIO_* env, it will cause server env not match error when set client env in front.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could affect initialisation, and the explanation is not clear to me. Could you give more details?
…io.sh Co-authored-by: Michiel <mdhont79@gmail.com> Signed-off-by: yywing <386542536@qq.com>
This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution. |
Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Pull Request. Do not hesitate to reopen it later if necessary. |
Description of the change
Wait for all node run create bucket done, then restart all node.
Benefits
Possible drawbacks
Applicable issues
Additional information