crossplane-function-sdk-python 0.13.0__tar.gz → 0.14.0__tar.gz

{crossplane_function_sdk_python-0.13.0 → crossplane_function_sdk_python-0.14.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: crossplane-function-sdk-python
-Version: 0.13.0
+Version: 0.14.0
 Summary: The Python SDK for Crossplane composition functions
 Project-URL: Documentation, https://github.com/crossplane/function-sdk-python#readme
 Project-URL: Issues, https://github.com/crossplane/function-sdk-python/issues

{crossplane_function_sdk_python-0.13.0 → crossplane_function_sdk_python-0.14.0}/crossplane/function/__version__.py

@@ -15,4 +15,4 @@
 """The version of function-sdk-python."""
 
 # This is set at build time, using "hatch version"
-__version__ = "0.13.0"
+__version__ = "0.14.0"

{crossplane_function_sdk_python-0.13.0 → crossplane_function_sdk_python-0.14.0}/crossplane/function/response.py

@@ -163,13 +163,16 @@ def require_resources(  # noqa: PLR0913
         namespace: The namespace to search in (optional).
 
     Raises:
-        ValueError: If both match_name and match_labels are provided, or neither.
+        ValueError: If both match_name and match_labels are provided.
 
     This tells Crossplane to fetch the specified resources and include them
     in the next call to the function in req.required_resources[name].
+
+    If neither match_name nor match_labels is provided, all resources of the
+    given api_version and kind are matched.
     """
-    if (match_name is None) == (match_labels is None):
-        msg = "Exactly one of match_name or match_labels must be provided"
+    if match_name is not None and match_labels is not None:
+        msg = "match_name and match_labels are mutually exclusive"
         raise ValueError(msg)
 
     selector = fnv1.ResourceSelector(

{crossplane_function_sdk_python-0.13.0 → crossplane_function_sdk_python-0.14.0}/crossplane/function/runtime.py

@@ -70,7 +70,7 @@ def load_credentials(tls_certs_dir: str) -> grpc.ServerCredentials:
 
 
 def serve(
-    function: grpcv1.FunctionRunnerService,
+    function: grpcv1.FunctionRunnerServiceServicer,
     address: str,
     *,
     creds: grpc.ServerCredentials,
@@ -134,20 +134,20 @@ def serve(
         loop.close()
 
 
-class BetaFunctionRunner(grpcv1beta1.FunctionRunnerService):
+class BetaFunctionRunner(grpcv1beta1.FunctionRunnerServiceServicer):
     """A BetaFunctionRunner handles beta gRPC RunFunctionRequests.
 
-    It handles requests by passing them to a wrapped v1.FunctionRunnerService.
+    It handles requests by passing them to a wrapped v1.FunctionRunnerServiceServicer.
     Incoming v1beta1 requests are converted to v1 by round-tripping them through
     serialization. Outgoing requests are converted from v1 to v1beta1 the same
     way.
     """
 
-    def __init__(self, wrapped: grpcv1.FunctionRunnerService):
+    def __init__(self, wrapped: grpcv1.FunctionRunnerServiceServicer):
         """Create a new BetaFunctionRunner."""
         self.wrapped = wrapped
 
-    async def RunFunction(  # noqa: N802  # gRPC requires this name.
+    async def RunFunction(  # noqa: N802  # gRPC requires this name. # pyright: ignore[reportIncompatibleMethodOverride]
         self, req: fnv1beta1.RunFunctionRequest, context: grpc.aio.ServicerContext
     ) -> fnv1beta1.RunFunctionResponse:
         """Run the underlying function."""

{crossplane_function_sdk_python-0.13.0 → crossplane_function_sdk_python-0.14.0}/tests/test_response.py

@@ -233,6 +233,21 @@ class TestResponse(unittest.TestCase):
                     match_name="worker-1",
                 ),
             ),
+            TestCase(
+                reason="Should match all resources of a kind with no match field.",
+                rsp=fnv1.RunFunctionResponse(),
+                name="all-pods",
+                api_version="v1",
+                kind="Pod",
+                match_name=None,
+                match_labels=None,
+                namespace="default",
+                want_selector=fnv1.ResourceSelector(
+                    api_version="v1",
+                    kind="Pod",
+                    namespace="default",
+                ),
+            ),
         ]
 
         for case in cases:
@@ -270,17 +285,6 @@ class TestResponse(unittest.TestCase):
                 match_labels={"app": "test"},
             )
 
-        # Should raise ValueError if neither match_name nor match_labels are provided
-        with self.assertRaises(ValueError):
-            response.require_resources(
-                rsp,
-                "test",
-                "v1",
-                "Pod",
-                match_name=None,
-                match_labels=None,
-            )
-
     def test_require_schema(self) -> None:
         @dataclasses.dataclass
         class TestCase: