-
Notifications
You must be signed in to change notification settings - Fork 14
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
Rmv blue-green plugin for native builds #62
Conversation
Before we merge this we should add an error to the cloudFoundryDeploy step in Piper that fails the step at the beginning if it is misconfigured: i.e. if cf push (maven and npm) is used instead of cf deploy (mta). That way we have a clean error message, rather than letting the deployment fail with something cryptic from the cf cli. |
could we also announce this in the community meeting, before rolling out a breaking change in the cf cli docker image . we would need to gracefully remove this parameter : https://www.project-piper.io/steps/cloudFoundryDeploy/#deploytype and its functionality from the step and the alternative could be the rolling deployment https://docs.cloudfoundry.org/devguide/deploy-apps/rolling-deploy.html via the piper parameter https://www.project-piper.io/steps/cloudFoundryDeploy/#cfnativedeployparameters . but we would need to give time to existing users to make this switch |
The deployType parameter is still relevant for mta deployments though. Is it enough to change the documentation? We could also describe the rolling deployment as an alternative in the deployType documentation. Of course it's good to announce this in a community meeting. How much time would you propose for users to make the switch? 3 months? |
yes 3 months is a good time. along with the announcement about rolling deployment . after 3 months we should carve out this piece of code from the cf deploy step and not support the functionality , parallely also remove the plugin from the docker image if we remove the plugin today from the docker image and keep the code and the functionality in the cf deploy step, then we end up with a call to i would like to avoid this as a breaking change for native cf users. |
I also discussed the option of keeping the plugin in the image, as the plugin still works but just runt into a rate limit if one uses it too heavily, with @1084565. We still want to remove it finally, as this plugin is not maintained and will give us trouble sooner or later wrt compliance. Then the task is now to show a deprecation warning in the cloudFoundryDeploy step, if somebody has this setup. In that message we should refer to the rolling update possibility and how to configure that in the cloudFoundryDeploy step. Additionally we will also add this information into the documentation of this step. @srinikitha09 Could you do this with the ongoing task? I will block this PR for the next 3 months. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocked until end of January 2024
@o-liver any update on this? |
Thanks for the reminder. We will work on this ASAP. FYI @kaylinche we should plan this accordingly. As mentioned by @anilkeshav27 we should simultaneously remove this. |
Do we still have any issues from merging this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes are fine with me. The announcement time for the deprecation is also over.
Is the Piper code adjusted? Because Piper uses the |
Any objections to merge this PR now? The due date is already over. |
No objections from my end. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ready to merge
This plugin has not been updated since ages and moreover, it uses the deprecated cf api v2. As we want to use cf api v3, either the plugin should be updated accordingly or we should get rid of the dependency on it.