-
Notifications
You must be signed in to change notification settings - Fork 900
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
fix(monitored deploy): fix the rollback config to match what orca expects #7532
fix(monitored deploy): fix the rollback config to match what orca expects #7532
Conversation
…ects Note: `Manual` type is not yet implemented in orca, hence it's not used in deck, other then the enum definition
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.
LGTM
disabled={!rollbackOnFailure} | ||
onChange={this.destroyFailedAsgChange} | ||
/> | ||
Destroy the failed server group after rollback <HelpField id="strategy.monitored.destroyFailedAsg" /> |
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.
Didn't you want to grey this out?
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.
yes... but it was too hard :)
f57c70b feat(core): alphabetize applications in project config dropdown (spinnaker#7549) 78cbe15 feat(core/jenkins): Refer to Jenkins controller instead of master (spinnaker#7531) 8c0d087 feat(core/presentation): add revalidate api to IFormInputValidation - Make all fields on IFormInputValidation non-optional b384038 feat(core/presentation): Add useDebouncedValue react hook renamed hooks files to '.hook.ts' and exported all from an index.ts 17be6af feat(kubernetes): support rolling restart operation for deployments (spinnaker#7538) 7e8cb7e fix(monitored deploy): fix the rollback config to match what orca expects (spinnaker#7532)
* chore(core): Bump version to 0.0.425 f57c70b feat(core): alphabetize applications in project config dropdown (#7549) 78cbe15 feat(core/jenkins): Refer to Jenkins controller instead of master (#7531) 8c0d087 feat(core/presentation): add revalidate api to IFormInputValidation - Make all fields on IFormInputValidation non-optional b384038 feat(core/presentation): Add useDebouncedValue react hook renamed hooks files to '.hook.ts' and exported all from an index.ts 17be6af feat(kubernetes): support rolling restart operation for deployments (#7538) 7e8cb7e fix(monitored deploy): fix the rollback config to match what orca expects (#7532) * chore(amazon): Bump version to 0.0.218 938d219 feat(provider/aws): Show load balancer warning based on settings (#7542) 828f7e7 fix(awslb): Preventing edits against orphaned load balancers (#7547) 6186e40 refactor(aws): move Resize item in the AmazonServerGroupAction dropdown into separate component
…ects (spinnaker#7532) Note: `Manual` type is not yet implemented in orca, hence it's not used in deck, other then the enum definition
* chore(core): Bump version to 0.0.425 f57c70b feat(core): alphabetize applications in project config dropdown (spinnaker#7549) 78cbe15 feat(core/jenkins): Refer to Jenkins controller instead of master (spinnaker#7531) 8c0d087 feat(core/presentation): add revalidate api to IFormInputValidation - Make all fields on IFormInputValidation non-optional b384038 feat(core/presentation): Add useDebouncedValue react hook renamed hooks files to '.hook.ts' and exported all from an index.ts 17be6af feat(kubernetes): support rolling restart operation for deployments (spinnaker#7538) 7e8cb7e fix(monitored deploy): fix the rollback config to match what orca expects (spinnaker#7532) * chore(amazon): Bump version to 0.0.218 938d219 feat(provider/aws): Show load balancer warning based on settings (spinnaker#7542) 828f7e7 fix(awslb): Preventing edits against orphaned load balancers (spinnaker#7547) 6186e40 refactor(aws): move Resize item in the AmazonServerGroupAction dropdown into separate component
Note:
Manual
type is not yet implemented in orca, hence it's not used in deck, other then the enum definition