-
Notifications
You must be signed in to change notification settings - Fork 217
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
Combine Rolling and Canary Deployments docs #537
base: master
Are you sure you want to change the base?
Combine Rolling and Canary Deployments docs #537
Conversation
cc. @pspinrad |
@@ -1,10 +1,10 @@ | |||
--- | |||
title: Configuring rolling app deployments |
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.
thought Should this file be renamed?
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.
If it's possible to forward old links to a new page, then that makes sense to me. I guess we could empty out this page and put a link in ourselves...
@@ -1,10 +1,10 @@ | |||
--- | |||
title: Configuring rolling app deployments | |||
title: Configuring app deployments | |||
owner: CAPI | |||
--- | |||
|
|||
You can use Cloud Foundry Command Line Interface (cf CLI) commands |
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.
thought @pspinrad Is it possible to get the existing links of like https://docs.cloudfoundry.org/devguide/deploy-apps/canary-deploy.html and https://docs.cloudfoundry.org/devguide/deploy-apps/rolling-deploy.html to redirect to this new combined page?
I also restructured the docs a bit to describe the different strategies before diving into the detailed API calls.