bugfix: Looking in wrong place for upgrade options
Fixes a bug where Armada Was looking for upgrade options (force, recreate_pods currently) underneath `upgrade` directly rather than `upgrade.options` where they are defined in the schema. Change-Id: Ia95129a19c87f5d59eaefccd04a7ac9e2acb0b3b
This commit is contained in:
parent
e6f294bacb
commit
6f76f8bec7
|
@ -121,8 +121,9 @@ class ChartDeploy(object):
|
|||
|
||||
upgrade = chart.get('upgrade', {})
|
||||
disable_hooks = upgrade.get('no_hooks', False)
|
||||
force = upgrade.get('force', False)
|
||||
recreate_pods = upgrade.get('recreate_pods', False)
|
||||
options = upgrade.get('options', {})
|
||||
force = options.get('force', False)
|
||||
recreate_pods = options.get('recreate_pods', False)
|
||||
|
||||
if upgrade:
|
||||
upgrade_pre = upgrade.get('pre', {})
|
||||
|
|
|
@ -437,8 +437,9 @@ class ArmadaHandlerTestCase(base.ArmadaTestCase):
|
|||
if diff:
|
||||
upgrade = chart.get('upgrade', {})
|
||||
disable_hooks = upgrade.get('no_hooks', False)
|
||||
force = upgrade.get('force', False)
|
||||
recreate_pods = upgrade.get(
|
||||
options = upgrade.get('options', {})
|
||||
force = options.get('force', False)
|
||||
recreate_pods = options.get(
|
||||
'recreate_pods', False)
|
||||
|
||||
expected_update_release_calls.append(
|
||||
|
|
Loading…
Reference in New Issue