aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/plugins/action
diff options
context:
space:
mode:
authorEgor Tensin <egor@tensin.name>2026-05-17 11:52:31 +0200
committerEgor Tensin <egor@tensin.name>2026-05-17 11:52:31 +0200
commit5b1e07450fec0f015de629131bc4e620e07889fa (patch)
treed3160f3a4cb6ec8279fe43364591681c88ca0b9f /plugins/action
parentworkflows: add CI workflow (diff)
downloadinfra-ansible-5b1e07450fec0f015de629131bc4e620e07889fa.tar.gz
infra-ansible-5b1e07450fec0f015de629131bc4e620e07889fa.zip
reformat Python files
Diffstat (limited to 'plugins/action')
-rwxr-xr-xplugins/action/apt_wrapper.py2
-rwxr-xr-xplugins/action/etckeeper_commit.py7
-rwxr-xr-xplugins/action/safe_git.py12
3 files changed, 15 insertions, 6 deletions
diff --git a/plugins/action/apt_wrapper.py b/plugins/action/apt_wrapper.py
index b322e10..804454b 100755
--- a/plugins/action/apt_wrapper.py
+++ b/plugins/action/apt_wrapper.py
@@ -22,7 +22,7 @@ class ActionModule(ActionBase):
after = self._parse_dpkg_log(task_vars)
- apt_result['packages'] = after[len(before):]
+ apt_result['packages'] = after[len(before) :]
return apt_result
def _parse_dpkg_log(self, task_vars, default=None):
diff --git a/plugins/action/etckeeper_commit.py b/plugins/action/etckeeper_commit.py
index d16e66f..06c7be0 100755
--- a/plugins/action/etckeeper_commit.py
+++ b/plugins/action/etckeeper_commit.py
@@ -24,11 +24,14 @@ class ActionModule(ActionBase):
if allowlist:
unexpected = [
- path for path in dirty_files
+ path
+ for path in dirty_files
if not any([pattern.fullmatch(path) for pattern in allowlist])
]
if unexpected:
- raise AnsibleActionFail(f"Unexpected modifications in repository:\n{'\n'.join(unexpected)}")
+ raise AnsibleActionFail(
+ f"Unexpected modifications in repository:\n{'\n'.join(unexpected)}"
+ )
return self._commit(msg, task_vars)
diff --git a/plugins/action/safe_git.py b/plugins/action/safe_git.py
index 7615236..8389169 100755
--- a/plugins/action/safe_git.py
+++ b/plugins/action/safe_git.py
@@ -34,12 +34,16 @@ class ActionModule(ActionBase):
# ansible.builtin.git would just remove everything, fuck that.
if not git.is_work_tree(self, dest, task_vars):
- raise AnsibleActionFail(f'Destination {dest} exists but is not a repository')
+ raise AnsibleActionFail(
+ f'Destination {dest} exists but is not a repository'
+ )
# Again, ansible.builtin.git would just remove everything, fuck that.
current_url = git.get_remote_url(self, dest, task_vars, remote=remote)
if current_url != repo:
- raise AnsibleActionFail(f"Existing remote URL '{current_url}' doesn't match '{repo}' for {dest}")
+ raise AnsibleActionFail(
+ f"Existing remote URL '{current_url}' doesn't match '{repo}' for {dest}"
+ )
# ansible.builtin.git would abort if the `force` parameter is set to
# `false`, but I don't want to depend on default values.
@@ -54,7 +58,9 @@ class ActionModule(ActionBase):
# (again, unlike the builtin module).
is_ancestor = git.is_ancestor(self, dest, remote, version, task_vars)
if not is_ancestor:
- raise AnsibleActionFail(f'HEAD in {dest} is not an ancestor of {remote}/{version}')
+ raise AnsibleActionFail(
+ f'HEAD in {dest} is not an ancestor of {remote}/{version}'
+ )
# Finally, everything looks good, delegate to the builtin module.
return self._execute_module(