ansible-collections/community.docker
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.docker.git synced 2025-12-17 04:18:42 +00:00
Code Packages Projects Releases Wiki Activity

Address possibly-used-before-assignment.

Browse Source
This commit is contained in:
Felix Fontein 2025-10-07 22:06:24 +02:00
parent 776f95f295
commit 25f89d3663
5 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.pylintrc
View File

@ -402,7 +402,6 @@ disable=raw-checker-failed,
no-member,
no-name-in-module,
not-an-iterable,
possibly-used-before-assignment,
protected-access,
raise-missing-from,
redefined-outer-name, # needed for test fixtures

2
plugins/connection/docker.py
View File

@ -425,11 +425,13 @@ class Connection(ConnectionBase):
+ to_native(become_output)
)
chunk = None
for key, event in events:
if key.fileobj == p.stdout:
chunk = p.stdout.read()
elif key.fileobj == p.stderr:
chunk = p.stderr.read()
# TODO: avoid chunk being set multiple times!
if not chunk:
stdout, stderr = p.communicate()

2
plugins/connection/nsenter.py
View File

@ -177,11 +177,13 @@ class Connection(ConnectionBase):
+ to_native(become_output)
)
chunk = None
for key, event in events:
if key.fileobj == p.stdout:
chunk = p.stdout.read()
elif key.fileobj == p.stderr:
chunk = p.stderr.read()
# TODO: avoid chunk being set multiple times!
if not chunk:
stdout, stderr = p.communicate()

2
plugins/module_utils/_module_container/base.py
View File

@ -859,7 +859,7 @@ def _preprocess_ports(module, values):
else:
port_binds = len(container_ports) * [(ipaddr,)]
else:
module.fail_json(
return module.fail_json(
msg=f'Invalid port description "{port}" - expected 1 to 3 colon-separated parts, but got {p_len}. '
"Maybe you forgot to use square brackets ([...]) around an IPv6 address?"
)

2
plugins/modules/docker_compose_v2.py
View File

@ -499,6 +499,8 @@ class ServicesManager(BaseComposeManager):
result = self.cmd_restart()
elif self.state == "absent":
result = self.cmd_down()
else:
raise AssertionError("Unexpected state")
result["containers"] = self.list_containers()
result["images"] = self.list_images()