Skip to content

Commit

Permalink
helm: use IfNotPresent for versioned helm chart release (#3289)
Browse files Browse the repository at this point in the history
Signed-off-by: zirain <zirain2009@gmail.com>
Co-authored-by: Guy Daich <guy.daich@sap.com>
  • Loading branch information
zirain and guydc authored Apr 30, 2024
1 parent 8b5f031 commit 2e0971b
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build_and_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -174,4 +174,4 @@ jobs:
- name: Build and Push EG Latest Helm Chart
if: github.event_name == 'push' && github.ref == 'refs/heads/main'
# use `0.0.0` as the default latest version.
run: OCI_REGISTRY=oci://docker.io/envoyproxy CHART_VERSION=v0.0.0-latest TAG=latest make helm-package helm-push
run: IMAGE_PULL_POLICY=Always OCI_REGISTRY=oci://docker.io/envoyproxy CHART_VERSION=v0.0.0-latest TAG=latest make helm-package helm-push
2 changes: 1 addition & 1 deletion charts/gateway-helm/values.tmpl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ deployment:
image:
repository: ${ImageRepository}
tag: '${ImageTag}'
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
imagePullSecrets: []
resources:
limits:
Expand Down
2 changes: 1 addition & 1 deletion site/content/en/latest/install/api.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ The Helm chart for Envoy Gateway
| createNamespace | bool | `false` | |
| deployment.envoyGateway.image.repository | string | `"${ImageRepository}"` | |
| deployment.envoyGateway.image.tag | string | `"${ImageTag}"` | |
| deployment.envoyGateway.imagePullPolicy | string | `"Always"` | |
| deployment.envoyGateway.imagePullPolicy | string | `"IfNotPresent"` | |
| deployment.envoyGateway.imagePullSecrets | list | `[]` | |
| deployment.envoyGateway.resources.limits.cpu | string | `"500m"` | |
| deployment.envoyGateway.resources.limits.memory | string | `"1024Mi"` | |
Expand Down
4 changes: 2 additions & 2 deletions test/helm/default.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -369,7 +369,7 @@ spec:
- name: KUBERNETES_CLUSTER_DOMAIN
value: cluster.local
image: docker.io/envoyproxy/gateway-dev:latest
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
livenessProbe:
httpGet:
path: /healthz
Expand Down Expand Up @@ -517,7 +517,7 @@ spec:
- name: KUBERNETES_CLUSTER_DOMAIN
value: cluster.local
image: docker.io/envoyproxy/gateway-dev:latest
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
name: envoy-gateway-certgen
restartPolicy: Never
securityContext:
Expand Down

0 comments on commit 2e0971b

Please sign in to comment.