diff options
author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2018-01-18 14:56:26 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-01-18 14:56:26 -0800 |
commit | bccd1a8f97036febc0e9bf8666ec6b9a5f4ced65 (patch) | |
tree | ff13ae073a3d46ac43b01da5f51c0a5563376042 /utils/test/fixture.py | |
parent | e1e5cdcdfd094079a2362d679bc08758a89a03d3 (diff) | |
parent | 3ce40db9e6c14ec7c9503c8a9027465f7bca9c61 (diff) | |
download | openshift-bccd1a8f97036febc0e9bf8666ec6b9a5f4ced65.tar.gz openshift-bccd1a8f97036febc0e9bf8666ec6b9a5f4ced65.tar.bz2 openshift-bccd1a8f97036febc0e9bf8666ec6b9a5f4ced65.tar.xz openshift-bccd1a8f97036febc0e9bf8666ec6b9a5f4ced65.zip |
Merge pull request #6488 from vrutkovs/utils-test-flake8
Automatic merge from submit-queue.
Fix flake8 errors in utils/test
Diffstat (limited to 'utils/test/fixture.py')
-rw-r--r-- | utils/test/fixture.py | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/utils/test/fixture.py b/utils/test/fixture.py index 873ac4a27..5c0d1d2c1 100644 --- a/utils/test/fixture.py +++ b/utils/test/fixture.py @@ -43,6 +43,7 @@ deployment: node: """ + def read_yaml(config_file_path): cfg_f = open(config_file_path, 'r') config = yaml.safe_load(cfg_f.read()) @@ -105,7 +106,7 @@ class OOCliFixture(OOInstallFixture): self.assertTrue('ip' in host) self.assertTrue('public_ip' in host) - #pylint: disable=too-many-arguments + # pylint: disable=too-many-arguments def _verify_get_hosts_to_run_on(self, mock_facts, load_facts_mock, run_playbook_mock, cli_input, exp_hosts_len=None, exp_hosts_to_run_on_len=None, @@ -152,7 +153,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,too-many-statements +# 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=('', False), storage=None): @@ -190,7 +191,7 @@ def build_input(ssh_user=None, hosts=None, variant_num=None, else: inputs.append('rpm') - #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 # will not be asked, and the user must enter the next host: @@ -224,13 +225,13 @@ 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, is_containerized) in add_nodes: + for (host, _, is_containerized) in add_nodes: inputs.append(host) if is_containerized: inputs.append('container') else: inputs.append('rpm') - #inputs.append('rpm') + # inputs.append('rpm') if i < len(add_nodes) - 1: inputs.append('y') # Add more hosts else: |