projectcontour / contour-operator

Experimental repository to explore an operator for deploying Contour
Apache License 2.0
43 stars 34 forks source link

Change the "imagePullPolicy" to allow Air-Gap install of contour operator #515

Open sbenhazez opened 2 years ago

sbenhazez commented 2 years ago

Use case : Air-Gap install of contour operator

Contour resources used for test : https://raw.githubusercontent.com/projectcontour/contour-operator/main/examples/operator/operator.yaml https://raw.githubusercontent.com/projectcontour/contour-operator/main/examples/gateway/gateway-nodeport.yaml

Images loaded in air-gap environment:

Problem for creating pod/contour-certgen-main :

Events:
  Type    Reason   Age                     From     Message
  ----    ------   ----                    ----     -------
  Normal  BackOff  4m30s (x728 over 169m)  kubelet  Back-off pulling image "ghcr.io/projectcontour/contour:main"

Image pull policy in contour object: The source of problem is that the object job.go force the policy to PullAlways :

Environment:

skriss commented 2 years ago

We should consider leaving the image pull policy as Always on the main branch, for any ghcr.io/projectcontour/contour:main images, but changing it to IfNotPresent for the release tags, via https://github.com/projectcontour/contour-operator/blob/main/hack/release/make-release-tag.sh.