[patch] remove redundant Raises sections from docstrings

Author: terc1997

src/mas/devops/ocp.py

@@ -86,9 +86,6 @@ def getClusterVersion(dynClient: DynamicClient) -> str:
 
     Returns:
         str: The cluster version string (e.g., "4.12.0"), or None if not found
-
-    Raises:
-        NotFoundError: If the ClusterVersion resource cannot be retrieved
     """
     clusterVersionAPI = dynClient.resources.get(api_version="config.openshift.io/v1", kind="ClusterVersion")
 
@@ -132,8 +129,6 @@ def getNamespace(dynClient: DynamicClient, namespace: str) -> dict:
     Returns:
         dict: The namespace resource as a dictionary, or an empty dict if not found
 
-    Raises:
-        NotFoundError: If the namespace does not exist
     """
     namespaceAPI = dynClient.resources.get(api_version="v1", kind="Namespace")
 
@@ -143,7 +138,6 @@ def getNamespace(dynClient: DynamicClient, namespace: str) -> dict:
         return ns
     except NotFoundError:
         logger.debug(f"Namespace {namespace} does not exist")
-
     return {}
 
 
@@ -161,9 +155,6 @@ def createNamespace(dynClient: DynamicClient, namespace: str, kyvernoLabel: str
 
     Returns:
         bool: Always returns True
-
-    Raises:
-        NotFoundError: If the namespace resource cannot be accessed
     """
     namespaceAPI = dynClient.resources.get(api_version="v1", kind="Namespace")
     try:
@@ -205,9 +196,6 @@ def deleteNamespace(dynClient: DynamicClient, namespace: str) -> bool:
 
     Returns:
         bool: Always returns True
-
-    Raises:
-        NotFoundError: If the namespace does not exist (caught and logged)
     """
     namespaceAPI = dynClient.resources.get(api_version="v1", kind="Namespace")
     try:
@@ -230,9 +218,6 @@ def waitForCRD(dynClient: DynamicClient, crdName: str) -> bool:
 
     Returns:
         bool: True if the CRD becomes established, False if timeout is reached
-
-    Raises:
-        NotFoundError: If the CRD is not found (caught and retried)
     """
     crdAPI = dynClient.resources.get(api_version="apiextensions.k8s.io/v1", kind="CustomResourceDefinition")
     maxRetries = 100
@@ -275,9 +260,6 @@ def waitForDeployment(dynClient: DynamicClient, namespace: str, deploymentName:
 
     Returns:
         bool: True if the deployment becomes ready, False if timeout is reached
-
-    Raises:
-        NotFoundError: If the deployment is not found (caught and retried)
     """
     deploymentAPI = dynClient.resources.get(api_version="apps/v1", kind="Deployment")
     maxRetries = 100
@@ -310,9 +292,6 @@ def getConsoleURL(dynClient: DynamicClient) -> str:
 
     Returns:
         str: The HTTPS URL of the OpenShift console (e.g., "https://console-openshift-console.apps.cluster.example.com")
-
-    Raises:
-        NotFoundError: If the console route is not found
     """
     routesAPI = dynClient.resources.get(api_version="route.openshift.io/v1", kind="Route")
     consoleRoute = routesAPI.get(name="console", namespace="openshift-console")
@@ -328,9 +307,6 @@ def getNodes(dynClient: DynamicClient) -> dict:
 
     Returns:
         list: List of node resources as dictionaries
-
-    Raises:
-        NotFoundError: If nodes cannot be retrieved
     """
     nodesAPI = dynClient.resources.get(api_version="v1", kind="Node")
     nodes = nodesAPI.get().to_dict()['items']
@@ -347,9 +323,6 @@ def getStorageClass(dynClient: DynamicClient, name: str) -> dict | None:
 
     Returns:
         StorageClass: The StorageClass resource, or None if not found
-
-    Raises:
-        NotFoundError: If the StorageClass does not exist (caught and returns None)
     """
     try:
         storageClassAPI = dynClient.resources.get(api_version="storage.k8s.io/v1", kind="StorageClass")
@@ -368,9 +341,6 @@ def getStorageClasses(dynClient: DynamicClient) -> list:
 
     Returns:
         list: List of StorageClass resources
-
-    Raises:
-        NotFoundError: If StorageClasses cannot be retrieved
     """
     storageClassAPI = dynClient.resources.get(api_version="storage.k8s.io/v1", kind="StorageClass")
     storageClasses = storageClassAPI.get().items
@@ -385,17 +355,14 @@ def getClusterIssuers(dynClient: DynamicClient) -> list:
         dynClient (DynamicClient): OpenShift Dynamic Client
 
     Returns:
-        list: List of ClusterIssuers resources
-
-    Raises:
-        NotFoundError: If ClusterIssuers cannot be retrieved
+        list: List of ClusterIssuers resources or an empty list if no cluster issuers
     """
     clusterIssuerAPI = dynClient.resources.get(api_version="cert-manager.io/v1", kind="ClusterIssuer")
     clusterIssuers = clusterIssuerAPI.get().items
     return clusterIssuers
 
 
-def getClusterIssuer(dynClient: DynamicClient, name: str) -> ResourceInstance:
+def getClusterIssuer(dynClient: DynamicClient, name: str) -> ResourceInstance | None:
     """
     Get a specific ClusterIssuer by name.
 
@@ -405,9 +372,6 @@ def getClusterIssuer(dynClient: DynamicClient, name: str) -> ResourceInstance:
 
     Returns:
         ClusterIssuer: The ClusterIssuer resource, or None if not found
-
-    Raises:
-        NotFoundError: If the ClusterIssuer does not exist (caught and returns None)
     """
     try:
         clusterIssuerAPI = dynClient.resources.get(api_version="cert-manager.io/v1", kind="ClusterIssuer")