[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

tests/model_serving/model_server/inference_service_configuration/conftest.py

@@ -11,7 +11,8 @@
 
 
 from tests.model_serving.model_server.inference_service_configuration.constants import (
-    ISVC_ENV_VARS, updated_pull_secret
+    ISVC_ENV_VARS,
+    updated_pull_secret,
 )
 from tests.model_serving.model_server.inference_service_configuration.utils import (
     update_inference_service,
@@ -71,13 +72,14 @@ def patched_isvc_replicas(
     ):
         yield ovms_kserve_inference_service
 
+
 @pytest.fixture(scope="class")
 def model_car_raw_inference_service_with_pull_secret(
     request: pytest.FixtureRequest,
     unprivileged_client: DynamicClient,
     unprivileged_model_namespace: Namespace,
     serving_runtime_from_template: ServingRuntime,
-    pull_secrets = ["pull-secret-1"]
+    pull_secrets=["pull-secret-1"],
 ) -> Generator[InferenceService, Any, Any]:
     with create_isvc(
         client=unprivileged_client,
@@ -92,6 +94,7 @@ def model_car_raw_inference_service_with_pull_secret(
     ) as isvc:
         yield isvc
 
+
 @pytest.fixture(scope="class")
 def updated_isvc_pull_secret(
     request: pytest.FixtureRequest,
@@ -101,13 +104,7 @@ def updated_isvc_pull_secret(
     with update_inference_service(
         client=unprivileged_client,
         isvc=model_car_raw_inference_service_with_pull_secret,
-        isvc_updated_dict={
-            "spec": {
-                "predictor": {
-                    "imagePullSecrets": [{"name": updated_pull_secret}]
-                }
-            }
-        },
+        isvc_updated_dict={"spec": {"predictor": {"imagePullSecrets": [{"name": updated_pull_secret}]}}},
         wait_for_new_pods=request.param["wait-for-new-pods"],
     ):
-        yield model_car_raw_inference_service_with_pull_secret
+        yield model_car_raw_inference_service_with_pull_secret

tests/model_serving/model_server/inference_service_configuration/constants.py

@@ -12,4 +12,4 @@
 ]
 
 original_pull_secret = "pull-secret-1"
-updated_pull_secret = "updated-pull-secret"
+updated_pull_secret = "updated-pull-secret"

tests/model_serving/model_server/inference_service_configuration/test_isvc_pull_secret_updates.py

@@ -1,10 +1,8 @@
 import pytest
 
-from ocp_resources.inference_service import InferenceService
 
-from tests.model_serving.model_server.inference_service_configuration.utils import update_inference_service, verify_pull_secret
-from tests.model_serving.model_server.inference_service_configuration.constants import original_pull_secret, updated_pull_secret
-from utilities.infra import get_pods_by_isvc_label
+from tests.model_serving.model_server.inference_service_configuration.utils import verify_pull_secret
+from tests.model_serving.model_server.inference_service_configuration.constants import original_pull_secret
 from utilities.constants import ModelFormat, ModelName, RuntimeTemplates
 
 
@@ -26,13 +24,11 @@
     ],
     indirect=True,
 )
-
 class TestISVCPullSecretUpdate:
     def test_initial_pull_secret_set(self, model_car_raw_inference_service_with_pull_secret):
         """Ensure initial pull secret is correctly set in the pod"""
         verify_pull_secret(model_car_raw_inference_service_with_pull_secret, original_pull_secret, True)
 
-
     def test_update_pull_secret(self, updated_isvc_pull_secret):
         """Update the pull secret and verify it is reflected in the new pod"""
         verify_pull_secret(updated_isvc_pull_secret, original_pull_secret, True)

tests/model_serving/model_server/inference_service_configuration/utils.py

@@ -109,6 +109,7 @@ def wait_for_new_running_inference_pods(
         LOGGER.error(f"Timeout waiting for pods {oring_pods_names} to be replaced")
         raise
 
+
 def verify_pull_secret(isvc: InferenceService, pull_secret: str, secret_exists: bool) -> None:
     """
     Verify that the ImagePullSecret in the InferenceService pods match the expected values.
@@ -125,6 +126,6 @@ def verify_pull_secret(isvc: InferenceService, pull_secret: str, secret_exists:
         client=isvc.client,
         isvc=isvc,
     )[0]
-    assert (
-        pod.instance.spec.imagePullSecrets[0].name == pull_secret
-    ), f"Expected pull secret name 'pull-secret-1', got {pod.instance.spec.imagePullSecrets[0].name}"
+    assert pod.instance.spec.imagePullSecrets[0].name == pull_secret, (
+        f"Expected pull secret name 'pull-secret-1', got {pod.instance.spec.imagePullSecrets[0].name}"
+    )