-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Comments
@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 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. |
@amisevsk the idea is to grab all the information about the deployment, not just pods info that we do at the moment |
@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. |
It is the latter. And yes, this is not an easy task. |
all PRs are merged to master. Closing |
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 toDeployment
or createDeploymentConfig
. There is a separate issue for implementing support ofDeploymentConfig
#5967The text was updated successfully, but these errors were encountered: