-
Notifications
You must be signed in to change notification settings - Fork 513
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
Allow Helm Chart deploy image from digest #780
base: master
Are you sure you want to change the base?
Allow Helm Chart deploy image from digest #780
Conversation
47510d7
to
f26b337
Compare
deployments/kubernetes/chart/reloader/templates/deployment.yaml
Outdated
Show resolved
Hide resolved
@CharmanderJieniJieni looks good, please just resolve the conflict before we merge this |
cf46946
to
a822163
Compare
@karl-johan-grahn patched against latest commits |
hi @CharmanderJieniJieni can u pls rebase on master branch, and update helm chart version prematurely in chart.yaml? |
GKE Binary Authorization only supports image pull by digest - [Ref](https://cloud.google.com/binary-authorization/docs/deploying-containers#deploy_the_container_image) * if `.values.reloader.deployment.image.digest` is defined, pull image from digest * otherwise pull image from tag
a822163
to
b433a2d
Compare
done |
@CharmanderJieniJieni pls rebase again. sorry for the trouble |
GKE Binary Authorization only supports image pull by digest - Ref
.values.reloader.deployment.image.digest
is defined, pull image from digest