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

Support Deployment object in Kubernetes/OpenShift recipe #11505

Closed
sleshchenko opened this issue Oct 5, 2018 · 5 comments
Closed

Support Deployment object in Kubernetes/OpenShift recipe #11505

sleshchenko opened this issue Oct 5, 2018 · 5 comments
Assignees
Labels
kind/enhancement A feature request - must adhere to the feature request template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.

Comments

@sleshchenko
Copy link
Member

Description

A common way for deploying an application on Kubernetes/OpenShift is using of Deployment/DeploymentConfig while we allow a user to specify only bare Pods.

It should not be difficult to implement supporting of Deployment in K8s/OS recipe since we convert bare Pods from recipe to Deployments objects.

As for DeploymentConfig, it should be investigated whether we can convert it to Deployment or create DeploymentConfig. There is a separate issue for implementing support of DeploymentConfig #5967

@sleshchenko sleshchenko added kind/enhancement A feature request - must adhere to the feature request template. status/open-for-dev An issue has had its specification reviewed and confirmed. Waiting for an engineer to take it. team/platform labels Oct 5, 2018
@amisevsk
Copy link
Contributor

@sleshchenko Sorry about the delay, I got distracted with other things.

I think supporting Deployment / DeploymentConfig shouldn't much trouble at all, depending on what we're expecting implementation-wise. Currently, the internal representation for workspaces is based off pods until we hit KubernetesDeployments, which wraps the Pod in a Deployment. So, if we want to support Deployment and DeploymentConfig recipes by translating into the current internal representation, that is simple. Both Deployment and DeploymentConfig (IIRC) contain pod specs somewhere.

If however we want to support specifying deployments directly (i.e. the recipe is directly linked to the deployment that is created), that might be more difficult. I would have to look into it more.

@garagatyi
Copy link

@amisevsk the idea is to grab all the information about the deployment, not just pods info that we do at the moment

@amisevsk
Copy link
Contributor

@garagatyi Right. I'm thinking mostly about how it looks internally to Che -- does the entire flow down to KubernetesDeployments also support Deployments as objects now, or is the relevant info from the Deployment pulled out and the control flow down to KubernetesDeployments remain unchanged? If it's the former I don't see a lot of trouble implementing it, if it's the latter, I'm not so sure.

@garagatyi
Copy link

It is the latter. And yes, this is not an easy task.

@ibuziuk
Copy link
Member

ibuziuk commented Jan 9, 2019

all PRs are merged to master. Closing

@ibuziuk ibuziuk closed this as completed Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Projects
None yet
Development

No branches or pull requests

4 participants