openstack/kolla
Code Issues Proposed changes

CI: bump hacking

Browse Source 
Change-Id: I5f2177b7c2edc0e35ed2e8e9bdfc4be2f262309f
This commit is contained in:
Michal Nasiadka 2024-10-07 12:27:46 +02:00
parent 830500bb55
commit 1864231693
5 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
kolla/common/sources.py
View File

@ -56,8 +56,8 @@ SOURCES = {
'location': ('$tarballs_base/openstack/designate/' 'location': ('$tarballs_base/openstack/designate/'
'designate-${openstack_branch}.tar.gz')}, 'designate-${openstack_branch}.tar.gz')},
'etcd': { 'etcd': {
# NOTE(wszumski): It is suggested to upgrade one minor version at a time: # NOTE(wszumski): Upgrade one minor version at a time:
# https://github.com/etcd-io/website/blob/cf046546dec9e1dcea966dc21ea38027c3290e9a/content/en/docs/v3.4/upgrades/upgrade_3_4.md#upgrade-requirements # https://etcd.io/docs/v3.4/upgrades/upgrade_3_4/
'version': '3.5.16', 'version': '3.5.16',
'type': 'url', 'type': 'url',
'sha256': { 'sha256': {

2
kolla/hacking/checks.py
View File

@ -49,7 +49,7 @@ def check_explicit_underscore_import(logical_line, filename):
UNDERSCORE_IMPORT_FILES.append(filename) UNDERSCORE_IMPORT_FILES.append(filename)
elif (translated_log.match(logical_line) or elif (translated_log.match(logical_line) or
string_translation.match(logical_line)): string_translation.match(logical_line)):
yield(0, "D703: Found use of _() without explicit import of _!") yield (0, "D703: Found use of _() without explicit import of _!")
@core.flake8ext @core.flake8ext

4
kolla/image/kolla_worker.py
View File

@ -566,8 +566,8 @@ class KollaWorker(object):
with open(self.conf.summary_json_file, "w") as f: with open(self.conf.summary_json_file, "w") as f:
json_summary(f, indent=4) json_summary(f, indent=4)
except OSError as e: except OSError as e:
LOG.error(f'Failed to write JSON build summary to ' LOG.error('Failed to write JSON build summary to '
'{self.conf.summary_json_file}') f'{self.conf.summary_json_file}')
LOG.error(f'Exception caught: {e}') LOG.error(f'Exception caught: {e}')
sys.exit(1) sys.exit(1)

3
kolla/template/methods.py
View File

@ -128,7 +128,8 @@ def handle_repos(context, reponames, mode):
>>/etc/apt/sources.list.d/{repo}.sources \ >>/etc/apt/sources.list.d/{repo}.sources \
&& """ && """
if repo_list[repo]['arch']: if repo_list[repo]['arch']:
commands += f"""echo 'Architectures: {repo_list[repo]['arch']}' \ commands += f"""echo 'Architectures: \
{repo_list[repo]['arch']}' \
>>/etc/apt/sources.list.d/{repo}.sources \ >>/etc/apt/sources.list.d/{repo}.sources \
&& """ && """
except KeyError: except KeyError:

2
test-requirements.txt
View File

@ -3,7 +3,7 @@ bashate>=0.5.1 # Apache-2.0
coverage!=4.4,>=4.0 # Apache-2.0 coverage!=4.4,>=4.0 # Apache-2.0
ddt>=1.0.1 # MIT ddt>=1.0.1 # MIT
docker>=3.0.0 # Apache-2.0 docker>=3.0.0 # Apache-2.0
hacking>=3.0.1,<3.1.0 # Apache-2.0 hacking>=7.0.0,<=7.1.0 # Apache-2.0
oslotest>=3.2.0 # Apache-2.0 oslotest>=3.2.0 # Apache-2.0
stestr>=2.2.0 # Apache-2.0 stestr>=2.2.0 # Apache-2.0
testtools>=2.2.0 # MIT testtools>=2.2.0 # MIT