google-cloud-orchestration-airflow-service-v1 1.1.0 → 1.3.0

data/lib/google/cloud/orchestration/airflow/service/v1/environments/rest/client.rb

@@ -159,8 +159,19 @@ module Google
                       endpoint: @config.endpoint,
                       endpoint_template: DEFAULT_ENDPOINT_TEMPLATE,
                       universe_domain: @config.universe_domain,
-                      credentials: credentials
+                      credentials: credentials,
+                      logger: @config.logger
                     )
+
+                    @environments_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
 
                   ##
@@ -170,6 +181,15 @@ module Google
                   #
                   attr_reader :operations_client
 
+                  ##
+                  # The logger used for request/response debug logging.
+                  #
+                  # @return [Logger]
+                  #
+                  def logger
+                    @environments_stub.logger
+                  end
+
                   # Service calls
 
                   ##
@@ -256,7 +276,7 @@ module Google
                     @environments_stub.create_environment 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)
@@ -336,7 +356,6 @@ module Google
 
                     @environments_stub.get_environment 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)
@@ -425,7 +444,7 @@ module Google
                     @environments_stub.list_environments request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @environments_stub, :list_environments, "environments", 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)
@@ -640,7 +659,7 @@ module Google
                     @environments_stub.update_environment 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)
@@ -728,7 +747,7 @@ module Google
                     @environments_stub.delete_environment 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)
@@ -817,7 +836,6 @@ module Google
 
                     @environments_stub.execute_airflow_command 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)
@@ -906,7 +924,6 @@ module Google
 
                     @environments_stub.stop_airflow_command 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)
@@ -994,7 +1011,6 @@ module Google
 
                     @environments_stub.poll_airflow_command 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)
@@ -1005,7 +1021,7 @@ module Google
                   # runs a single Composer component.
                   #
                   # This method is supported for Cloud Composer environments in versions
-                  # composer-3.*.*-airflow-*.*.* and newer.
+                  # composer-2.*.*-airflow-*.*.* and newer.
                   #
                   # @overload list_workloads(request, options = nil)
                   #   Pass arguments to `list_workloads` via a request object, either of type
@@ -1095,7 +1111,7 @@ module Google
                     @environments_stub.list_workloads request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @environments_stub, :list_workloads, "workloads", 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)
@@ -1211,7 +1227,7 @@ module Google
                     @environments_stub.check_upgrade 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)
@@ -1296,7 +1312,6 @@ module Google
 
                     @environments_stub.create_user_workloads_secret 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)
@@ -1380,7 +1395,6 @@ module Google
 
                     @environments_stub.get_user_workloads_secret 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)
@@ -1473,7 +1487,7 @@ module Google
                     @environments_stub.list_user_workloads_secrets request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @environments_stub, :list_user_workloads_secrets, "user_workloads_secrets", 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)
@@ -1555,7 +1569,6 @@ module Google
 
                     @environments_stub.update_user_workloads_secret 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)
@@ -1638,7 +1651,6 @@ module Google
 
                     @environments_stub.delete_user_workloads_secret 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)
@@ -1723,7 +1735,6 @@ module Google
 
                     @environments_stub.create_user_workloads_config_map 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)
@@ -1806,7 +1817,6 @@ module Google
 
                     @environments_stub.get_user_workloads_config_map 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)
@@ -1899,7 +1909,7 @@ module Google
                     @environments_stub.list_user_workloads_config_maps request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @environments_stub, :list_user_workloads_config_maps, "user_workloads_config_maps", 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)
@@ -1981,7 +1991,6 @@ module Google
 
                     @environments_stub.update_user_workloads_config_map 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)
@@ -2064,7 +2073,6 @@ module Google
 
                     @environments_stub.delete_user_workloads_config_map 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)
@@ -2158,7 +2166,7 @@ module Google
                     @environments_stub.save_snapshot 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)
@@ -2264,7 +2272,7 @@ module Google
                     @environments_stub.load_snapshot 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)
@@ -2352,7 +2360,7 @@ module Google
                     @environments_stub.database_failover 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)
@@ -2432,7 +2440,6 @@ module Google
 
                     @environments_stub.fetch_database_properties 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)
@@ -2480,6 +2487,13 @@ module Google
                   #    *  (`Signet::OAuth2::Client`) A signet oauth2 client object
                   #       (see the [signet docs](https://rubydoc.info/gems/signet/Signet/OAuth2/Client))
                   #    *  (`nil`) indicating no credentials
+                  #
+                  #   Warning: If you accept a credential configuration (JSON file or Hash) from an
+                  #   external source for authentication to Google Cloud, you must validate it before
+                  #   providing it to a Google API client library. Providing an unvalidated credential
+                  #   configuration to Google APIs can compromise the security of your systems and data.
+                  #   For more information, refer to [Validate credential configurations from external
+                  #   sources](https://cloud.google.com/docs/authentication/external/externally-sourced-credentials).
                   #   @return [::Object]
                   # @!attribute [rw] scope
                   #   The OAuth scopes
@@ -2512,6 +2526,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
@@ -2533,6 +2552,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/orchestration/airflow/service/v1/environments/rest/operations.rb

@@ -117,14 +117,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.
@@ -206,7 +198,7 @@ module Google
                     @operations_stub.list_operations request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @operations_stub, :list_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)
@@ -295,7 +287,7 @@ module Google
                     @operations_stub.get_operation 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)
@@ -377,7 +369,6 @@ module Google
 
                     @operations_stub.delete_operation 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)
@@ -392,8 +383,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
@@ -465,7 +457,6 @@ module Google
 
                     @operations_stub.cancel_operation 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)
@@ -513,6 +504,13 @@ module Google
                   #    *  (`Signet::OAuth2::Client`) A signet oauth2 client object
                   #       (see the [signet docs](https://rubydoc.info/gems/signet/Signet/OAuth2/Client))
                   #    *  (`nil`) indicating no credentials
+                  #
+                  #   Warning: If you accept a credential configuration (JSON file or Hash) from an
+                  #   external source for authentication to Google Cloud, you must validate it before
+                  #   providing it to a Google API client library. Providing an unvalidated credential
+                  #   configuration to Google APIs can compromise the security of your systems and data.
+                  #   For more information, refer to [Validate credential configurations from external
+                  #   sources](https://cloud.google.com/docs/authentication/external/externally-sourced-credentials).
                   #   @return [::Object]
                   # @!attribute [rw] scope
                   #   The OAuth scopes
@@ -545,6 +543,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
@@ -566,6 +569,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
@@ -685,16 +689,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "list_operations",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Longrunning::ListOperationsResponse.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##
@@ -723,16 +729,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "get_operation",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Longrunning::Operation.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##
@@ -761,16 +769,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "delete_operation",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Protobuf::Empty.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##
@@ -799,16 +809,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "cancel_operation",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Protobuf::Empty.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##