google-cloud-logging-v2 0.1.3 → 0.5.0

data/lib/google/cloud/logging/v2/config_service/paths.rb

@@ -89,13 +89,13 @@ module Google
             # @return [::String]
             def cmek_settings_path **args
               resources = {
-                "project"         => (proc do |project:|
+                "project" => (proc do |project:|
                   "projects/#{project}/cmekSettings"
                 end),
-                "organization"    => (proc do |organization:|
+                "organization" => (proc do |organization:|
                   "organizations/#{organization}/cmekSettings"
                 end),
-                "folder"          => (proc do |folder:|
+                "folder" => (proc do |folder:|
                   "folders/#{folder}/cmekSettings"
                 end),
                 "billing_account" => (proc do |billing_account:|
@@ -198,19 +198,19 @@ module Google
             # @return [::String]
             def log_bucket_path **args
               resources = {
-                "bucket:location:project"         => (proc do |project:, location:, bucket:|
+                "bucket:location:project" => (proc do |project:, location:, bucket:|
                   raise ::ArgumentError, "project cannot contain /" if project.to_s.include? "/"
                   raise ::ArgumentError, "location cannot contain /" if location.to_s.include? "/"
 
                   "projects/#{project}/locations/#{location}/buckets/#{bucket}"
                 end),
-                "bucket:location:organization"    => (proc do |organization:, location:, bucket:|
+                "bucket:location:organization" => (proc do |organization:, location:, bucket:|
                   raise ::ArgumentError, "organization cannot contain /" if organization.to_s.include? "/"
                   raise ::ArgumentError, "location cannot contain /" if location.to_s.include? "/"
 
                   "organizations/#{organization}/locations/#{location}/buckets/#{bucket}"
                 end),
-                "bucket:folder:location"          => (proc do |folder:, location:, bucket:|
+                "bucket:folder:location" => (proc do |folder:, location:, bucket:|
                   raise ::ArgumentError, "folder cannot contain /" if folder.to_s.include? "/"
                   raise ::ArgumentError, "location cannot contain /" if location.to_s.include? "/"
 
@@ -267,17 +267,17 @@ module Google
             # @return [::String]
             def log_exclusion_path **args
               resources = {
-                "exclusion:project"         => (proc do |project:, exclusion:|
+                "exclusion:project" => (proc do |project:, exclusion:|
                   raise ::ArgumentError, "project cannot contain /" if project.to_s.include? "/"
 
                   "projects/#{project}/exclusions/#{exclusion}"
                 end),
-                "exclusion:organization"    => (proc do |organization:, exclusion:|
+                "exclusion:organization" => (proc do |organization:, exclusion:|
                   raise ::ArgumentError, "organization cannot contain /" if organization.to_s.include? "/"
 
                   "organizations/#{organization}/exclusions/#{exclusion}"
                 end),
-                "exclusion:folder"          => (proc do |folder:, exclusion:|
+                "exclusion:folder" => (proc do |folder:, exclusion:|
                   raise ::ArgumentError, "folder cannot contain /" if folder.to_s.include? "/"
 
                   "folders/#{folder}/exclusions/#{exclusion}"
@@ -332,17 +332,17 @@ module Google
             # @return [::String]
             def log_sink_path **args
               resources = {
-                "project:sink"         => (proc do |project:, sink:|
+                "project:sink" => (proc do |project:, sink:|
                   raise ::ArgumentError, "project cannot contain /" if project.to_s.include? "/"
 
                   "projects/#{project}/sinks/#{sink}"
                 end),
-                "organization:sink"    => (proc do |organization:, sink:|
+                "organization:sink" => (proc do |organization:, sink:|
                   raise ::ArgumentError, "organization cannot contain /" if organization.to_s.include? "/"
 
                   "organizations/#{organization}/sinks/#{sink}"
                 end),
-                "folder:sink"          => (proc do |folder:, sink:|
+                "folder:sink" => (proc do |folder:, sink:|
                   raise ::ArgumentError, "folder cannot contain /" if folder.to_s.include? "/"
 
                   "folders/#{folder}/sinks/#{sink}"
@@ -359,6 +359,87 @@ module Google
               resource.call(**args)
             end
 
+            ##
+            # Create a fully-qualified LogView resource string.
+            #
+            # @overload log_view_path(project:, location:, bucket:, view:)
+            #   The resource will be in the following format:
+            #
+            #   `projects/{project}/locations/{location}/buckets/{bucket}/views/{view}`
+            #
+            #   @param project [String]
+            #   @param location [String]
+            #   @param bucket [String]
+            #   @param view [String]
+            #
+            # @overload log_view_path(organization:, location:, bucket:, view:)
+            #   The resource will be in the following format:
+            #
+            #   `organizations/{organization}/locations/{location}/buckets/{bucket}/views/{view}`
+            #
+            #   @param organization [String]
+            #   @param location [String]
+            #   @param bucket [String]
+            #   @param view [String]
+            #
+            # @overload log_view_path(folder:, location:, bucket:, view:)
+            #   The resource will be in the following format:
+            #
+            #   `folders/{folder}/locations/{location}/buckets/{bucket}/views/{view}`
+            #
+            #   @param folder [String]
+            #   @param location [String]
+            #   @param bucket [String]
+            #   @param view [String]
+            #
+            # @overload log_view_path(billing_account:, location:, bucket:, view:)
+            #   The resource will be in the following format:
+            #
+            #   `billingAccounts/{billing_account}/locations/{location}/buckets/{bucket}/views/{view}`
+            #
+            #   @param billing_account [String]
+            #   @param location [String]
+            #   @param bucket [String]
+            #   @param view [String]
+            #
+            # @return [::String]
+            def log_view_path **args
+              resources = {
+                "bucket:location:project:view" => (proc do |project:, location:, bucket:, view:|
+                  raise ::ArgumentError, "project cannot contain /" if project.to_s.include? "/"
+                  raise ::ArgumentError, "location cannot contain /" if location.to_s.include? "/"
+                  raise ::ArgumentError, "bucket cannot contain /" if bucket.to_s.include? "/"
+
+                  "projects/#{project}/locations/#{location}/buckets/#{bucket}/views/#{view}"
+                end),
+                "bucket:location:organization:view" => (proc do |organization:, location:, bucket:, view:|
+                  raise ::ArgumentError, "organization cannot contain /" if organization.to_s.include? "/"
+                  raise ::ArgumentError, "location cannot contain /" if location.to_s.include? "/"
+                  raise ::ArgumentError, "bucket cannot contain /" if bucket.to_s.include? "/"
+
+                  "organizations/#{organization}/locations/#{location}/buckets/#{bucket}/views/#{view}"
+                end),
+                "bucket:folder:location:view" => (proc do |folder:, location:, bucket:, view:|
+                  raise ::ArgumentError, "folder cannot contain /" if folder.to_s.include? "/"
+                  raise ::ArgumentError, "location cannot contain /" if location.to_s.include? "/"
+                  raise ::ArgumentError, "bucket cannot contain /" if bucket.to_s.include? "/"
+
+                  "folders/#{folder}/locations/#{location}/buckets/#{bucket}/views/#{view}"
+                end),
+                "billing_account:bucket:location:view" => (proc do |billing_account:, location:, bucket:, view:|
+                  raise ::ArgumentError, "billing_account cannot contain /" if billing_account.to_s.include? "/"
+                  raise ::ArgumentError, "location cannot contain /" if location.to_s.include? "/"
+                  raise ::ArgumentError, "bucket cannot contain /" if bucket.to_s.include? "/"
+
+                  "billingAccounts/#{billing_account}/locations/#{location}/buckets/#{bucket}/views/#{view}"
+                end)
+              }
+
+              resource = resources[args.keys.sort.join(":")]
+              raise ::ArgumentError, "no resource found for values #{args.keys}" if resource.nil?
+              resource.call(**args)
+            end
+
             ##
             # Create a fully-qualified Organization resource string.
             #

data/lib/google/cloud/logging/v2/logging_service/client.rb

@@ -60,7 +60,7 @@ module Google
                 parent_config = while namespace.any?
                                   parent_name = namespace.join "::"
                                   parent_const = const_get parent_name
-                                  break parent_const.configure if parent_const&.respond_to? :configure
+                                  break parent_const.configure if parent_const.respond_to? :configure
                                   namespace.pop
                                 end
                 default_config = Client::Configuration.new parent_config
@@ -68,41 +68,49 @@ module Google
                 default_config.rpcs.delete_log.timeout = 60.0
                 default_config.rpcs.delete_log.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config.rpcs.write_log_entries.timeout = 60.0
                 default_config.rpcs.write_log_entries.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config.rpcs.list_log_entries.timeout = 60.0
                 default_config.rpcs.list_log_entries.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config.rpcs.list_monitored_resource_descriptors.timeout = 60.0
                 default_config.rpcs.list_monitored_resource_descriptors.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config.rpcs.list_logs.timeout = 60.0
                 default_config.rpcs.list_logs.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
+                }
+
+                default_config.rpcs.tail_log_entries.timeout = 3600.0
+                default_config.rpcs.tail_log_entries.retry_policy = {
+                  initial_delay: 0.1,
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config
@@ -166,7 +174,13 @@ module Google
 
               # Create credentials
               credentials = @config.credentials
-              credentials ||= Credentials.default scope: @config.scope
+              # Use self-signed JWT if the scope and endpoint are unchanged from default,
+              # but only if the default endpoint does not have a region prefix.
+              enable_self_signed_jwt = @config.scope == Client.configure.scope &&
+                                       @config.endpoint == Client.configure.endpoint &&
+                                       !@config.endpoint.split(".").first.include?("-")
+              credentials ||= Credentials.default scope: @config.scope,
+                                                  enable_self_signed_jwt: enable_self_signed_jwt
               if credentials.is_a?(String) || credentials.is_a?(Hash)
                 credentials = Credentials.new credentials, scope: @config.scope
               end
@@ -425,6 +439,11 @@ module Google
             #         "billingAccounts/[BILLING_ACCOUNT_ID]"
             #         "folders/[FOLDER_ID]"
             #
+            #     May alternatively be one or more views
+            #       projects/[PROJECT_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
+            #       organization/[ORGANIZATION_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
+            #       billingAccounts/[BILLING_ACCOUNT_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
+            #       folders/[FOLDER_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
             #
             #     Projects listed in the `project_ids` field are added to this list.
             #   @param filter [::String]
@@ -575,7 +594,7 @@ module Google
             #   @param options [::Gapic::CallOptions, ::Hash]
             #     Overrides the default settings for this call, e.g, timeout, retries, etc. Optional.
             #
-            # @overload list_logs(parent: nil, page_size: nil, page_token: nil)
+            # @overload list_logs(parent: nil, page_size: nil, page_token: nil, resource_names: nil)
             #   Pass arguments to `list_logs` via keyword arguments. Note that at
             #   least one keyword argument is required. To specify no parameters, or to keep all
             #   the default parameter values, pass an empty Hash as a request object (see above).
@@ -596,6 +615,18 @@ module Google
             #     preceding call to this method.  `pageToken` must be the value of
             #     `nextPageToken` from the previous response.  The values of other method
             #     parameters should be identical to those in the previous call.
+            #   @param resource_names [::Array<::String>]
+            #     Optional. The resource name that owns the logs:
+            #       projects/[PROJECT_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
+            #       organization/[ORGANIZATION_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
+            #       billingAccounts/[BILLING_ACCOUNT_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
+            #       folders/[FOLDER_ID]/locations/[LOCATION_ID]/buckets/[BUCKET_ID]/views/[VIEW_ID]
+            #
+            #     To support legacy queries, it could also be:
+            #         "projects/[PROJECT_ID]"
+            #         "organizations/[ORGANIZATION_ID]"
+            #         "billingAccounts/[BILLING_ACCOUNT_ID]"
+            #         "folders/[FOLDER_ID]"
             #
             # @yield [response, operation] Access the result along with the RPC operation
             # @yieldparam response [::Google::Cloud::Logging::V2::ListLogsResponse]
@@ -642,6 +673,59 @@ module Google
               raise ::Google::Cloud::Error.from_error(e)
             end
 
+            ##
+            # Streaming read of log entries as they are ingested. Until the stream is
+            # terminated, it will continue reading logs.
+            #
+            # @param request [::Gapic::StreamInput, ::Enumerable<::Google::Cloud::Logging::V2::TailLogEntriesRequest, ::Hash>]
+            #   An enumerable of {::Google::Cloud::Logging::V2::TailLogEntriesRequest} instances.
+            # @param options [::Gapic::CallOptions, ::Hash]
+            #   Overrides the default settings for this call, e.g, timeout, retries, etc. Optional.
+            #
+            # @yield [response, operation] Access the result along with the RPC operation
+            # @yieldparam response [::Enumerable<::Google::Cloud::Logging::V2::TailLogEntriesResponse>]
+            # @yieldparam operation [::GRPC::ActiveCall::Operation]
+            #
+            # @return [::Enumerable<::Google::Cloud::Logging::V2::TailLogEntriesResponse>]
+            #
+            # @raise [::Google::Cloud::Error] if the RPC is aborted.
+            #
+            def tail_log_entries request, options = nil
+              unless request.is_a? ::Enumerable
+                raise ::ArgumentError, "request must be an Enumerable" unless request.respond_to? :to_enum
+                request = request.to_enum
+              end
+
+              request = request.lazy.map do |req|
+                ::Gapic::Protobuf.coerce req, to: ::Google::Cloud::Logging::V2::TailLogEntriesRequest
+              end
+
+              # Converts hash and nil to an options object
+              options = ::Gapic::CallOptions.new(**options.to_h) if options.respond_to? :to_h
+
+              # Customize the options with defaults
+              metadata = @config.rpcs.tail_log_entries.metadata.to_h
+
+              # Set x-goog-api-client and x-goog-user-project headers
+              metadata[:"x-goog-api-client"] ||= ::Gapic::Headers.x_goog_api_client \
+                lib_name: @config.lib_name, lib_version: @config.lib_version,
+                gapic_version: ::Google::Cloud::Logging::V2::VERSION
+              metadata[:"x-goog-user-project"] = @quota_project_id if @quota_project_id
+
+              options.apply_defaults timeout:      @config.rpcs.tail_log_entries.timeout,
+                                     metadata:     metadata,
+                                     retry_policy: @config.rpcs.tail_log_entries.retry_policy
+              options.apply_defaults metadata:     @config.metadata,
+                                     retry_policy: @config.retry_policy
+
+              @logging_service_stub.call_rpc :tail_log_entries, 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)
+            end
+
             ##
             # Configuration class for the LoggingService API.
             #
@@ -734,7 +818,7 @@ module Google
               config_attr :scope,         nil, ::String, ::Array, nil
               config_attr :lib_name,      nil, ::String, nil
               config_attr :lib_version,   nil, ::String, nil
-              config_attr(:channel_args,  { "grpc.service_config_disable_resolution"=>1 }, ::Hash, nil)
+              config_attr(:channel_args,  { "grpc.service_config_disable_resolution" => 1 }, ::Hash, nil)
               config_attr :interceptors,  nil, ::Array, nil
               config_attr :timeout,       nil, ::Numeric, nil
               config_attr :metadata,      nil, ::Hash, nil
@@ -755,7 +839,7 @@ module Google
               def rpcs
                 @rpcs ||= begin
                   parent_rpcs = nil
-                  parent_rpcs = @parent_config.rpcs if defined?(@parent_config) && @parent_config&.respond_to?(:rpcs)
+                  parent_rpcs = @parent_config.rpcs if defined?(@parent_config) && @parent_config.respond_to?(:rpcs)
                   Rpcs.new parent_rpcs
                 end
               end
@@ -767,7 +851,7 @@ module Google
               # Each configuration object is of type `Gapic::Config::Method` and includes
               # the following configuration fields:
               #
-              #  *  `timeout` (*type:* `Numeric`) - The call timeout in milliseconds
+              #  *  `timeout` (*type:* `Numeric`) - The call timeout in seconds
               #  *  `metadata` (*type:* `Hash{Symbol=>String}`) - Additional gRPC headers
               #  *  `retry_policy (*type:* `Hash`) - The retry policy. The policy fields
               #     include the following keys:
@@ -803,19 +887,26 @@ module Google
                 # @return [::Gapic::Config::Method]
                 #
                 attr_reader :list_logs
+                ##
+                # RPC-specific configuration for `tail_log_entries`
+                # @return [::Gapic::Config::Method]
+                #
+                attr_reader :tail_log_entries
 
                 # @private
                 def initialize parent_rpcs = nil
-                  delete_log_config = parent_rpcs&.delete_log if parent_rpcs&.respond_to? :delete_log
+                  delete_log_config = parent_rpcs.delete_log if parent_rpcs.respond_to? :delete_log
                   @delete_log = ::Gapic::Config::Method.new delete_log_config
-                  write_log_entries_config = parent_rpcs&.write_log_entries if parent_rpcs&.respond_to? :write_log_entries
+                  write_log_entries_config = parent_rpcs.write_log_entries if parent_rpcs.respond_to? :write_log_entries
                   @write_log_entries = ::Gapic::Config::Method.new write_log_entries_config
-                  list_log_entries_config = parent_rpcs&.list_log_entries if parent_rpcs&.respond_to? :list_log_entries
+                  list_log_entries_config = parent_rpcs.list_log_entries if parent_rpcs.respond_to? :list_log_entries
                   @list_log_entries = ::Gapic::Config::Method.new list_log_entries_config
-                  list_monitored_resource_descriptors_config = parent_rpcs&.list_monitored_resource_descriptors if parent_rpcs&.respond_to? :list_monitored_resource_descriptors
+                  list_monitored_resource_descriptors_config = parent_rpcs.list_monitored_resource_descriptors if parent_rpcs.respond_to? :list_monitored_resource_descriptors
                   @list_monitored_resource_descriptors = ::Gapic::Config::Method.new list_monitored_resource_descriptors_config
-                  list_logs_config = parent_rpcs&.list_logs if parent_rpcs&.respond_to? :list_logs
+                  list_logs_config = parent_rpcs.list_logs if parent_rpcs.respond_to? :list_logs
                   @list_logs = ::Gapic::Config::Method.new list_logs_config
+                  tail_log_entries_config = parent_rpcs.tail_log_entries if parent_rpcs.respond_to? :tail_log_entries
+                  @tail_log_entries = ::Gapic::Config::Method.new tail_log_entries_config
 
                   yield self if block_given?
                 end

data/lib/google/cloud/logging/v2/logging_service/paths.rb

@@ -90,17 +90,17 @@ module Google
             # @return [::String]
             def log_path **args
               resources = {
-                "log:project"         => (proc do |project:, log:|
+                "log:project" => (proc do |project:, log:|
                   raise ::ArgumentError, "project cannot contain /" if project.to_s.include? "/"
 
                   "projects/#{project}/logs/#{log}"
                 end),
-                "log:organization"    => (proc do |organization:, log:|
+                "log:organization" => (proc do |organization:, log:|
                   raise ::ArgumentError, "organization cannot contain /" if organization.to_s.include? "/"
 
                   "organizations/#{organization}/logs/#{log}"
                 end),
-                "folder:log"          => (proc do |folder:, log:|
+                "folder:log" => (proc do |folder:, log:|
                   raise ::ArgumentError, "folder cannot contain /" if folder.to_s.include? "/"
 
                   "folders/#{folder}/logs/#{log}"

data/lib/google/cloud/logging/v2/metrics_service/client.rb

@@ -60,7 +60,7 @@ module Google
                 parent_config = while namespace.any?
                                   parent_name = namespace.join "::"
                                   parent_const = const_get parent_name
-                                  break parent_const.configure if parent_const&.respond_to? :configure
+                                  break parent_const.configure if parent_const.respond_to? :configure
                                   namespace.pop
                                 end
                 default_config = Client::Configuration.new parent_config
@@ -68,17 +68,17 @@ module Google
                 default_config.rpcs.list_log_metrics.timeout = 60.0
                 default_config.rpcs.list_log_metrics.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config.rpcs.get_log_metric.timeout = 60.0
                 default_config.rpcs.get_log_metric.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config.rpcs.create_log_metric.timeout = 60.0
@@ -86,17 +86,17 @@ module Google
                 default_config.rpcs.update_log_metric.timeout = 60.0
                 default_config.rpcs.update_log_metric.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config.rpcs.delete_log_metric.timeout = 60.0
                 default_config.rpcs.delete_log_metric.retry_policy = {
                   initial_delay: 0.1,
-                  max_delay:     60.0,
-                  multiplier:    1.3,
-                  retry_codes:   [4, 13, 14]
+              max_delay: 60.0,
+              multiplier: 1.3,
+              retry_codes: [4, 13, 14]
                 }
 
                 default_config
@@ -160,7 +160,13 @@ module Google
 
               # Create credentials
               credentials = @config.credentials
-              credentials ||= Credentials.default scope: @config.scope
+              # Use self-signed JWT if the scope and endpoint are unchanged from default,
+              # but only if the default endpoint does not have a region prefix.
+              enable_self_signed_jwt = @config.scope == Client.configure.scope &&
+                                       @config.endpoint == Client.configure.endpoint &&
+                                       !@config.endpoint.split(".").first.include?("-")
+              credentials ||= Credentials.default scope: @config.scope,
+                                                  enable_self_signed_jwt: enable_self_signed_jwt
               if credentials.is_a?(String) || credentials.is_a?(Hash)
                 credentials = Credentials.new credentials, scope: @config.scope
               end
@@ -631,7 +637,7 @@ module Google
               config_attr :scope,         nil, ::String, ::Array, nil
               config_attr :lib_name,      nil, ::String, nil
               config_attr :lib_version,   nil, ::String, nil
-              config_attr(:channel_args,  { "grpc.service_config_disable_resolution"=>1 }, ::Hash, nil)
+              config_attr(:channel_args,  { "grpc.service_config_disable_resolution" => 1 }, ::Hash, nil)
               config_attr :interceptors,  nil, ::Array, nil
               config_attr :timeout,       nil, ::Numeric, nil
               config_attr :metadata,      nil, ::Hash, nil
@@ -652,7 +658,7 @@ module Google
               def rpcs
                 @rpcs ||= begin
                   parent_rpcs = nil
-                  parent_rpcs = @parent_config.rpcs if defined?(@parent_config) && @parent_config&.respond_to?(:rpcs)
+                  parent_rpcs = @parent_config.rpcs if defined?(@parent_config) && @parent_config.respond_to?(:rpcs)
                   Rpcs.new parent_rpcs
                 end
               end
@@ -664,7 +670,7 @@ module Google
               # Each configuration object is of type `Gapic::Config::Method` and includes
               # the following configuration fields:
               #
-              #  *  `timeout` (*type:* `Numeric`) - The call timeout in milliseconds
+              #  *  `timeout` (*type:* `Numeric`) - The call timeout in seconds
               #  *  `metadata` (*type:* `Hash{Symbol=>String}`) - Additional gRPC headers
               #  *  `retry_policy (*type:* `Hash`) - The retry policy. The policy fields
               #     include the following keys:
@@ -703,15 +709,15 @@ module Google
 
                 # @private
                 def initialize parent_rpcs = nil
-                  list_log_metrics_config = parent_rpcs&.list_log_metrics if parent_rpcs&.respond_to? :list_log_metrics
+                  list_log_metrics_config = parent_rpcs.list_log_metrics if parent_rpcs.respond_to? :list_log_metrics
                   @list_log_metrics = ::Gapic::Config::Method.new list_log_metrics_config
-                  get_log_metric_config = parent_rpcs&.get_log_metric if parent_rpcs&.respond_to? :get_log_metric
+                  get_log_metric_config = parent_rpcs.get_log_metric if parent_rpcs.respond_to? :get_log_metric
                   @get_log_metric = ::Gapic::Config::Method.new get_log_metric_config
-                  create_log_metric_config = parent_rpcs&.create_log_metric if parent_rpcs&.respond_to? :create_log_metric
+                  create_log_metric_config = parent_rpcs.create_log_metric if parent_rpcs.respond_to? :create_log_metric
                   @create_log_metric = ::Gapic::Config::Method.new create_log_metric_config
-                  update_log_metric_config = parent_rpcs&.update_log_metric if parent_rpcs&.respond_to? :update_log_metric
+                  update_log_metric_config = parent_rpcs.update_log_metric if parent_rpcs.respond_to? :update_log_metric
                   @update_log_metric = ::Gapic::Config::Method.new update_log_metric_config
-                  delete_log_metric_config = parent_rpcs&.delete_log_metric if parent_rpcs&.respond_to? :delete_log_metric
+                  delete_log_metric_config = parent_rpcs.delete_log_metric if parent_rpcs.respond_to? :delete_log_metric
                   @delete_log_metric = ::Gapic::Config::Method.new delete_log_metric_config
 
                   yield self if block_given?