-
Notifications
You must be signed in to change notification settings - Fork 239
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
Use Deployment objects instead of Pod to manage jenkins instances lifecycle #195
Comments
Pod gives full control of the Jenkins. I have an idea to spin up two pods at the same time and switch traffic from one to another when the old one needs to be killed. With deployment you can't do this. |
I wonder if we could leverage the deployment controller and still do the blue-green deployment |
Rolling-update allows indeed to achieve blue/green deployment.
https://kubernetes.io/docs/tasks/run-application/rolling-update-replication-controller/ |
@tomaszsek can you assign this to me ? |
hi @tomaszsek, @waveywaves et al, sorry, I am slightly late on the show, but would it not be more beneficial to use StatefulSet instead of Deployment? I.e. volumeClaimTemplates would allow stable storage... |
Any movement on this? Is there any way I can help move this along? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this issue is still affecting you, just comment with any updates and we'll keep it open. Thank you for your contributions. |
Currently the jenkins operator monitors pod configuration changes and performs restarts of pods in case of a change. Also, it monitors the existence of the pod and recreates it if it deleted.
Kubernetes provides Deployment object to perform these 2 features [1].
It would be preferable if the jenkins-operator instantiates a Deployment object and lets the kubernetes Deployment Controller manage the pod lifecycle.
[1] https://kubernetes.io/docs/concepts/workloads/controllers/deployment/
The text was updated successfully, but these errors were encountered: