Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

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

Open
sbenhazez opened this issue May 24, 2022 · 1 comment
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug.

Comments

@sbenhazez
Copy link

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:

  • ghcr.io/projectcontour/contour-operator:main
  • ghcr.io/projectcontour/contour:main
  • docker.io/envoyproxy/envoy:v1.22.0
  • gcr.io/kubebuilder/kube-rbac-proxy:v0.8.0

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 :
- daemonset.go : ImagePullPolicy: corev1.PullIfNotPresent,
- deployment.go : ImagePullPolicy: corev1.PullIfNotPresent,
- job.go: ImagePullPolicy : corev1.PullAlways

Environment:

  • Software version:
  • Kubernetes version: (use kubectl version): v1.23.6
  • Kubernetes installer & version: K3s (v1.23.6+k3s1 )
  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release): Debian 10
@sbenhazez sbenhazez added the kind/bug Categorizes issue or PR as related to a bug. label May 24, 2022
@skriss
Copy link
Member

skriss commented May 24, 2022

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.

@skriss skriss added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label May 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

2 participants