qontract-reconcile 0.10.2.dev261__tar.gz → 0.10.2.dev262__tar.gz

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: qontract-reconcile
-Version: 0.10.2.dev261
+Version: 0.10.2.dev262
 Summary: Collection of tools to reconcile services with their desired state as defined in the app-interface DB.
 Project-URL: homepage, https://github.com/app-sre/qontract-reconcile
 Project-URL: repository, https://github.com/app-sre/qontract-reconcile

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/pyproject.toml

@@ -228,6 +228,13 @@ preview = true
 [tool.ruff.lint.isort]
 known-first-party = ["reconcile", "tools"]
 
+[tool.ruff.lint.pep8-naming]
+classmethod-decorators = [
+    "classmethod",
+    "pydantic.validator",
+    "pydantic.root_validator",
+]
+
 [tool.coverage.run]
 branch = true
 omit = ["*/test/*"]

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/reconcile/aws_saml_roles/integration.py

@@ -58,7 +58,7 @@ class AwsSamlRolesIntegrationParams(PydanticRunParams):
     log_cached_log_output: bool = False
 
     @validator("max_session_duration_hours")
-    def max_session_duration_range(cls, v: str | int) -> int:  # noqa: N805
+    def max_session_duration_range(cls, v: str | int) -> int:
         if 1 <= int(v) <= 12:
             return int(v)
         raise ValueError("max_session_duration_hours must be between 1 and 12 hours")
@@ -69,7 +69,7 @@ class CustomPolicy(BaseModel):
     policy: dict[str, Any]
 
     @validator("name")
-    def name_size(cls, v: str) -> str:  # noqa: N805
+    def name_size(cls, v: str) -> str:
         """Check the policy name size.
 
         See https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_iam-quotas.html
@@ -81,7 +81,7 @@ class CustomPolicy(BaseModel):
         return v
 
     @validator("policy")
-    def policy_size(cls, v: dict[str, Any]) -> dict[str, Any]:  # noqa: N805
+    def policy_size(cls, v: dict[str, Any]) -> dict[str, Any]:
         """Check the policy size.
 
         See https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_iam-quotas.html
@@ -104,7 +104,7 @@ class AwsRole(BaseModel):
     managed_policies: list[ManagedPolicy]
 
     @validator("name")
-    def name_size(cls, v: str) -> str:  # noqa: N805
+    def name_size(cls, v: str) -> str:
         """Check the role name size.
 
         See https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_iam-quotas.html
@@ -116,7 +116,7 @@ class AwsRole(BaseModel):
         return v
 
     @root_validator
-    def validate_policies(cls, values: dict[str, Any]) -> dict[str, Any]:  # noqa: N805
+    def validate_policies(cls, values: dict[str, Any]) -> dict[str, Any]:
         """Check the policies.
 
         See https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_iam-quotas.html

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/reconcile/aws_version_sync/integration.py

@@ -108,15 +108,14 @@ class ExternalResource(BaseModel):
 
     @validator("resource_engine_version", pre=True)
     def parse_resource_engine_version(
-        cls,  # noqa: N805
-        v: str | semver.VersionInfo,
+        cls, v: str | semver.VersionInfo
     ) -> semver.VersionInfo:
         if isinstance(v, semver.VersionInfo):
             return v
         return parse_semver(str(v), optional_minor_and_patch=True)
 
     @root_validator(pre=True)
-    def set_resource_engine_version_format(cls, values: dict) -> dict:  # noqa: N805
+    def set_resource_engine_version_format(cls, values: dict) -> dict:
         resource_engine_version, resource_engine_version_format = (
             str(values.get("resource_engine_version")),
             values.get("resource_engine_version_format"),

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/reconcile/ocm_labels/integration.py

@@ -63,10 +63,7 @@ class OcmLabelsIntegrationParams(PydanticRunParams):
     ignored_label_prefixes: list[str] = []
 
     @validator("managed_label_prefixes", "ignored_label_prefixes")
-    def must_end_with_dot(
-        cls,  # noqa: N805
-        v: list[str],
-    ) -> list[str]:
+    def must_end_with_dot(cls, v: list[str]) -> list[str]:
         return [prefix + "." if not prefix.endswith(".") else prefix for prefix in v]
 
 

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/reconcile/rhidp/common.py

@@ -62,8 +62,7 @@ class ClusterAuth(BaseModel):
 
     @root_validator
     def name_no_spaces(
-        cls,  # noqa: N805
-        values: MutableMapping[str, Any],
+        cls, values: MutableMapping[str, Any]
     ) -> MutableMapping[str, Any]:
         values["name"] = values["name"].replace(" ", "-")
         return values

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/reconcile/utils/glitchtip/models.py

@@ -49,8 +49,7 @@ class Team(BaseModel):
 
     @root_validator(pre=True)
     def name_xor_slug_must_be_set(
-        cls,  # noqa: N805
-        values: MutableMapping[str, Any],
+        cls, values: MutableMapping[str, Any]
     ) -> MutableMapping[str, Any]:
         assert ("name" in values or "slug" in values) and not (
             "name" in values and "slug" in values
@@ -58,10 +57,7 @@ class Team(BaseModel):
         return values
 
     @root_validator
-    def slugify(
-        cls,  # noqa: N805
-        values: MutableMapping[str, Any],
-    ) -> MutableMapping[str, Any]:
+    def slugify(cls, values: MutableMapping[str, Any]) -> MutableMapping[str, Any]:
         values["slug"] = values.get("slug") or slugify(values.get("name", ""))
         values["name"] = slugify(values.get("name", "")) or values.get("slug")
         return values
@@ -98,10 +94,7 @@ class ProjectAlertRecipient(BaseModel):
         use_enum_values = True
 
     @validator("recipient_type")
-    def recipient_type_enforce_enum_type(
-        cls,  # noqa: N805
-        v: str | RecipientType,
-    ) -> RecipientType:
+    def recipient_type_enforce_enum_type(cls, v: str | RecipientType) -> RecipientType:
         if isinstance(v, RecipientType):
             return v
         return RecipientType[v.upper()]
@@ -129,10 +122,7 @@ class ProjectAlert(BaseModel):
         allow_population_by_field_name = True
 
     @root_validator
-    def empty_name(
-        cls,  # noqa: N805
-        values: MutableMapping[str, Any],
-    ) -> MutableMapping[str, Any]:
+    def empty_name(cls, values: MutableMapping[str, Any]) -> MutableMapping[str, Any]:
         # name is an empty string if the alert was created manually because it can't be set via UI
         # use the pk instead.
         values["name"] = values.get("name") or f"alert-{values.get('pk')}"
@@ -163,10 +153,7 @@ class Project(BaseModel):
         allow_population_by_field_name = True
 
     @root_validator
-    def slugify(
-        cls,  # noqa: N805
-        values: MutableMapping[str, Any],
-    ) -> MutableMapping[str, Any]:
+    def slugify(cls, values: MutableMapping[str, Any]) -> MutableMapping[str, Any]:
         values["slug"] = values.get("slug") or slugify(values["name"])
         return values
 
@@ -207,10 +194,7 @@ class Organization(BaseModel):
     users: list[User] = []
 
     @root_validator
-    def slugify(
-        cls,  # noqa: N805
-        values: MutableMapping[str, Any],
-    ) -> MutableMapping[str, Any]:
+    def slugify(cls, values: MutableMapping[str, Any]) -> MutableMapping[str, Any]:
         values["slug"] = values.get("slug") or slugify(values["name"])
         return values
 

{qontract_reconcile-0.10.2.dev261 → qontract_reconcile-0.10.2.dev262}/reconcile/utils/mr/user_maintenance.py

@@ -24,7 +24,7 @@ class PathSpec(BaseModel):
     path: str
 
     @validator("path")
-    def prepend_data_to_path(cls, v):  # noqa: N805
+    def prepend_data_to_path(cls, v):
         return "data" + v