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

ImagePullPolicy ignored for CCRuntime #395

Closed
teodor-pripoae opened this issue Jul 16, 2024 · 2 comments
Closed

ImagePullPolicy ignored for CCRuntime #395

teodor-pripoae opened this issue Jul 16, 2024 · 2 comments

Comments

@teodor-pripoae
Copy link
Contributor

ImagePullPolicy is set to never in the runtime manifest but pods for pre-install use ImagePullPolicy: Always

To Reproduce

apiVersion: confidentialcontainers.org/v1beta1
kind: CcRuntime
metadata:
  name: ccruntime-sample
  namespace: confidential-containers-system
spec:
  ccNodeSelector:
    matchLabels:
      node.kubernetes.io/worker: ""
  config:
    cleanupCmd:
    - /opt/kata-artifacts/scripts/kata-deploy.sh
    - reset
    debug: false
    defaultRuntimeClassName: kata-qemu
    environmentVariables:
    - name: INSTALL_OFFICIAL_CONTAINERD
      value: "false"
    imagePullPolicy: Never
    installCmd:
    - /opt/kata-artifacts/scripts/kata-deploy.sh
    - install

Describe the results you expected
Pods having ImagePullPolicy always

Describe the results you received:

kubectl -n confidential-containers-system get pod cc-operator-pre-install-daemon-d2d4l -o yaml | grep imagePull
    imagePullPolicy: Always

Additional context
https://github.com/confidential-containers/operator/blob/main/controllers/ccruntime_controller.go#L920

teodor-pripoae added a commit to teodor-pripoae/confidential-containers-operator that referenced this issue Jul 18, 2024
teodor-pripoae added a commit to teodor-pripoae/confidential-containers-operator that referenced this issue Jul 18, 2024
teodor-pripoae added a commit to teodor-pripoae/confidential-containers-operator that referenced this issue Jul 18, 2024
bpradipt added a commit that referenced this issue Oct 17, 2024
fix: add support for imagePullPolicy for installer daemonsets (#395)
@Xynnn007
Copy link
Member

Thanks @teodor-pripoae for rasing this up. I met the same issue and now I am using kubectl edit as a workaround now, though I know the best way is to set in yaml

@mythi
Copy link
Contributor

mythi commented Nov 19, 2024

This issue is fixed by #396 but I believe it did not get into v0.10.0.

Anyway, marking this as closed.

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

No branches or pull requests

3 participants