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

Remove helm chart env #1165

Merged
merged 4 commits into from
Aug 22, 2024
Merged

Remove helm chart env #1165

merged 4 commits into from
Aug 22, 2024

Conversation

longshuicy
Copy link
Member

@longshuicy longshuicy commented Aug 12, 2024

Revert back the env
Introduce releaseName variable and compute all the necessary services uri
Backward compatible

Test:

kubectl config use-context software-dev
helm diff upgrade --namespace clowder2 clowder2 . --values chen-clowder2-software-dev.yaml

Download chen-clowder2-software-dev.yaml in box folder: https://uofi.app.box.com/folder/205332339557?s=cz324ukkqkhw3gggdryif60zu9lqmc2w


kubectl config use-context clowder 
helm diff upgrade --namespace littlebeats littlebeats . --values littlebeats.yaml      

littlebeats.yaml see: https://git.ncsa.illinois.edu/kubernetes/clusters/clowder/-/merge_requests/5/diffs#2155018879c64e49902152810a2b9d92bedb24cd

@lmarini lmarini merged commit 0a64e47 into release/v2.0-beta-3 Aug 22, 2024
6 checks passed
@lmarini lmarini deleted the update-helm-chart-env branch August 22, 2024 14:17
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.

2 participants