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

parameter devfileRegistryPullPolicy is not handled #18847

Closed
2 of 22 tasks
benoitf opened this issue Jan 20, 2021 · 0 comments
Closed
2 of 22 tasks

parameter devfileRegistryPullPolicy is not handled #18847

benoitf opened this issue Jan 20, 2021 · 0 comments
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator good first issue Community, this issue looks easy to start with for a new contributor. Just take it. We'll help you! kind/bug Outline of a bug - must adhere to the bug report template. level/beginner A relatively easy issue that's well suited to being addressed by a pull request from the community. severity/P1 Has a major impact to usage or development of the system.
Milestone

Comments

@benoitf
Copy link
Contributor

benoitf commented Jan 20, 2021

Describe the bug

Experimenting happy path on devfile registry, I tried to change the devfile registry policy to IfNotPresent instead of default Always but it was always resulting in error:

12:53:34] ✅  Post installation checklist [failed]
[12:53:34] → Failed to download image, reason: ImagePullBackOff, message: Back-off pulling image "che-devfile-registry-image:latest".
 ›   Error: Error: Failed to download image, reason: ImagePullBackOff, message:
 ›    Back-off pulling image "che-devfile-registry-image:latest".

Che version

  • latest
  • nightly
  • other: please specify

Steps to reproduce

Use a custom resource specServer and use devfileRegistryPullPolicy = 'IfNotPresent'

We see that is related to a probably copy/paste error https://github.com/eclipse/che-operator/blob/482155b7e23a767e2906f0734727a9d23bd65256/pkg/deploy/devfile-registry/deployment.go#L24
as it's referencing PluginRegistryPullPolicy instead of DevfileRegistryPullPolicy

Expected behavior

Policy being handle as specified

Runtime

  • kubernetes (include output of kubectl version)
  • Openshift (include output of oc version)
  • minikube (include output of minikube version and kubectl version)
  • minishift (include output of minishift version and oc version)
  • docker-desktop + K8S (include output of docker version and kubectl version)
  • other: (please specify)

Screenshots

Installation method

  • chectl
    • provide a full command that was used to deploy Eclipse Che (including the output)
    • provide an output of chectl version command
  • OperatorHub
  • I don't know

Environment

  • my computer
    • Windows
    • Linux
    • macOS
  • Cloud
    • Amazon
    • Azure
    • GCE
    • other (please specify)
  • other: please specify

Eclipse Che Logs

Additional context

https://github.com/eclipse/che-operator/blob/482155b7e23a767e2906f0734727a9d23bd65256/pkg/deploy/devfile-registry/deployment.go#L24

@benoitf benoitf added kind/bug Outline of a bug - must adhere to the bug report template. level/beginner A relatively easy issue that's well suited to being addressed by a pull request from the community. good first issue Community, this issue looks easy to start with for a new contributor. Just take it. We'll help you! area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator labels Jan 20, 2021
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Jan 20, 2021
@flacatus flacatus added severity/P1 Has a major impact to usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Jan 20, 2021
@tolusha tolusha added this to the 7.27 milestone Feb 1, 2021
@tolusha tolusha mentioned this issue Feb 5, 2021
57 tasks
@tolusha tolusha closed this as completed Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator good first issue Community, this issue looks easy to start with for a new contributor. Just take it. We'll help you! kind/bug Outline of a bug - must adhere to the bug report template. level/beginner A relatively easy issue that's well suited to being addressed by a pull request from the community. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

4 participants