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

feat(helm): Uplift to Helm v3, various requests, tag latest image #249

Merged
merged 5 commits into from
Jan 28, 2023

Conversation

karlskewes
Copy link
Contributor

Typical practice is to tag the most recently built image with latest. Currently the project tags with master and others.
See: https://github.com/docker/metadata-action#latest-tag

For Helm changes, each one was tested one by one via: helm template . | kubectl apply -f -
Confirm Sloop comes up with expected change and then: helm template . | kubectl delete -f - to remove resources from the cluster.

Closes: #86 #165 #233

New images are only pushed to GHCR.
When the PVC is enabled, a PV will be created and mounted to the pod
instead of the emptyDir.
For pulling images from a private registry.
@salesforce-cla
Copy link

Thanks for the contribution! Before we can merge this, we need @karlskewes to sign the Salesforce.com Contributor License Agreement.

@sana-jawad
Copy link
Collaborator

Thanks @karlskewes for the contribution

@karlskewes
Copy link
Contributor Author

I'm not sure what has happened with the CLA.

Oops, an error occurred
This exception has been logged with id 80djblo96.

@sana-jawad sana-jawad merged commit 1ac931c into salesforce:master Jan 28, 2023
@sana-jawad
Copy link
Collaborator

ception has bee

No worries I have merged it

@karlskewes karlskewes deleted the helm3 branch January 28, 2023 16:42
@karlskewes
Copy link
Contributor Author

Part of #245 "On merge PR to master:"

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

Successfully merging this pull request may close these issues.

Add helm options for clusters without PVCs
2 participants