Skip to content
This repository has been archived by the owner on Jul 23, 2020. It is now read-only.

Che multi-cluster support #1683

Closed
2 tasks done
alexeykazakov opened this issue Dec 15, 2017 · 18 comments
Closed
2 tasks done

Che multi-cluster support #1683

alexeykazakov opened this issue Dec 15, 2017 · 18 comments

Comments

@alexeykazakov
Copy link
Member

alexeykazakov commented Dec 15, 2017

As part of our multi-cluster support we should start using OSO-Proxy for all communications with OSO instead of calling OSO API directly from Che.

  • Change OSO api URL to OSO-Proxy URL and use OSIO user token instead of OSO token
  • Remove OSO token lookup.

Part of #948

@l0rd
Copy link
Collaborator

l0rd commented Dec 18, 2017

@ibuziuk will contact @alexeykazakov and @aslakknutsen for updates on che-starter and che-server side

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 18, 2017

created a separate issue for che-starter - redhat-developer/che-starter#254

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 18, 2017

@l0rd one thing I want to clarify in terms of multi-cluster support - is it planned to have 1 multi-tenant che per cluster or 1 multi-tenant che for all clusters ?

@l0rd
Copy link
Collaborator

l0rd commented Dec 19, 2017

@alexeykazakov @ibuziuk I had a discussion about that a few weeks earlier with @kbsingh and I thought it was already acked that we would have multiple Che servers, one for every cluster. I still believe that's the best configuration but I don't have a firm position so let me provide the pros and cons of multiple Che servers and we can discuss it here:

Pros

  • It scales better. Even if we think Che server is already able to serve hundreds of concurrent users it may be more thoughtful to prevent problems.
  • If we change our mind it should be easier to move from multiple Che server to a single Che server than the contrary

Cons

  • Users on different cluster won't be able to share their Che workspaces (but that's a feature that we currently don't have on OSIO)
  • It may be more complicated to manage multiple Che server than to manage a single one

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 19, 2017

I also do not have strong opinion on this - both approaches have its for and against. It sounds like workspace sharing feature is the only thing that affects UX and by choosing 1 mt che-server per cluster workspace sharing will be available only for users on the same cluster. I believe this requires some input from PM side. cc: @slemeur @bmicklea

@bmicklea
Copy link
Collaborator

It would be ideal to have some kind of Che server federator, but since OpenShift itself lacks federation it's probably not the end of the world. In practice most development teams are small enough that they can be handled by a single cluster.

If I did need to have collaboration with someone from another cluster then I can simply invite them to create an account on my cluster. It's hacky but it would work, right?

@ibuziuk
Copy link
Collaborator

ibuziuk commented Dec 19, 2017

If I did need to have collaboration with someone from another cluster then I can simply invite them to create an account on my cluster. It's hacky but it would work, right?
Not sure about that. As I understand multi-cluster is just impl. detail and user is not in charge of picking cluster while account creation.

@alexeykazakov
Copy link
Member Author

It scales better.

One che server per cluster IMO scales worse. Maybe it's better performance wise but it makes any manipulations with clusters (new clusters, updates in existing ones) more complicated.

If talking about performance then shouldn't we move to support of multiple che-server replicas instead?

@l0rd
Copy link
Collaborator

l0rd commented Dec 20, 2017

One che server per cluster IMO scales worse. Maybe it's better performance wise but it makes any manipulations with clusters (new clusters, updates in existing ones) more complicated.

Agree. We need to decide between making something slightly more complicated vs having a little performance risk.

If talking about performance then shouldn't we move to support of multiple che-server replicas instead?

We need to solve this before eclipse-che/che#7662 but yes I agree with you.

@l0rd
Copy link
Collaborator

l0rd commented Dec 20, 2017

After discussion with @ibuziuk we agreed that the best approach will be to have a single multi-cluster Che server. And to raise priority for eclipse-che/che#7662 to beeing able to scale when needed.

@ibuziuk
Copy link
Collaborator

ibuziuk commented Jan 9, 2018

@alexeykazakov @aslakknutsen does OSO-proxy already available on prod-preview ?

@alexeykazakov
Copy link
Member Author

Not yet. But should be soon.

@maxandersen
Copy link
Collaborator

marking as crucial dependency for #948 as without it we can't complete multicluster support.

@ibuziuk
Copy link
Collaborator

ibuziuk commented Jan 22, 2018

@maxandersen FYI, the fix on che / che-starter side is ready we just do not want to merge it till OSO proxy would be available on prod - redhat-developer/rh-che#514 (comment)

@ibuziuk
Copy link
Collaborator

ibuziuk commented Jan 24, 2018

@alexeykazakov @aslakknutsen che with oso-proxy is deployed on prod-preview and quick start scenario seems to work just fine \o/

@qodfathr
Copy link
Collaborator

Removing SEV2 because it's not a bug. crucial-dep now relates the importance of this issue.

@ibuziuk
Copy link
Collaborator

ibuziuk commented Jan 29, 2018

Che OSO proxy support is now on prod. Once it will be confirmed that multi-cluster is working with che as expected the issue can be closed

@aslakknutsen
Copy link
Collaborator

Verified in preview and prod on new cluster.

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

8 participants