Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

It should be possible to start Che workspace when namespace of type user is missing #1042

Closed
garagatyi opened this issue Nov 3, 2018 · 9 comments
Assignees
Milestone

Comments

@garagatyi
Copy link

garagatyi commented Nov 3, 2018

Issue problem:
On workspace provisioning step we get information about namepsace of type user. But if such namespace is missing user can't start any workspace.

@ibuziuk
Copy link
Member

ibuziuk commented Mar 26, 2019

@alexeykazakov this is the second iteration of #1296
should we consider taking it to the next sprint?

@alexeykazakov
Copy link
Member

@ibuziuk IMO yes. But @gorkem can confirm if this is what we want (to be ready to drop usernamespaces the next sprint).

@ibuziuk
Copy link
Member

ibuziuk commented Mar 26, 2019

ok waiting for confirmation before adding to the pri doc for the next sprint

@ibuziuk
Copy link
Member

ibuziuk commented Apr 2, 2019

@alexeykazakov we are NOT planning to take this issue to the next sprint. If the issue should be tackled during the next sprint please comment

@amisevsk
Copy link
Collaborator

amisevsk commented Apr 4, 2019

FWIW it looks like a relatively simple issue to fix, so if it's pressing I wouldn't be against spending half a day working on it next sprint -- this wouldn't interfere with planning much. WDYT @ibuziuk ?

If it turns out to be more complicated than it looks, we can drop it from the sprint but if it can be fixed in a few hours I think we could include it.

@amisevsk
Copy link
Collaborator

amisevsk commented Apr 4, 2019

From what I can tell, what we're doing now is getting namespace, cluster URL, and certs from the user namespace and injecting them into the workspace (presumably so that oc commands in the workspace use the user namespace. Simply replacing the user namespace with -che would only change where oc commands are run. The caveat here is that whatever we do with oc would then have to fit in the -che namespace alongside the workspace, which could be a problem.

@ibuziuk
Copy link
Member

ibuziuk commented Apr 8, 2019

@amisevsk no, I suppose we should not work on this issue until user namespace would be confirmed to be removed, once it is done we would simply do a cleanup including removal of login to OSO installer. At the moment we do not plan to add this issue to the next sprint, so no work is needed in this deirection cc: @alexeykazakov @gorkem

@ibuziuk ibuziuk added this to the 7.6.0 milestone Nov 27, 2019
@ibuziuk ibuziuk self-assigned this Dec 5, 2019
ibuziuk added a commit to ibuziuk/rh-che that referenced this issue Dec 11, 2019
ibuziuk added a commit to ibuziuk/rh-che that referenced this issue Dec 11, 2019
…owing an exception in namespace of type 'user' is missing (this might happen only if there is a bug on init-tenant or 'api/user/services' side)

Signed-off-by: Ilya Buziuk <[email protected]>
ibuziuk added a commit to ibuziuk/rh-che that referenced this issue Dec 11, 2019
…owing an exception in namespace of type 'user' is missing (this might happen only if there is a bug on init-tenant or 'api/user/services' side)

Signed-off-by: Ilya Buziuk <[email protected]>
ibuziuk added a commit to ibuziuk/rh-che that referenced this issue Dec 11, 2019
…owing an exception in namespace of type 'user' is missing (this might happen only if there is a bug on init-tenant or 'api/user/services' side)

Signed-off-by: Ilya Buziuk <[email protected]>
ibuziuk added a commit that referenced this issue Dec 13, 2019
ibuziuk added a commit that referenced this issue Dec 13, 2019
…on in namespace of type 'user' is missing (this might happen only if there is a bug on init-tenant or 'api/user/services' side)

Signed-off-by: Ilya Buziuk <[email protected]>
@ibuziuk ibuziuk modified the milestones: 7.6.0, 7.7.0 Dec 18, 2019
@ibuziuk
Copy link
Member

ibuziuk commented Dec 18, 2019

Going to close after the next production update

@ibuziuk ibuziuk modified the milestones: 7.7.0, 7.8.0 Jan 8, 2020
@ibuziuk
Copy link
Member

ibuziuk commented Jan 22, 2020

The fix is on prod. Closing

@ibuziuk ibuziuk closed this as completed Jan 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants