Fix flake8 failures on tripleo-ci check jobs
This patch fixes flake8 failures which are hitting tripleo-ci jobs. Change-Id: I9af17b5bc888e1679adb49bff383ee73c6426ad3 Closes-Bug: #2004557
This commit is contained in:
parent
74c54178bc
commit
062c151332
|
@ -418,7 +418,6 @@ export STANDALONE_TARGET_HASH="{standalone_target_hash}"
|
|||
|
||||
|
||||
if __name__ == '__main__':
|
||||
|
||||
default_log_file = '{}.log'.format(os.path.basename(__file__))
|
||||
default_output_file = '{}.out'.format(os.path.basename(__file__))
|
||||
|
||||
|
|
|
@ -6,7 +6,6 @@ import pytest
|
|||
|
||||
@pytest.fixture
|
||||
def hash_mock_setup():
|
||||
|
||||
# We need this variables to be arrays to emulate a reference in python
|
||||
# https://stackoverflow.com/questions/12116127/python-yield-generator-variable-scope
|
||||
# For python3 we have special keyword nonlocal so we can refer the variable
|
||||
|
@ -133,7 +132,6 @@ def hash_mock_setup():
|
|||
def test_overcloud_upgrade_is_n_minus_one_to_n(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
hash_mock_setup(
|
||||
hash_mock,
|
||||
{
|
||||
|
@ -261,7 +259,6 @@ def test_overcloud_upgrade_is_n_minus_one_to_n(
|
|||
def test_period_overcloud_upgrade_is_n_minus_one_to_n(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
hash_mock_setup(
|
||||
hash_mock,
|
||||
{
|
||||
|
@ -459,7 +456,6 @@ def test_undercloud_upgrade_is_n_minus_one_to_n(
|
|||
def test_period_undercloud_upgrade_is_n_minus_one_to_n(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
expected_release = expected_releases['undercloud_install_release']
|
||||
|
||||
hash_mock_setup(
|
||||
|
@ -512,7 +508,6 @@ def test_period_undercloud_upgrade_is_n_minus_one_to_n(
|
|||
def test_standalone_upgrade_is_n_minus_one_to_n(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
expected_release = expected_releases['standalone_deploy_release']
|
||||
|
||||
hash_mock_setup(
|
||||
|
@ -583,7 +578,6 @@ def test_standalone_upgrade_is_n_minus_one_to_n(
|
|||
def test_period_standalone_upgrade_is_n_minus_one_to_n(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
expected_release = expected_releases['standalone_deploy_release']
|
||||
|
||||
hash_mock_setup(
|
||||
|
@ -712,7 +706,6 @@ def test_period_standalone_upgrade_is_n_minus_one_to_n(
|
|||
def test_overcloud_update_target_is_hash(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
expected_release = expected_releases['overcloud_deploy_release']
|
||||
|
||||
hash_mock_setup(
|
||||
|
@ -839,7 +832,6 @@ def test_overcloud_update_target_is_hash(
|
|||
def test_period_overcloud_update_target_is_hash(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
expected_release = expected_releases['overcloud_deploy_release']
|
||||
|
||||
hash_mock_setup(
|
||||
|
@ -1087,7 +1079,6 @@ def test_noop_target_is_the_same(
|
|||
def test_periodic_noop_target_is_the_same(
|
||||
hash_mock, hash_mock_setup, stable_release, expected_releases
|
||||
):
|
||||
|
||||
hash_mock_setup(
|
||||
hash_mock,
|
||||
{
|
||||
|
|
|
@ -97,7 +97,6 @@ def close_bug(bug_tasks):
|
|||
|
||||
|
||||
def main():
|
||||
|
||||
parser = argparse.ArgumentParser(
|
||||
description="Print launchpad bugs as influxdb lines"
|
||||
)
|
||||
|
|
|
@ -37,7 +37,6 @@ class LaunchpadReport(object):
|
|||
bug_statuses_open = ['Confirmed', 'Triaged', 'In Progress', 'Fix Committed']
|
||||
bug_statuses_closed = ['Fix Released']
|
||||
for label, config_string in self.bugs.items():
|
||||
|
||||
c = config_string.split(',')
|
||||
project = launchpad.projects[c[0]]
|
||||
filter_re = c[1]
|
||||
|
@ -60,6 +59,5 @@ class LaunchpadReport(object):
|
|||
importance='Critical',
|
||||
tags='alert',
|
||||
):
|
||||
|
||||
bugs_with_alerts_closed[task.bug.id] = task.bug
|
||||
return bugs_with_alerts_open, bugs_with_alerts_closed
|
||||
|
|
|
@ -42,7 +42,6 @@ class StatusReport(object):
|
|||
"""
|
||||
|
||||
def __init__(self, config):
|
||||
|
||||
self.config = config
|
||||
self.brief_status = {}
|
||||
self.detailed_status = {}
|
||||
|
|
Loading…
Reference in New Issue