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.

Sign up for GitHub

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

Jump to bottom

chectl not checking externalPlugin/DevfileRegistry:true nor plugin/devfileRegistryUrl: in cr.yaml #15601

Closed
16 tasks
gattytto opened this issue Jan 3, 2020 · 3 comments
Closed
16 tasks

chectl not checking externalPlugin/DevfileRegistry:true nor plugin/devfileRegistryUrl: in cr.yaml #15601

gattytto opened this issue Jan 3, 2020 · 3 comments
Assignees
@AndrienkoAleksandr
Labels
area/chectl Issues related to chectl, the CLI of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Milestone
7.10

Comments

@gattytto
Copy link

gattytto commented Jan 3, 2020

Describe the bug

chectl does not take account of the following properties in cr.yaml:
devfileRegistryUrl: 'http://some-url'
pluginRegistryUrl: 'http://some-url '
externalPluginRegistry: true
externalDevfileRegistry: true

so when running chectl with the argument --che-operator-cr-yaml=someCR.yaml the chectl progress indicator fails waiting for those deployments

Che version

  • latest
  • [ *] nightly
  • other: please specify

Steps to reproduce

Expected behavior

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

image

Installation method

  • [ *] chectl
  • [ *] che-operator
  • minishift-addon
  • I don't know

Environment

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

Additional context

related to #15600
@gattytto gattytto added the kind/bug Outline of a bug - must adhere to the bug report template. label Jan 3, 2020
@gattytto
Copy link
Author

gattytto commented Jan 3, 2020

image

@gattytto
Copy link
Author

gattytto commented Jan 3, 2020

in this scenario the che server works as expected even after the chectl progress reporting failure

@tolusha tolusha added area/chectl Issues related to chectl, the CLI of Che severity/P1 Has a major impact to usage or development of the system. team/deploy labels Jan 3, 2020
@tolusha tolusha added this to the Backlog - Deploy milestone Jan 3, 2020
@gattytto gattytto mentioned this issue Jan 15, 2020
Closed
@tolusha tolusha mentioned this issue Feb 19, 2020
Closed
46 tasks
@AndrienkoAleksandr AndrienkoAleksandr self-assigned this Feb 24, 2020
@AndrienkoAleksandr AndrienkoAleksandr mentioned this issue Feb 27, 2020
Merged
@tolusha tolusha closed this as completed Mar 4, 2020
@tolusha tolusha modified the milestones: Backlog - Deploy, 7.10.0 Mar 4, 2020
@tolusha
Copy link
Contributor

tolusha commented Mar 4, 2020

plugin-registry-url and devfile-registry-url are not taken into account on purpose when che-operator-cr-yaml is used. We added warning indicating that.
If it is needed to set external devfile/plugin registries it is possible to do in cr file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AndrienkoAleksandr AndrienkoAleksandr

Labels
area/chectl Issues related to chectl, the CLI of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Milestone
7.10
Development

No branches or pull requests
3 participants
@gattytto @tolusha @AndrienkoAleksandr