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:
Sean Eagan 2019-01-18 15:57:52 -06:00
parent e6f294bacb
commit 6f76f8bec7
2 changed files with 6 additions and 4 deletions

View File

@ -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', {})

View File

@ -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(