google-cloud-spanner-admin-instance-v1 1.3.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c12a23a53590ba4b88196b0e358b59623c2f9d7c00290540ddaf9e26ed94c43
-  data.tar.gz: dade9c8f576e31c8f36d4bfc55f7b55cbd781dda356c8d7496235a49fc0a7333
+  metadata.gz: 96acd673d20b00e3741ae06619e7862339482f0f2f6383b93f948d0f48c89a2a
+  data.tar.gz: b526506524a3bdc43048e377124924394737a48b2b775718a76f8f95e34e90c3
 SHA512:
-  metadata.gz: b43d70df84753c1db2cb98eccbb9e6a99080a9509d772c3fe84e65443c26df24fccec3b34e79b1cea42fe8cedd791550050574f42f00acefc0399bd84e6118a0
-  data.tar.gz: 640a6733f2c82e84bb49fb9f62ccc79b079c944f5377ba105164d717b80384aaa3379548f6b017fce29f77cea36e80dd53ec6a46776a9167061d238af18137a4
+  metadata.gz: 6c38818d6839ead43f5fed31b7d8ef62caaf86f2d46c552ad1c4f27992a3b81a9ee6f3c8bdbab919a8ed0e44c1a46a9fc8dd66710109d987c2e018dfdc1c7b1f
+  data.tar.gz: 83c7ade0d395b7ce8d431aa5d230ee185662dea2af2314d901cdc6b292e891d2f0e4493b216772ef44ce68f31646d17d85f0f79d8fded71368cbb171c4132e34

data/README.md

@@ -43,33 +43,43 @@ for class and method documentation.
 See also the [Product Documentation](https://cloud.google.com/spanner)
 for general usage information.
 
-## Enabling Logging
-
-To enable logging for this library, set the logger for the underlying [gRPC](https://github.com/grpc/grpc/tree/master/src/ruby) library.
-The logger that you set may be a Ruby stdlib [`Logger`](https://ruby-doc.org/current/stdlibs/logger/Logger.html) as shown below,
-or a [`Google::Cloud::Logging::Logger`](https://cloud.google.com/ruby/docs/reference/google-cloud-logging/latest)
-that will write logs to [Cloud Logging](https://cloud.google.com/logging/). See [grpc/logconfig.rb](https://github.com/grpc/grpc/blob/master/src/ruby/lib/grpc/logconfig.rb)
-and the gRPC [spec_helper.rb](https://github.com/grpc/grpc/blob/master/src/ruby/spec/spec_helper.rb) for additional information.
-
-Configuring a Ruby stdlib logger:
+## Debug Logging
+
+This library comes with opt-in Debug Logging that can help you troubleshoot
+your application's integration with the API. When logging is activated, key
+events such as requests and responses, along with data payloads and metadata
+such as headers and client configuration, are logged to the standard error
+stream.
+
+**WARNING:** Client Library Debug Logging includes your data payloads in
+plaintext, which could include sensitive data such as PII for yourself or your
+customers, private keys, or other security data that could be compromising if
+leaked. Always practice good data hygiene with your application logs, and follow
+the principle of least access. Google also recommends that Client Library Debug
+Logging be enabled only temporarily during active debugging, and not used
+permanently in production.
+
+To enable logging, set the environment variable `GOOGLE_SDK_RUBY_LOGGING_GEMS`
+to the value `all`. Alternatively, you can set the value to a comma-delimited
+list of client library gem names. This will select the default logging behavior,
+which writes logs to the standard error stream. On a local workstation, this may
+result in logs appearing on the console. When running on a Google Cloud hosting
+service such as [Google Cloud Run](https://cloud.google.com/run), this generally
+results in logs appearing alongside your application logs in the
+[Google Cloud Logging](https://cloud.google.com/logging/) service.
+
+You can customize logging by modifying the `logger` configuration when
+constructing a client object. For example:
 
 ```ruby
+require "google/cloud/spanner/admin/instance/v1"
 require "logger"
 
-module MyLogger
-  LOGGER = Logger.new $stderr, level: Logger::WARN
-  def logger
-    LOGGER
-  end
-end
-
-# Define a gRPC module-level logger method before grpc/logconfig.rb loads.
-module GRPC
-  extend MyLogger
+client = ::Google::Cloud::Spanner::Admin::Instance::V1::InstanceAdmin::Client.new do |config|
+  config.logger = Logger.new "my-app.log"
 end
 ```
 
-
 ## Google Cloud Samples
 
 To browse ready to use code samples check [Google Cloud Samples](https://cloud.google.com/docs/samples).

data/lib/google/cloud/spanner/admin/instance/v1/instance_admin/client.rb

@@ -224,8 +224,19 @@ module Google
                     universe_domain: @config.universe_domain,
                     channel_args: @config.channel_args,
                     interceptors: @config.interceptors,
-                    channel_pool_config: @config.channel_pool
+                    channel_pool_config: @config.channel_pool,
+                    logger: @config.logger
                   )
+
+                  @instance_admin_stub.stub_logger&.info do |entry|
+                    entry.set_system_name
+                    entry.set_service
+                    entry.message = "Created client for #{entry.service}"
+                    entry.set_credentials_fields credentials
+                    entry.set "customEndpoint", @config.endpoint if @config.endpoint
+                    entry.set "defaultTimeout", @config.timeout if @config.timeout
+                    entry.set "quotaProject", @quota_project_id if @quota_project_id
+                  end
                 end
 
                 ##
@@ -235,6 +246,15 @@ module Google
                 #
                 attr_reader :operations_client
 
+                ##
+                # The logger used for request/response debug logging.
+                #
+                # @return [Logger]
+                #
+                def logger
+                  @instance_admin_stub.logger
+                end
+
                 # Service calls
 
                 ##
@@ -332,7 +352,7 @@ module Google
                   @instance_admin_stub.call_rpc :list_instance_configs, request, options: options do |response, operation|
                     response = ::Gapic::PagedEnumerable.new @instance_admin_stub, :list_instance_configs, request, response, operation, options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -419,7 +439,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :get_instance_config, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -568,7 +587,7 @@ module Google
                   @instance_admin_stub.call_rpc :create_instance_config, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -721,7 +740,7 @@ module Google
                   @instance_admin_stub.call_rpc :update_instance_config, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -827,7 +846,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :delete_instance_config, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -983,7 +1001,7 @@ module Google
                     wrap_lro_operation = ->(op_response) { ::Gapic::Operation.new op_response, @operations_client }
                     response = ::Gapic::PagedEnumerable.new @instance_admin_stub, :list_instance_config_operations, request, response, operation, options, format_resource: wrap_lro_operation
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1110,7 +1128,7 @@ module Google
                   @instance_admin_stub.call_rpc :list_instances, request, options: options do |response, operation|
                     response = ::Gapic::PagedEnumerable.new @instance_admin_stub, :list_instances, request, response, operation, options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1217,7 +1235,7 @@ module Google
                   @instance_admin_stub.call_rpc :list_instance_partitions, request, options: options do |response, operation|
                     response = ::Gapic::PagedEnumerable.new @instance_admin_stub, :list_instance_partitions, request, response, operation, options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1309,7 +1327,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :get_instance, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1444,7 +1461,7 @@ module Google
                   @instance_admin_stub.call_rpc :create_instance, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1586,7 +1603,7 @@ module Google
                   @instance_admin_stub.call_rpc :update_instance, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1683,7 +1700,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :delete_instance, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1785,7 +1801,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :set_iam_policy, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1879,7 +1894,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :get_iam_policy, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -1976,7 +1990,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :test_iam_permissions, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -2064,7 +2077,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :get_instance_partition, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -2204,7 +2216,7 @@ module Google
                   @instance_admin_stub.call_rpc :create_instance_partition, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -2303,7 +2315,6 @@ module Google
 
                   @instance_admin_stub.call_rpc :delete_instance_partition, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -2450,7 +2461,7 @@ module Google
                   @instance_admin_stub.call_rpc :update_instance_partition, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -2616,7 +2627,7 @@ module Google
                     wrap_lro_operation = ->(op_response) { ::Gapic::Operation.new op_response, @operations_client }
                     response = ::Gapic::PagedEnumerable.new @instance_admin_stub, :list_instance_partition_operations, request, response, operation, options, format_resource: wrap_lro_operation
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -2774,7 +2785,7 @@ module Google
                   @instance_admin_stub.call_rpc :move_instance, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -2863,6 +2874,11 @@ module Google
                 #   default endpoint URL. The default value of nil uses the environment
                 #   universe (usually the default "googleapis.com" universe).
                 #   @return [::String,nil]
+                # @!attribute [rw] logger
+                #   A custom logger to use for request/response debug logging, or the value
+                #   `:default` (the default) to construct a default logger, or `nil` to
+                #   explicitly disable logging.
+                #   @return [::Logger,:default,nil]
                 #
                 class Configuration
                   extend ::Gapic::Config
@@ -2887,6 +2903,7 @@ module Google
                   config_attr :retry_policy,  nil, ::Hash, ::Proc, nil
                   config_attr :quota_project, nil, ::String, nil
                   config_attr :universe_domain, nil, ::String, nil
+                  config_attr :logger, :default, ::Logger, nil, :default
 
                   # @private
                   def initialize parent_config = nil

data/lib/google/cloud/spanner/admin/instance/v1/instance_admin/operations.rb

@@ -126,14 +126,6 @@ module Google
                 # Lists operations that match the specified filter in the request. If the
                 # server doesn't support this method, it returns `UNIMPLEMENTED`.
                 #
-                # NOTE: the `name` binding allows API services to override the binding
-                # to use different resource name schemes, such as `users/*/operations`. To
-                # override the binding, API services can add a binding such as
-                # `"/v1/{name=users/*}/operations"` to their service configuration.
-                # For backwards compatibility, the default name includes the operations
-                # collection id, however overriding users must ensure the name binding
-                # is the parent resource, without the operations collection id.
-                #
                 # @overload list_operations(request, options = nil)
                 #   Pass arguments to `list_operations` via a request object, either of type
                 #   {::Google::Longrunning::ListOperationsRequest} or an equivalent Hash.
@@ -223,7 +215,7 @@ module Google
                     wrap_lro_operation = ->(op_response) { ::Gapic::Operation.new op_response, @operations_client }
                     response = ::Gapic::PagedEnumerable.new @operations_stub, :list_operations, request, response, operation, options, format_resource: wrap_lro_operation
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -319,7 +311,7 @@ module Google
                   @operations_stub.call_rpc :get_operation, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -408,7 +400,6 @@ module Google
 
                   @operations_stub.call_rpc :delete_operation, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -423,8 +414,9 @@ module Google
                 # other methods to check whether the cancellation succeeded or whether the
                 # operation completed despite cancellation. On successful cancellation,
                 # the operation is not deleted; instead, it becomes an operation with
-                # an {::Google::Longrunning::Operation#error Operation.error} value with a {::Google::Rpc::Status#code google.rpc.Status.code} of 1,
-                # corresponding to `Code.CANCELLED`.
+                # an {::Google::Longrunning::Operation#error Operation.error} value with a
+                # {::Google::Rpc::Status#code google.rpc.Status.code} of `1`, corresponding to
+                # `Code.CANCELLED`.
                 #
                 # @overload cancel_operation(request, options = nil)
                 #   Pass arguments to `cancel_operation` via a request object, either of type
@@ -503,7 +495,6 @@ module Google
 
                   @operations_stub.call_rpc :cancel_operation, request, options: options do |response, operation|
                     yield response, operation if block_given?
-                    return response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -601,7 +592,7 @@ module Google
                   @operations_stub.call_rpc :wait_operation, request, options: options do |response, operation|
                     response = ::Gapic::Operation.new response, @operations_client, options: options
                     yield response, operation if block_given?
-                    return response
+                    throw :response, response
                   end
                 rescue ::GRPC::BadStatus => e
                   raise ::Google::Cloud::Error.from_error(e)
@@ -690,6 +681,11 @@ module Google
                 #   default endpoint URL. The default value of nil uses the environment
                 #   universe (usually the default "googleapis.com" universe).
                 #   @return [::String,nil]
+                # @!attribute [rw] logger
+                #   A custom logger to use for request/response debug logging, or the value
+                #   `:default` (the default) to construct a default logger, or `nil` to
+                #   explicitly disable logging.
+                #   @return [::Logger,:default,nil]
                 #
                 class Configuration
                   extend ::Gapic::Config
@@ -714,6 +710,7 @@ module Google
                   config_attr :retry_policy,  nil, ::Hash, ::Proc, nil
                   config_attr :quota_project, nil, ::String, nil
                   config_attr :universe_domain, nil, ::String, nil
+                  config_attr :logger, :default, ::Logger, nil, :default
 
                   # @private
                   def initialize parent_config = nil

data/lib/google/cloud/spanner/admin/instance/v1/instance_admin/rest/client.rb

@@ -217,8 +217,19 @@ module Google
                       endpoint: @config.endpoint,
                       endpoint_template: DEFAULT_ENDPOINT_TEMPLATE,
                       universe_domain: @config.universe_domain,
-                      credentials: credentials
+                      credentials: credentials,
+                      logger: @config.logger
                     )
+
+                    @instance_admin_stub.logger(stub: true)&.info do |entry|
+                      entry.set_system_name
+                      entry.set_service
+                      entry.message = "Created client for #{entry.service}"
+                      entry.set_credentials_fields credentials
+                      entry.set "customEndpoint", @config.endpoint if @config.endpoint
+                      entry.set "defaultTimeout", @config.timeout if @config.timeout
+                      entry.set "quotaProject", @quota_project_id if @quota_project_id
+                    end
                   end
 
                   ##
@@ -228,6 +239,15 @@ module Google
                   #
                   attr_reader :operations_client
 
+                  ##
+                  # The logger used for request/response debug logging.
+                  #
+                  # @return [Logger]
+                  #
+                  def logger
+                    @instance_admin_stub.logger
+                  end
+
                   # Service calls
 
                   ##
@@ -318,7 +338,7 @@ module Google
                     @instance_admin_stub.list_instance_configs request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @instance_admin_stub, :list_instance_configs, "instance_configs", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -398,7 +418,6 @@ module Google
 
                     @instance_admin_stub.get_instance_config request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -540,7 +559,7 @@ module Google
                     @instance_admin_stub.create_instance_config request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -686,7 +705,7 @@ module Google
                     @instance_admin_stub.update_instance_config request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -785,7 +804,6 @@ module Google
 
                     @instance_admin_stub.delete_instance_config request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -933,7 +951,7 @@ module Google
                     @instance_admin_stub.list_instance_config_operations request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @instance_admin_stub, :list_instance_config_operations, "operations", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1052,7 +1070,6 @@ module Google
 
                     @instance_admin_stub.list_instances request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1151,7 +1168,6 @@ module Google
 
                     @instance_admin_stub.list_instance_partitions request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1236,7 +1252,6 @@ module Google
 
                     @instance_admin_stub.get_instance request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1364,7 +1379,7 @@ module Google
                     @instance_admin_stub.create_instance request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1499,7 +1514,7 @@ module Google
                     @instance_admin_stub.update_instance request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1589,7 +1604,6 @@ module Google
 
                     @instance_admin_stub.delete_instance request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1684,7 +1698,6 @@ module Google
 
                     @instance_admin_stub.set_iam_policy request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1771,7 +1784,6 @@ module Google
 
                     @instance_admin_stub.get_iam_policy request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1861,7 +1873,6 @@ module Google
 
                     @instance_admin_stub.test_iam_permissions request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1942,7 +1953,6 @@ module Google
 
                     @instance_admin_stub.get_instance_partition request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2075,7 +2085,7 @@ module Google
                     @instance_admin_stub.create_instance_partition request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2167,7 +2177,6 @@ module Google
 
                     @instance_admin_stub.delete_instance_partition request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2307,7 +2316,7 @@ module Google
                     @instance_admin_stub.update_instance_partition request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2465,7 +2474,7 @@ module Google
                     @instance_admin_stub.list_instance_partition_operations request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2616,7 +2625,7 @@ module Google
                     @instance_admin_stub.move_instance request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2696,6 +2705,11 @@ module Google
                   #   default endpoint URL. The default value of nil uses the environment
                   #   universe (usually the default "googleapis.com" universe).
                   #   @return [::String,nil]
+                  # @!attribute [rw] logger
+                  #   A custom logger to use for request/response debug logging, or the value
+                  #   `:default` (the default) to construct a default logger, or `nil` to
+                  #   explicitly disable logging.
+                  #   @return [::Logger,:default,nil]
                   #
                   class Configuration
                     extend ::Gapic::Config
@@ -2717,6 +2731,7 @@ module Google
                     config_attr :retry_policy,  nil, ::Hash, ::Proc, nil
                     config_attr :quota_project, nil, ::String, nil
                     config_attr :universe_domain, nil, ::String, nil
+                    config_attr :logger, :default, ::Logger, nil, :default
 
                     # @private
                     def initialize parent_config = nil