-
Notifications
You must be signed in to change notification settings - Fork 34
feat: flag to manage sequenced rollout #201 #202
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
Conversation
|
Still unclear how this field will fix the issue? I called the Mgmt API with that property set to |
|
There's an error on the query param for the flag: |
|
Just testing this out now, the use of sequencedRollout true or false does not prevent the deployment to another env in the same envGroup. The use of generateDeployChangeReport will indicate: "New deployment will not receive any traffic for basepath /notarget, as this basepath is already claimed by proxy notarget in environment test." It's up to the user in the UI or the tool invoking the APIs to choose to continue with deployment or not. Even in the UI I can ignore the generateDeployChangeReport and deploy to another env in the same envGroup. |
|
Why is |
Probably for backward compatibility? |
I think that behavior is a bug. |
kurtkanaskie
left a comment
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.
One minor editorial on L56
undeployment will proceed...
No description provided.