diff --git a/vmaas/reposcan/database/upgrade.py b/vmaas/reposcan/database/upgrade.py index ead6c9335..0fefa67ad 100755 --- a/vmaas/reposcan/database/upgrade.py +++ b/vmaas/reposcan/database/upgrade.py @@ -229,7 +229,7 @@ def _set_current_db_version(version, conn): conn.commit() @staticmethod - def _insert_log_entry(version, status, script, conn, returncode=None, stdout=None, stderr=None): + def _insert_log_entry(version, status, script, conn, *, returncode=None, stdout=None, stderr=None): with conn.cursor() as cur: cur.execute("""insert into db_upgrade_log (version, status, script, returncode, stdout, stderr) diff --git a/vmaas/reposcan/download/downloader.py b/vmaas/reposcan/download/downloader.py index 45ce58010..6fd9e5517 100644 --- a/vmaas/reposcan/download/downloader.py +++ b/vmaas/reposcan/download/downloader.py @@ -28,7 +28,7 @@ class DownloadItem: and result HTTP status code of the download operation. """ - def __init__(self, source_url: str | None = None, target_path: Path | None = None, + def __init__(self, *, source_url: str | None = None, target_path: Path | None = None, ca_cert: str | None = None, cert: str | None = None, key: str | None = None) -> None: self.source_url = source_url self.target_path = target_path diff --git a/vmaas/reposcan/redhatcsaf/modeling.py b/vmaas/reposcan/redhatcsaf/modeling.py index 32e7938b6..e66d93a86 100644 --- a/vmaas/reposcan/redhatcsaf/modeling.py +++ b/vmaas/reposcan/redhatcsaf/modeling.py @@ -182,6 +182,7 @@ class CsafProducts: def to_tuples( self, attributes: tuple[str, ...], + *, missing_only: bool = False, with_id: bool = False, with_cpe_id: bool = False, diff --git a/vmaas/reposcan/repodata/repository.py b/vmaas/reposcan/repodata/repository.py index f9132f30d..7d4c8af0a 100644 --- a/vmaas/reposcan/repodata/repository.py +++ b/vmaas/reposcan/repodata/repository.py @@ -13,7 +13,7 @@ class Repository: """ # pylint: disable=too-many-instance-attributes - def __init__(self, repo_url, content_set, basearch, releasever, cert_name=None, ca_cert=None, cert=None, key=None): + def __init__(self, repo_url, content_set, basearch, releasever, *, cert_name=None, ca_cert=None, cert=None, key=None): self.repo_url = repo_url self.repomd = None self.primary = None diff --git a/vmaas/reposcan/repodata/repository_controller.py b/vmaas/reposcan/repodata/repository_controller.py index eaa4b4fec..acaad0862 100644 --- a/vmaas/reposcan/repodata/repository_controller.py +++ b/vmaas/reposcan/repodata/repository_controller.py @@ -199,7 +199,7 @@ def add_db_repositories(self): cert_name=repo_dict["cert_name"], ca_cert=repo_dict["ca_cert"], cert=repo_dict["cert"], key=repo_dict["key"])) - def add_repository(self, repo_url, content_set, basearch, releasever, + def add_repository(self, repo_url, content_set, basearch, releasever, *, cert_name=None, ca_cert=None, cert=None, key=None): """Queue repository to import/check updates.""" repo_url = repo_url.strip() diff --git a/vmaas/webapp/app.py b/vmaas/webapp/app.py index b37b60694..5aecc1573 100755 --- a/vmaas/webapp/app.py +++ b/vmaas/webapp/app.py @@ -34,6 +34,9 @@ from vmaas.common.middlewares import ErrorHandlerMiddleware, TimingLoggingMiddleware from vmaas.common.strtobool import strtobool +# NOTE: this module is going to be removed soon +# pylint: disable=too-many-positional-arguments + # pylint: disable=too-many-lines CFG = Config() diff --git a/vmaas/webapp/updates.py b/vmaas/webapp/updates.py index 0fc21eeda..db27b36b4 100755 --- a/vmaas/webapp/updates.py +++ b/vmaas/webapp/updates.py @@ -13,6 +13,9 @@ SECURITY_ERRATA_TYPE = 'security' +# NOTE: this module is going to be removed soon +# pylint: disable=too-many-positional-arguments + def filter_non_security(errata_detail: dict, security_only: bool) -> bool: """Decide whether the errata should be filtered base on 'securiyt only' rule."""