fix: Changed field behavior for an existing field per_unit_storage_throughput. It is now required

Google APIs · copybara-github · commit 85d120d7cf5e · 2025-06-03T14:03:16.000-07:00
docs: expand on and update various comments

PiperOrigin-RevId: 766812120
diff --git a/google/cloud/lustre/v1/BUILD.bazel b/google/cloud/lustre/v1/BUILD.bazel
@@ -305,9 +305,7 @@ ruby_grpc_library(
 ruby_cloud_gapic_library(
     name = "lustre_ruby_gapic",
     srcs = [":lustre_proto_with_info"],
-    extra_protoc_parameters = [
-        "ruby-cloud-gem-name=google-cloud-lustre-v1",
-    ],
+    extra_protoc_parameters = ["ruby-cloud-gem-name=google-cloud-lustre-v1"],
     grpc_service_config = "lustre_v1_grpc_service_config.json",
     rest_numeric_enums = True,
     service_yaml = "lustre_v1.yaml",
@@ -342,7 +340,6 @@ load(
 
 csharp_proto_library(
     name = "lustre_csharp_proto",
-    extra_opts = [],
     deps = [":lustre_proto"],
 )
 
diff --git a/google/cloud/lustre/v1/instance.proto b/google/cloud/lustre/v1/instance.proto
@@ -72,7 +72,7 @@ message Instance {
   ];
 
   // Required. The storage capacity of the instance in gibibytes (GiB). Allowed
-  // values are from `18000` to `936000`, in increments of 9000.
+  // values are from `18000` to `954000`, in increments of 9000.
   int64 capacity_gib = 2 [(google.api.field_behavior) = REQUIRED];
 
   // Required. Immutable. The full name of the VPC network to which the instance
@@ -105,11 +105,10 @@ message Instance {
   // Optional. Labels as key value pairs.
   map<string, string> labels = 9 [(google.api.field_behavior) = OPTIONAL];
 
-  // Optional. The throughput of the instance in MB/s/TiB.
-  // Valid values are 250, 500, 1000.
-  // Default value is 1000.
+  // Required. The throughput of the instance in MB/s/TiB.
+  // Valid values are 125, 250, 500, 1000.
   int64 per_unit_storage_throughput = 11
-      [(google.api.field_behavior) = OPTIONAL];
+      [(google.api.field_behavior) = REQUIRED];
 
   // Optional. Indicates whether you want to enable support for GKE clients. By
   // default, GKE clients are not supported.
diff --git a/google/cloud/lustre/v1/transfer.proto b/google/cloud/lustre/v1/transfer.proto
@@ -42,9 +42,11 @@ enum TransferType {
 // Message for importing data to Lustre.
 message ImportDataRequest {
   // A Cloud Storage URI of a folder to import file data from, in the
-  // form of `gs://<bucket_name>/<path_inside_bucket>`
+  // form of `gs://<bucket_name>/<path_inside_bucket>/`.
   oneof source {
     // The Cloud Storage source bucket and, optionally, path inside the bucket.
+    // If a path inside the bucket is specified, it must end with a forward
+    // slash (`/`).
     GcsPath gcs_path = 2;
   }
 
@@ -54,7 +56,8 @@ message ImportDataRequest {
     LustrePath lustre_path = 3;
   }
 
-  // Required. Name of the resource.
+  // Required. The name of the Managed Lustre instance in the format
+  // `projects/{project}/locations/{location}/instances/{instance}`.
   string name = 1 [
     (google.api.field_behavior) = REQUIRED,
     (google.api.resource_reference) = { type: "lustre.googleapis.com/Instance" }
@@ -67,7 +70,7 @@ message ImportDataRequest {
   ];
 
   // Optional. User-specified service account used to perform the transfer.
-  // If unspecified, the default Lustre P4 service account will be used.
+  // If unspecified, the default Managed Lustre service agent will be used.
   string service_account = 5 [
     (google.api.field_behavior) = OPTIONAL,
     (google.api.resource_reference) = {
@@ -76,21 +79,26 @@ message ImportDataRequest {
   ];
 }
 
-// Message for exporting data from Lustre.
+// Export data from Managed Lustre to a Cloud Storage bucket.
 message ExportDataRequest {
   // The source of the data transfer.
   oneof source {
-    // Lustre path source.
+    // The root directory path to the Managed Lustre file system. Must start
+    // with `/`. Default is `/`.
     LustrePath lustre_path = 2;
   }
 
   // The destination of the data transfer.
   oneof destination {
-    // Cloud Storage destination.
+    // The URI to a Cloud Storage bucket, or a path within a bucket, using
+    // the format `gs://<bucket_name>/<optional_path_inside_bucket>/`. If a
+    // path inside the bucket is specified, it must end with a forward slash
+    // (`/`).
     GcsPath gcs_path = 3;
   }
 
-  // Required. Name of the resource.
+  // Required. The name of the Managed Lustre instance in the format
+  // `projects/{project}/locations/{location}/instances/{instance}`.
   string name = 1 [
     (google.api.field_behavior) = REQUIRED,
     (google.api.resource_reference) = { type: "lustre.googleapis.com/Instance" }
@@ -183,17 +191,21 @@ message ImportDataMetadata {
   string api_version = 8 [(google.api.field_behavior) = OUTPUT_ONLY];
 }
 
-// Cloud Storage as the source of a data transfer.
+// Specifies a Cloud Storage bucket and, optionally, a path inside the bucket.
 message GcsPath {
-  // Required. URI to a Cloud Storage path in the format:
-  // `gs://<bucket_name>`.
+  // Required. The URI to a Cloud Storage bucket, or a path within a bucket,
+  // using the format `gs://<bucket_name>/<optional_path_inside_bucket>/`. If a
+  // path inside the bucket is specified, it must end with a forward slash
+  // (`/`).
   string uri = 1 [(google.api.field_behavior) = REQUIRED];
 }
 
-// LustrePath represents a path in the Lustre file system.
+// The root directory path to the Lustre file system.
 message LustrePath {
-  // Optional. Root directory path to the Managed Lustre file system, starting
-  // with `/`. Defaults to `/` if unset.
+  // Optional. The root directory path to the Managed Lustre file system. Must
+  // start with
+  // `/`. Default is `/`. If you're importing data into Managed Lustre, any
+  // path other than the default must already exist on the file system.
   string path = 1 [(google.api.field_behavior) = OPTIONAL];
 }
 
