Keep extra_resources support at the proto level

We still add required_resources though, as the new (preferred) way to do
the same thing.

Signed-off-by: Nic Cope <[email protected]>

Author: negz

crossplane/function/proto/v1/run_function.proto

@@ -27,7 +27,7 @@ package apiextensions.fn.proto.v1;
 import "google/protobuf/duration.proto";
 import "google/protobuf/struct.proto";
 
-option go_package = "github.com/crossplane/crossplane/proto/fn/v1";
+option go_package = "github.com/crossplane/crossplane/v2/proto/fn/v1";
 
 // A FunctionRunnerService is a function.
 service FunctionRunnerService {
@@ -69,15 +69,27 @@ message RunFunctionRequest {
 
   // Optional resources that the function specified in its requirements. Note
   // that resources is a map to Resources, plural. The map key corresponds to
-  // the key in a RunFunctionResponse's requirements.resources field. If a
-  // function requested required resources that did not exist, Crossplane sets
+  // the key in a RunFunctionResponse's requirements.extra_resources field. If a
+  // function requested extra resources that did not exist, Crossplane sets
   // the map key to an empty Resources message to indicate that it attempted to
-  // satisfy the request.
-  map<string, Resources> required_resources = 6;
+  // satisfy the request. This field is only populated when the function uses
+  // extra_resources in its requirements.
+  //
+  // Deprecated: Use required_resources instead.
+  map<string, Resources> extra_resources = 6 [deprecated = true];
 
   // Optional credentials that this function may use to communicate with an
   // external system.
   map<string, Credentials> credentials = 7;
+
+  // Optional resources that the function specified in its requirements. Note
+  // that resources is a map to Resources, plural. The map key corresponds to
+  // the key in a RunFunctionResponse's requirements.resources field. If a
+  // function requested required resources that did not exist, Crossplane sets
+  // the map key to an empty Resources message to indicate that it attempted to
+  // satisfy the request. This field is only populated when the function uses
+  // resources in its requirements.
+  map<string, Resources> required_resources = 8;
 }
 
 // Credentials that a function may use to communicate with an external system.
@@ -150,7 +162,13 @@ message RequestMeta {
 message Requirements {
   // Resources that this function requires. The map key uniquely identifies the
   // group of resources.
-  map<string, ResourceSelector> resources = 1;
+  //
+  // Deprecated: Use resources instead.
+  map<string, ResourceSelector> extra_resources = 1 [deprecated = true];
+
+  // Resources that this function requires. The map key uniquely identifies the
+  // group of resources.
+  map<string, ResourceSelector> resources = 2;
 }
 
 // ResourceSelector selects a group of resources, either by name or by label.
@@ -349,4 +367,4 @@ enum Status {
   STATUS_CONDITION_TRUE = 2;
 
   STATUS_CONDITION_FALSE = 3;
-}
+}