suppress what remains

Author: jiridanek

ci/check-software-versions.py

@@ -252,7 +252,7 @@ def process_tag(tag):
         if stop_and_remove_container(container_id) != 0:
             log.error(f"Failed to stop/remove the container '{container_id}' for the '{image_ref}' tag!")
             print_delimiter()
-            return 1
+            return 1  # noqa: B012 `return` inside `finally` blocks cause exceptions to be silenced
         print_delimiter()
 
     return ret_code
@@ -295,7 +295,7 @@ def main():
     )
 
     args = parser.parse_args()
-    global prune_podman_data
+    global prune_podman_data  # noqa: PLW0603 Using the global statement to update `prune_podman_data` is discouraged
     prune_podman_data = args.prune_podman_data
 
     ret_code = 0

ci/security-scan/quay_security_analysis.py

@@ -111,7 +111,7 @@ def process_image(image, commit_id_path, release_version_n, hash_n):
     for feature in data["data"]["Layer"]["Features"]:
         if len(feature["Vulnerabilities"]) > 0:
             for vulnerability in feature["Vulnerabilities"]:
-                vulnerabilities.append(vulnerability)
+                vulnerabilities.append(vulnerability)  # noqa: PERF402 Use `list` or `list.copy` to create a copy of a list
 
     severity_levels = [entry.get("Severity", "Unknown") for entry in vulnerabilities]
     severity_counts = Counter(severity_levels)

tests/containers/conftest.py

@@ -183,7 +183,7 @@ def append[T](self, resource: T, cleanup_func: Callable[[T], None] | None = None
 
             This is somewhat similar to Go's `defer`."""
             self.resources.append((resource, cleanup_func))
-            return resource.__enter__()  # ruff: noqa: PLC2801 Unnecessary dunder call to `__enter__`
+            return resource.__enter__()  # noqa: PLC2801 Unnecessary dunder call to `__enter__`
 
         def destroy(self):
             """Runs __exit__() on the registered resources as a cleanup."""