diff options
Diffstat (limited to 'utils/test')
| -rw-r--r-- | utils/test/cli_installer_tests.py | 42 | ||||
| -rw-r--r-- | utils/test/fixture.py | 16 | 
2 files changed, 34 insertions, 24 deletions
| diff --git a/utils/test/cli_installer_tests.py b/utils/test/cli_installer_tests.py index ea380d565..72e8521d0 100644 --- a/utils/test/cli_installer_tests.py +++ b/utils/test/cli_installer_tests.py @@ -681,9 +681,9 @@ class AttendedCliTests(OOCliFixture):          run_playbook_mock.return_value = 0          cli_input = build_input(hosts=[ -            ('10.0.0.1', True), -            ('10.0.0.2', False), -            ('10.0.0.3', False)], +            ('10.0.0.1', True, False), +            ('10.0.0.2', False, False), +            ('10.0.0.3', False, False)],                                        ssh_user='root',                                        variant_num=1,                                        confirm_facts='y') @@ -722,10 +722,10 @@ class AttendedCliTests(OOCliFixture):          run_playbook_mock.return_value = 0          cli_input = build_input(hosts=[ -            ('10.0.0.1', True), -            ('10.0.0.2', False), +            ('10.0.0.1', True, False), +            ('10.0.0.2', False, False),              ], -                                      add_nodes=[('10.0.0.3', False)], +                                      add_nodes=[('10.0.0.3', False, False)],                                        ssh_user='root',                                        variant_num=1,                                        confirm_facts='y') @@ -773,9 +773,9 @@ class AttendedCliTests(OOCliFixture):          mock_facts['10.0.0.2']['common']['version'] = "3.0.0"          cli_input = build_input(hosts=[ -            ('10.0.0.1', True), +            ('10.0.0.1', True, False),              ], -                                      add_nodes=[('10.0.0.2', False)], +                                      add_nodes=[('10.0.0.2', False, False)],                                        ssh_user='root',                                        variant_num=1,                                        schedulable_masters_ok=True, @@ -796,10 +796,10 @@ class AttendedCliTests(OOCliFixture):          run_playbook_mock.return_value = 0          cli_input = build_input(hosts=[ -            ('10.0.0.1', True), -            ('10.0.0.2', True), -            ('10.0.0.3', True), -            ('10.0.0.4', False)], +            ('10.0.0.1', True, False), +            ('10.0.0.2', True, False), +            ('10.0.0.3', True, False), +            ('10.0.0.4', False, False)],                                        ssh_user='root',                                        variant_num=1,                                        confirm_facts='y', @@ -837,9 +837,9 @@ class AttendedCliTests(OOCliFixture):          run_playbook_mock.return_value = 0          cli_input = build_input(hosts=[ -            ('10.0.0.1', True), -            ('10.0.0.2', True), -            ('10.0.0.3', True)], +            ('10.0.0.1', True, False), +            ('10.0.0.2', True, False), +            ('10.0.0.3', True, False)],                                        ssh_user='root',                                        variant_num=1,                                        confirm_facts='y', @@ -872,10 +872,10 @@ class AttendedCliTests(OOCliFixture):          run_playbook_mock.return_value = 0          cli_input = build_input(hosts=[ -                                      ('10.0.0.1', True), -                                      ('10.0.0.2', True), -                                      ('10.0.0.3', False), -                                      ('10.0.0.4', True)], +                                      ('10.0.0.1', True, False), +                                      ('10.0.0.2', True, False), +                                      ('10.0.0.3', False, False), +                                      ('10.0.0.4', True, False)],                                        ssh_user='root',                                        variant_num=1,                                        confirm_facts='y', @@ -893,7 +893,7 @@ class AttendedCliTests(OOCliFixture):          run_playbook_mock.return_value = 0          cli_input = build_input(hosts=[ -            ('10.0.0.1', True)], +            ('10.0.0.1', True, False)],                                        ssh_user='root',                                        variant_num=1,                                        confirm_facts='y') @@ -921,7 +921,7 @@ class AttendedCliTests(OOCliFixture):          run_playbook_mock.return_value = 0          cli_input = build_input(hosts=[ -            ('10.0.0.1', True)], +            ('10.0.0.1', True, False)],                                        ssh_user='root',                                        variant_num=2,                                        confirm_facts='y') diff --git a/utils/test/fixture.py b/utils/test/fixture.py index 90bd9e1ef..be759578a 100644 --- a/utils/test/fixture.py +++ b/utils/test/fixture.py @@ -138,7 +138,7 @@ class OOCliFixture(OOInstallFixture):          self.assertEquals(exp_hosts_to_run_on_len, len(hosts_to_run_on)) -#pylint: disable=too-many-arguments,too-many-branches +#pylint: disable=too-many-arguments,too-many-branches,too-many-statements  def build_input(ssh_user=None, hosts=None, variant_num=None,                  add_nodes=None, confirm_facts=None, schedulable_masters_ok=None,                  master_lb=None): @@ -163,13 +163,19 @@ def build_input(ssh_user=None, hosts=None, variant_num=None,      num_masters = 0      if hosts:          i = 0 -        for (host, is_master) in hosts: +        for (host, is_master, is_containerized) in hosts:              inputs.append(host)              if is_master:                  inputs.append('y')                  num_masters += 1              else:                  inputs.append('n') + +            if is_containerized: +                inputs.append('container') +            else: +                inputs.append('rpm') +              #inputs.append('rpm')              # We should not be prompted to add more hosts if we're currently at              # 2 masters, this is an invalid HA configuration, so this question @@ -196,8 +202,12 @@ def build_input(ssh_user=None, hosts=None, variant_num=None,              inputs.append('y')          inputs.append('1')  # Add more nodes          i = 0 -        for (host, is_master) in add_nodes: +        for (host, is_master, is_containerized) in add_nodes:              inputs.append(host) +            if is_containerized: +                inputs.append('container') +            else: +                inputs.append('rpm')              #inputs.append('rpm')              if i < len(add_nodes) - 1:                  inputs.append('y')  # Add more hosts | 
