Skip to content

Commit 7f2db36

Browse files
Google APIscopybara-github
authored andcommitted
feat: Add ValidateDirectoryService API for testing AD connection of a storage pool
docs: Removed the format for `replication` in message `google.cloud.netapp.v1.HybridReplicationParameters` PiperOrigin-RevId: 713157619
1 parent 130b113 commit 7f2db36

4 files changed

Lines changed: 39 additions & 3 deletions

File tree

google/cloud/netapp/v1/cloud_netapp_service.proto

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,20 @@ service NetApp {
110110
};
111111
}
112112

113+
// ValidateDirectoryService does a connectivity check for a directory service
114+
// policy attached to the storage pool.
115+
rpc ValidateDirectoryService(ValidateDirectoryServiceRequest)
116+
returns (google.longrunning.Operation) {
117+
option (google.api.http) = {
118+
post: "/v1/{name=projects/*/locations/*/storagePools/*}:validateDirectoryService"
119+
body: "*"
120+
};
121+
option (google.longrunning.operation_info) = {
122+
response_type: "google.protobuf.Empty"
123+
metadata_type: "OperationMetadata"
124+
};
125+
}
126+
113127
// This operation will switch the active/replica zone for a regional
114128
// storagePool.
115129
rpc SwitchActiveReplicaZone(SwitchActiveReplicaZoneRequest)

google/cloud/netapp/v1/common.proto

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,15 @@ enum EncryptionType {
5656
CLOUD_KMS = 2;
5757
}
5858

59+
// Type of directory service
60+
enum DirectoryServiceType {
61+
// Directory service type is not specified.
62+
DIRECTORY_SERVICE_TYPE_UNSPECIFIED = 0;
63+
64+
// Active directory policy attached to the storage pool.
65+
ACTIVE_DIRECTORY = 1;
66+
}
67+
5968
// Metadata for a given
6069
// [google.cloud.location.Location][google.cloud.location.Location].
6170
message LocationMetadata {

google/cloud/netapp/v1/storage_pool.proto

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -254,3 +254,18 @@ message StoragePool {
254254
// Optional. Specifies the active zone for regional storagePool.
255255
string zone = 21 [(google.api.field_behavior) = OPTIONAL];
256256
}
257+
258+
// ValidateDirectoryServiceRequest validates the directory service policy
259+
// attached to the storage pool.
260+
message ValidateDirectoryServiceRequest {
261+
// Required. Name of the storage pool
262+
string name = 1 [
263+
(google.api.field_behavior) = REQUIRED,
264+
(google.api.resource_reference) = {
265+
type: "netapp.googleapis.com/StoragePool"
266+
}
267+
];
268+
269+
// Type of directory service policy attached to the storage pool.
270+
DirectoryServiceType directory_service_type = 2;
271+
}

google/cloud/netapp/v1/volume.proto

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -645,9 +645,7 @@ message TieringPolicy {
645645

646646
// The Hybrid Replication parameters for the volume.
647647
message HybridReplicationParameters {
648-
// Required. Desired Identifier (name) of the replication which will be created for this volume.
649-
// Format:
650-
// `projects/{project_id}/locations/{location}/volumes/{volume_id}/replications/{replication_id}`
648+
// Required. Desired name for the replication of this volume.
651649
string replication = 1 [
652650
(google.api.field_behavior) = REQUIRED,
653651
(google.api.resource_reference) = {

0 commit comments

Comments
 (0)