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 helm chart for versions 5.0.16 and 5.0.17 #2313

Merged

Conversation

pjuarezd
Copy link
Member

@pjuarezd pjuarezd commented Sep 5, 2024

This adds the helm charts for the versions v5.0.16 and v5.0.17, meanwhile ensures that 6.0.3 remains as the latest version.

Verification:

> cd operator
> python3 -m http.server 8080 &
> helm repo add local-minio http://localhost:8080

> helm search repo local-minio                                     
NAME                      	CHART VERSION	APP VERSION	DESCRIPTION                    
local-minio/minio-operator	4.3.7        	v4.3.7     	A Helm chart for MinIO Operator
local-minio/operator      	6.0.3        	v6.0.3     	A Helm chart for MinIO Operator
local-minio/tenant        	6.0.3        	v6.0.3     	A Helm chart for MinIO Operator

> helm search repo local-minio --versions | grep -E "5.0.16|5.0.17"
local-minio/operator      	5.0.17       	v5.0.17    	A Helm chart for MinIO Operator
local-minio/operator      	5.0.16       	v5.0.16    	A Helm chart for MinIO Operator
local-minio/tenant        	5.0.17       	v5.0.17    	A Helm chart for MinIO Operator
local-minio/tenant        	5.0.16       	v5.0.16    	A Helm chart for MinIO Operator

@pjuarezd pjuarezd self-assigned this Sep 5, 2024
ravindk89
ravindk89 previously approved these changes Sep 5, 2024
@thearchitector
Copy link

thearchitector commented Sep 9, 2024

there are CVE fixes in the 5.0.17 version that are not present in the most recent chart release (5.0.15), so i would appreciate this!

EDIT: looks like the .17 values specify .16 as the image tag. i imagine that's a mistake

Signed-off-by: pjuarezd <pjuarezd@users.noreply.github.com>
@pjuarezd pjuarezd force-pushed the add-helm-chart-for-5.0.16-5.0.17 branch from 4350ab6 to 4e7bf56 Compare September 11, 2024 06:39
Signed-off-by: pjuarezd <pjuarezd@users.noreply.github.com>
@pjuarezd
Copy link
Member Author

there are CVE fixes in the 5.0.17 version that are not present in the most recent chart release (5.0.15), so i would appreciate this!

EDIT: looks like the .17 values specify .16 as the image tag. i imagine that's a mistake

That's right, helm chart 5.0.17 was using 5.0.16 image in the compressed zip, already fixed thanks for checking!

@harshavardhana harshavardhana merged commit 3d90112 into minio:master Sep 11, 2024
21 checks passed
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.

4 participants