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

Allow workspace creation according to factory configuration #4332

Closed
gorkem opened this issue Sep 17, 2018 · 11 comments
Closed

Allow workspace creation according to factory configuration #4332

gorkem opened this issue Sep 17, 2018 · 11 comments

Comments

@gorkem
Copy link
Collaborator

gorkem commented Sep 17, 2018

When we are creating a workspace for a repository with .factory.json in it. Create workspace according to the definiton of .factory.json.

@garagatyi
Copy link

@gorkem do you mean the creation of a workspace using OSIO flow or directly in Che?

@gorkem
Copy link
Collaborator Author

gorkem commented Oct 14, 2018

This is for OSIO flow. Right now we are always creating workspace according to passed ID but if we start putting factory.json or any other devfile that we are starting to work on, it will be ignored when opening from OSIO. I think we should prioritize devfile/factory.json over anything else.

@ibuziuk
Copy link
Collaborator

ibuziuk commented Nov 18, 2018

@gorkem is it planned to support both factory.json & dev-file [1] ?

[1] #4538

@ibuziuk
Copy link
Collaborator

ibuziuk commented Nov 28, 2018

@gorkem during today's call with @skabashnyuk and @mshaposhnik it was discussed that potentially devfile would be a replacement for factory.json / chefile and be the only portable format for workspace generation. Taken this into account I'm wondering if it really makes sense to have this issue in the osio backlog, or the only focus should be on devfile integration as part of osio flow - #4538

@gorkem
Copy link
Collaborator Author

gorkem commented Nov 28, 2018

Ultimately, all the integration from Toolchain to Che should become based on devfile meaning that the old stack_id based integration should go away. Are we tracking it somewhere? #4538 seems to be very import specific.

What is the plan to support/convert existing factory.json users?

@ibuziuk
Copy link
Collaborator

ibuziuk commented Nov 29, 2018

Ultimately, all the integration from Toolchain to Che should become based on devfile meaning that the old stack_id based integration should go away. Are we tracking it somewhere? #4538 seems to be very import specific.

There is a separate issue Workspace creation from WIT in che 7 without stackID

What is the plan to support/convert existing factory.json users?

Could you please clarify what are factory.json users in the context of openshift.io flow ? currently factory.json is not supported on osio flow at all, so there is nothing to support. The question is should we bring the support of both factory.json and devfile, or focus only on devfile integration (AFAIK, eventually it is planned that devfile would be the only portable format for workspace generation and would replace factory.json / chefile. @skabashnyuk @mshaposhnik is it the right assumption ?)

@gorkem
Copy link
Collaborator Author

gorkem commented Nov 29, 2018

Something openshift.io flow did not support in the past does not mean it does not exist.

Do we think there is a significant number of users of factory.json that we want to support on che.openshift.io? @slemeur
Of course this is assuming there is a migration path, compatibility for factory.json upstream

@slemeur
Copy link
Collaborator

slemeur commented Jan 10, 2019

I don't think we have a significant number of users of factory.json.

The work on the devfile has been progressing nicely and it should now be possible to use the devfile to create Che 7 based workspaces with the factory flow.
That could be something to experiment @skabashnyuk @mshaposhnik to validate the work currently in progress on the devfile.

@ibuziuk
Copy link
Collaborator

ibuziuk commented Jan 10, 2019

@slemeur @gorkem so, are you ok if we close this issue and focus on devfile support in openshift.io flow - #4538 ?

@slemeur
Copy link
Collaborator

slemeur commented Jan 11, 2019

Yes, fine for me.

@ibuziuk
Copy link
Collaborator

ibuziuk commented Jan 11, 2019

Closing in favour of dev file support - #4538
dev file is supposed to be the only portable format for workspace generation in future

@ibuziuk ibuziuk closed this as completed Jan 11, 2019
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