Remove deprecated --enable-rollback option
This option in stack-update command was deprecated a long ago[1].
[1] 847a8fd566
Change-Id: I39d96d22f811e420a72e22f384a3ea0709324e8c
This commit is contained in:
parent
c34fc8ae5d
commit
dac63ec7d4
@ -1699,7 +1699,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_text = self.shell(
|
update_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--existing' % template_file)
|
'--existing' % template_file)
|
||||||
|
|
||||||
required = [
|
required = [
|
||||||
@ -1731,7 +1731,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_text = self.shell(
|
update_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--parameters="KeyPairName=updated_key" '
|
'--parameters="KeyPairName=updated_key" '
|
||||||
'--existing' % template_file)
|
'--existing' % template_file)
|
||||||
|
|
||||||
@ -1767,7 +1767,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_text = self.shell(
|
update_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--existing '
|
'--existing '
|
||||||
'--clear-parameter=InstanceType '
|
'--clear-parameter=InstanceType '
|
||||||
'--clear-parameter=DBUsername '
|
'--clear-parameter=DBUsername '
|
||||||
@ -1807,7 +1807,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_text = self.shell(
|
update_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--existing '
|
'--existing '
|
||||||
'--parameters="KeyPairName=updated_key" '
|
'--parameters="KeyPairName=updated_key" '
|
||||||
'--clear-parameter=InstanceType '
|
'--clear-parameter=InstanceType '
|
||||||
@ -1873,7 +1873,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_text = self.shell(
|
update_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--existing '
|
'--existing '
|
||||||
'--parameters="KeyPairName=updated_key" '
|
'--parameters="KeyPairName=updated_key" '
|
||||||
'--tags=tag1,tag2 ' % template_file)
|
'--tags=tag1,tag2 ' % template_file)
|
||||||
@ -1942,7 +1942,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_preview_text = self.shell(
|
update_preview_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--parameters="KeyPairName=updated_key" '
|
'--parameters="KeyPairName=updated_key" '
|
||||||
'--dry-run ' % template_file)
|
'--dry-run ' % template_file)
|
||||||
|
|
||||||
@ -1963,7 +1963,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_preview_text = self.shell(
|
update_preview_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--show-nested '
|
'--show-nested '
|
||||||
'--parameters="KeyPairName=updated_key" '
|
'--parameters="KeyPairName=updated_key" '
|
||||||
'--dry-run ' % template_file)
|
'--dry-run ' % template_file)
|
||||||
@ -1985,7 +1985,7 @@ class ShellTestUserPass(ShellBase):
|
|||||||
update_preview_text = self.shell(
|
update_preview_text = self.shell(
|
||||||
'stack-update teststack2/2 '
|
'stack-update teststack2/2 '
|
||||||
'--template-file=%s '
|
'--template-file=%s '
|
||||||
'--enable-rollback '
|
'--rollback on '
|
||||||
'--existing '
|
'--existing '
|
||||||
'--parameters="KeyPairName=updated_key" '
|
'--parameters="KeyPairName=updated_key" '
|
||||||
'--dry-run ' % template_file)
|
'--dry-run ' % template_file)
|
||||||
|
@ -449,12 +449,6 @@ def do_stack_show(hc, args):
|
|||||||
@utils.arg('-t', '--timeout', metavar='<TIMEOUT>',
|
@utils.arg('-t', '--timeout', metavar='<TIMEOUT>',
|
||||||
type=int,
|
type=int,
|
||||||
help=_('Stack update timeout in minutes.'))
|
help=_('Stack update timeout in minutes.'))
|
||||||
@utils.arg('-r', '--enable-rollback', default=False, action="store_true",
|
|
||||||
help=_('DEPRECATED! Use %(arg)s argument instead. '
|
|
||||||
'Enable rollback on stack update failure. '
|
|
||||||
'NOTE: default behavior is now to use the rollback value '
|
|
||||||
'of existing stack.')
|
|
||||||
% {'arg': '--rollback'})
|
|
||||||
@utils.arg('--rollback', default=None, metavar='<VALUE>',
|
@utils.arg('--rollback', default=None, metavar='<VALUE>',
|
||||||
help=_('Set rollback on update failure. '
|
help=_('Set rollback on update failure. '
|
||||||
'Values %(true)s set rollback to enabled. '
|
'Values %(true)s set rollback to enabled. '
|
||||||
@ -544,11 +538,6 @@ def do_stack_update(hc, args):
|
|||||||
raise exc.CommandError(str(ex))
|
raise exc.CommandError(str(ex))
|
||||||
else:
|
else:
|
||||||
fields['disable_rollback'] = not rollback
|
fields['disable_rollback'] = not rollback
|
||||||
# TODO(pshchelo): remove the following 'else' clause after deprecation
|
|
||||||
# period of --enable-rollback switch and assign -r shortcut to --rollback
|
|
||||||
else:
|
|
||||||
if args.enable_rollback:
|
|
||||||
fields['disable_rollback'] = False
|
|
||||||
|
|
||||||
if args.dry_run is True:
|
if args.dry_run is True:
|
||||||
if args.show_nested:
|
if args.show_nested:
|
||||||
|
@ -0,0 +1,5 @@
|
|||||||
|
---
|
||||||
|
upgrade:
|
||||||
|
- |
|
||||||
|
The ``heat stack-update`` command no longer supports
|
||||||
|
the ``--enable-rollback`` option. Use the ``--rollback`` option instead.
|
Loading…
Reference in New Issue
Block a user