Skip to content

Commit 1864231

Browse files
committed
CI: bump hacking
Change-Id: I5f2177b7c2edc0e35ed2e8e9bdfc4be2f262309f
1 parent 830500b commit 1864231

File tree

5 files changed

+8
-7
lines changed

5 files changed

+8
-7
lines changed

kolla/common/sources.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@
5656
'location': ('$tarballs_base/openstack/designate/'
5757
'designate-${openstack_branch}.tar.gz')},
5858
'etcd': {
59-
# NOTE(wszumski): It is suggested to upgrade one minor version at a time:
60-
# https://github.com/etcd-io/website/blob/cf046546dec9e1dcea966dc21ea38027c3290e9a/content/en/docs/v3.4/upgrades/upgrade_3_4.md#upgrade-requirements
59+
# NOTE(wszumski): Upgrade one minor version at a time:
60+
# https://etcd.io/docs/v3.4/upgrades/upgrade_3_4/
6161
'version': '3.5.16',
6262
'type': 'url',
6363
'sha256': {

kolla/hacking/checks.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def check_explicit_underscore_import(logical_line, filename):
4949
UNDERSCORE_IMPORT_FILES.append(filename)
5050
elif (translated_log.match(logical_line) or
5151
string_translation.match(logical_line)):
52-
yield(0, "D703: Found use of _() without explicit import of _!")
52+
yield (0, "D703: Found use of _() without explicit import of _!")
5353

5454

5555
@core.flake8ext

kolla/image/kolla_worker.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -566,8 +566,8 @@ def json_summary(f, **kwargs):
566566
with open(self.conf.summary_json_file, "w") as f:
567567
json_summary(f, indent=4)
568568
except OSError as e:
569-
LOG.error(f'Failed to write JSON build summary to '
570-
'{self.conf.summary_json_file}')
569+
LOG.error('Failed to write JSON build summary to '
570+
f'{self.conf.summary_json_file}')
571571
LOG.error(f'Exception caught: {e}')
572572
sys.exit(1)
573573

kolla/template/methods.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,8 @@ def handle_repos(context, reponames, mode):
128128
>>/etc/apt/sources.list.d/{repo}.sources \
129129
&& """
130130
if repo_list[repo]['arch']:
131-
commands += f"""echo 'Architectures: {repo_list[repo]['arch']}' \
131+
commands += f"""echo 'Architectures: \
132+
{repo_list[repo]['arch']}' \
132133
>>/etc/apt/sources.list.d/{repo}.sources \
133134
&& """
134135
except KeyError:

test-requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ bashate>=0.5.1 # Apache-2.0
33
coverage!=4.4,>=4.0 # Apache-2.0
44
ddt>=1.0.1 # MIT
55
docker>=3.0.0 # Apache-2.0
6-
hacking>=3.0.1,<3.1.0 # Apache-2.0
6+
hacking>=7.0.0,<=7.1.0 # Apache-2.0
77
oslotest>=3.2.0 # Apache-2.0
88
stestr>=2.2.0 # Apache-2.0
99
testtools>=2.2.0 # MIT

0 commit comments

Comments
 (0)