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

[chectl] workspace:inject no more finding default workspace pod #16019

Closed
3 of 23 tasks
apupier opened this issue Feb 13, 2020 · 11 comments
Closed
3 of 23 tasks

[chectl] workspace:inject no more finding default workspace pod #16019

apupier opened this issue Feb 13, 2020 · 11 comments
Assignees
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

Comments

@apupier
Copy link
Contributor

apupier commented Feb 13, 2020

Describe the bug

[apupier@linux ~]$ chectl workspace:inject -k
  ✔ Verify if namespace che exists
  ✖ Verify if the workspaces is running
    → No workspace pod is found
    Injecting configurations
 ›   Error: No workspace pod is found

Che version

  • latest 7.9.0-SNASPHOT
  • nightly
  • other: please specify

Steps to reproduce

  1. Start a workspace and wait that it has been started
  2. call chectl workspace:inject -k from host
  3. See error

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

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: please specify

Additional context

@apupier apupier added kind/bug Outline of a bug - must adhere to the bug report template. area/chectl Issues related to chectl, the CLI of Che labels Feb 13, 2020
@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 Feb 13, 2020
@tolusha
Copy link
Contributor

tolusha commented Feb 13, 2020

@apupier
In which namespace the workspace is run?
If it isn't che then could you try with -n <namespace> flag?

@apupier
Copy link
Contributor Author

apupier commented Feb 13, 2020

In which namespace the workspace is run?

it should be in che namespace as I installed with chectl server:start -p minikube

@tolusha
Copy link
Contributor

tolusha commented Feb 13, 2020

yeap, but could you double check pls?

@apupier
Copy link
Contributor Author

apupier commented Feb 13, 2020

I have 2 namespaces available:
che and che-che

in deployments, there is a "workspacexxxx" for che-che, looks alike the che is containing "only" devfile-registry-xxx, plugin-registry-xxx and a che-xxxx

@apupier
Copy link
Contributor Author

apupier commented Feb 13, 2020

[apupier@linux ~]$ chectl workspace:inject -k -n che-che
  ✔ Verify if namespace che-che exists
  ✔ Verify if the workspaces is running
  ✔ Injecting configurations

looks alike now the namespace by defautl to look at has changed and is che-che instead of che

@tolusha
Copy link
Contributor

tolusha commented Feb 13, 2020

yes, workspace goes in a different namspace.
Can we close the issue?

@apupier
Copy link
Contributor Author

apupier commented Feb 13, 2020

I think we need to keep it open and adjust the behavior of chectl workspace:inject to match the new default behavior.

It sounds like a regression

@tolusha
Copy link
Contributor

tolusha commented Feb 13, 2020

In general way chectl doesn't know where workspace is created

@tolusha
Copy link
Contributor

tolusha commented Feb 13, 2020

But I do agree the chectl might be smarter

@tolusha tolusha added severity/P2 Has a minor but important impact to the usage or development of the system. team/deploy and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Feb 13, 2020
@tolusha tolusha changed the title chectl workspace:inject no more finding workspace pod Add ability to find workspace running in non default namespace Feb 13, 2020
@apupier apupier changed the title Add ability to find workspace running in non default namespace chectl workspace:inject no more finding default workspace pod Feb 13, 2020
@apupier
Copy link
Contributor Author

apupier commented Feb 19, 2020

why the severity is only P2 although it is a regression?

@tolusha tolusha added severity/P1 Has a major impact to usage or development of the system. and removed severity/P2 Has a minor but important impact to the usage or development of the system. labels Feb 24, 2020
@tolusha tolusha added this to the Backlog - Deploy milestone Feb 24, 2020
@tolusha tolusha mentioned this issue Mar 27, 2020
43 tasks
@tolusha
Copy link
Contributor

tolusha commented Apr 14, 2020

@tolusha tolusha changed the title chectl workspace:inject no more finding default workspace pod [chectl] workspace:inject no more finding default workspace pod Apr 15, 2020
@tolusha tolusha self-assigned this Apr 15, 2020
@tolusha tolusha closed this as completed Apr 15, 2020
@tolusha tolusha modified the milestones: Backlog - Deploy, 7.12 Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

No branches or pull requests

3 participants