Rh-che: Fix Che Server to WIT Relationship

Created on 14 May 2019  路  11Comments  路  Source: redhat-developer/rh-che

Creating this issue in order to track the Che-related work required to address upstream issue (requires access): https://jira.coreos.com/browse/ODC-228.

Related auth service PR: https://github.com/fabric8-services/fabric8-auth/pull/838

Description

WIT Service is currently used by Che Server (/api/user/services). See https://github.com/redhat-developer/rh-che#interaction-of-red-hat-che-with-other-openshiftio-services. WIT will be removed in ODC-123, so we need to remove the dependency from Che.

Acceptance Criteria

The endpoint and functionality of /api/user/services is moved from WIT to Auth Service (can鈥檛 use Tenant Service because it鈥檚 not exposed).
Remove all calls from Auth to WIT.
Ensure that Che is not calling WIT directly anymore.
Note: This needs also work from the Che team. Part of this story is make sure this is happening.

kintask leveintermediate

All 11 comments

@l0rd looks we would need to take in the next sprint. Adding the item to prio - should be ~ 3 sp

@ibuziuk prod-preview has now been updated with the new auth endpoints. The Che team may now commence the migration of the WIT endpoints to auth, and test against https://auth.prod-preview.openshift.io

@sbryzak we need to first update to the latest upstream versions, once done I will start looking into this.

@sbryzak could you clarify which endpoints should we use now instead of https://github.com/redhat-developer/rh-che/blob/master/openshift/rh-che.config.yaml#L22 ? cc: @alexeykazakov

@ibuziuk it's https://auth.prod-preview.openshift.io/api/user/services for prod-preview and https://auth.openshift.io/api/user/services for prod. So, it's the same endpoint/path but different domain/service. auth instead of api (wit)

@alexeykazakov @sbryzak looks like new endpoint does not work as expected - https://github.com/redhat-developer/rh-che/pull/1443#issuecomment-499098140

@ibuziuk I've deployed an updated release of auth to prod-preview which should hopefully fix the error now. Could you please retry the endpoint again and let me know how it goes?

PR updated and ready for review - https://github.com/redhat-developer/rh-che/pull/1443

@ibuziuk I noticed the PR has been merged, is it ok if I close this issue now?

no, we have not yet promoted it t prod - should be done this week

The fix is promoted to prod. Closing.
@alexeykazakov @sbryzak we can now terminate the wit on prod-preview and observe the che.prod-preview.openshift.io behavior.

Was this page helpful?
0 / 5 - 0 ratings