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

chore: bump stackable-operator version and fix minio installation command #553

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

xeniape
Copy link
Member

@xeniape xeniape commented Dec 9, 2024

Description

When running the minio installation command, the following error is thrown:

helm install minio \
--version 4.0.2 \
--namespace default \
--set mode=standalone \
--set replicas=1 \
--set persistence.enabled=false \
--set buckets[0].name=hive,buckets[0].policy=none \
--set users[0].accessKey=hive,users[0].secretKey=hivehive,users[0].policy=readwrite \
--set resources.requests.memory=1Gi \
--set service.type=NodePort,service.nodePort=null \
--set consoleService.type=NodePort,consoleService.nodePort=null \
--repo https://charts.min.io/ minio
zsh: no matches found: buckets[0].name=hive,buckets[0].policy=none

This PR adds double quotes to the two settings throwing this error, which fixes it.

Edit: Also bumped stackable-operator version as part of this PR due to cargo deny errors.

@xeniape xeniape self-assigned this Dec 9, 2024
sbernauer
sbernauer previously approved these changes Dec 9, 2024
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Thanks!
Not on my computer, but maybe this is a zsh vs bash thing

@sbernauer
Copy link
Member

Please feel free to run cargo update

@xeniape xeniape changed the title docs: fix minio installation command chore: bump stackable-operator version and fix minio installation command Dec 10, 2024
@xeniape xeniape requested a review from sbernauer December 10, 2024 13:57
@sbernauer
Copy link
Member

Btw, the command already worked in bash, but not zsh ;)

@xeniape
Copy link
Member Author

xeniape commented Dec 16, 2024

Btw, the command already worked in bash, but not zsh ;)

Yes, tried the updated command with bash to make sure it didn't break there after the change.

@xeniape xeniape added this pull request to the merge queue Dec 16, 2024
Merged via the queue into main with commit 8f100f4 Dec 16, 2024
17 checks passed
@xeniape xeniape deleted the docs/fix-minio-command branch December 16, 2024 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Development: Done
Development

Successfully merging this pull request may close these issues.

2 participants