topological_inventory-providers-common 1.0.8 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 22571a5f48d3d126fc3fddd2992643b2dddccc6c2bb20b5eb8263901bf6ca148
-  data.tar.gz: 437ce9d295f10ab3e10b2b32ea64c89efecd66ff2538c6152f2d67d38f4a9f39
+  metadata.gz: 134e945b201072b8bdd7fc6af73b04a6f60698da831ef3989ab973453a1a5159
+  data.tar.gz: 20e4f260c411551572b557f4e4454fdc03c9e3b2ac9c8f5997e6600780298c1a
 SHA512:
-  metadata.gz: 4abc37ec7c40eef6bdd0e4555404607228edf396b3c9482b7f735419f1359e68bd13139515d9dc2f0a359e28f23e51f38e39877405c5f3e66f7e414f559fce17
-  data.tar.gz: e2652340645a001205c36f4cfad7fe75f0a0dd18301f27cce72fcf856fb48c4be6b2863c00aa7ed7ae40141f1153096ed1bebae6a478e8854f13a52e8524788f
+  metadata.gz: 0a847566f6fb20e9518faef660c626788e6afd20d26cec28a5899e645c68cedd7c017974d0cb1c3c2524253007386b9728a1bb67a0f0a9101fe62bd388f03ccb
+  data.tar.gz: 6e786659b1bf913fe1e22d9c0618986d03dc4bc169d782e446ecd089b97a136b639f83be85c562006dccde83a8d025b102863b565aa571746767fa292e4dae89

data/CHANGELOG.md

@@ -4,8 +4,23 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [1.0.8] - 2020-08-10
+## [2.0.0]
+Operations/API clients refactoring
+
+## [1.0.12] - 2020-10-01
+Add Operations Async Worker class #55
+
+## [1.0.11] - 2020-09-04
+Make Collector Poll Time a parameter so we can tweak the collection interval #51
+
+## [1.0.10] - 2020-08-26
+Add HealthCheck class for operations workers #48
+Set the LOG_LEVEL if present #50
+
+## [1.0.9] - 2020-08-17
 Added refresh-type to save and sweep inventory #45
+
+## [1.0.8] - 2020-08-12
 Add => to error messages that rubocop missed #44
 
 ## [1.0.7] - 2020-07-27
@@ -47,7 +62,12 @@ manageiq-loggers to >= 0.4.2 #20
 ## [1.0.0] - 2020-03-19
 ### Initial release to rubygems.org
 
-[Unreleased]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.8...HEAD
+[Unreleased]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v2.0.0...HEAD
+[2.0.0]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.12...v2.0.0
+[1.0.12]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.11...v1.0.12
+[1.0.11]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.10...v1.0.11
+[1.0.10]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.9...v1.0.10
+[1.0.9]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.8...v1.0.9
 [1.0.8]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.7...v1.0.8
 [1.0.7]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.6...v1.0.7
 [1.0.6]: https://github.com/RedHatInsights/topological_inventory-providers-common/compare/v1.0.5...v1.0.6

data/lib/topological_inventory/providers/common.rb

@@ -1,7 +1,6 @@
 require "topological_inventory/providers/common/version"
 require "topological_inventory/providers/common/logging"
-require "topological_inventory/providers/common/operations/processor"
-require "topological_inventory/providers/common/operations/endpoint_client"
+require "topological_inventory/providers/common/operations/health_check"
 require "topological_inventory/providers/common/collectors_pool"
 require "topological_inventory/providers/common/collector"
 

data/lib/topological_inventory/providers/common/collector.rb

@@ -104,7 +104,8 @@ module TopologicalInventory
                            refresh_state_uuid = nil,
                            refresh_state_part_uuid = nil,
                            refresh_state_part_collected_at = nil,
-                           refresh_state_part_sent_at = Time.now.utc)
+                           refresh_state_part_sent_at = Time.now.utc,
+                           refresh_type = default_refresh_type)
           return 0 if collections.empty?
 
           SaveInventory::Saver.new(:client => ingress_api_client, :logger => logger).save(
@@ -116,7 +117,8 @@ module TopologicalInventory
               :refresh_state_uuid              => refresh_state_uuid,
               :refresh_state_part_uuid         => refresh_state_part_uuid,
               :refresh_state_part_collected_at => refresh_state_part_collected_at,
-              :refresh_state_part_sent_at      => refresh_state_part_sent_at
+              :refresh_state_part_sent_at      => refresh_state_part_sent_at,
+              :refresh_type                    => refresh_type
             )
           )
         rescue => e
@@ -134,7 +136,8 @@ module TopologicalInventory
                             total_parts,
                             sweep_scope,
                             refresh_state_started_at = nil,
-                            refresh_state_sent_at = Time.now.utc)
+                            refresh_state_sent_at = Time.now.utc,
+                            refresh_type = default_refresh_type)
           return if !total_parts || sweep_scope.empty?
 
           SaveInventory::Saver.new(:client => ingress_api_client, :logger => logger).save(
@@ -147,7 +150,8 @@ module TopologicalInventory
               :total_parts              => total_parts,
               :sweep_scope              => sweep_scope,
               :refresh_state_started_at => refresh_state_started_at,
-              :refresh_state_sent_at    => refresh_state_sent_at
+              :refresh_state_sent_at    => refresh_state_sent_at,
+              :refresh_type             => refresh_type
             )
           )
         rescue => e
@@ -165,6 +169,10 @@ module TopologicalInventory
           "Default"
         end
 
+        def default_refresh_type
+          'full-refresh'
+        end
+
         def ingress_api_client
           TopologicalInventoryIngressApiClient::DefaultApi.new
         end

data/lib/topological_inventory/providers/common/collectors_pool.rb

@@ -5,8 +5,9 @@ module TopologicalInventory
     module Common
       class CollectorsPool
         SECRET_FILENAME = "credentials".freeze
+        COLLECTOR_POLL_TIME = ENV['COLLECTOR_POLL_TIME']&.to_i || 300
 
-        def initialize(config_name, metrics, collector_poll_time: 60, thread_pool_size: 2)
+        def initialize(config_name, metrics, collector_poll_time: COLLECTOR_POLL_TIME, thread_pool_size: 2)
           self.config_name         = config_name
           self.collector_status    = Concurrent::Map.new
           self.metrics             = metrics

data/lib/topological_inventory/providers/common/logging.rb

@@ -23,17 +23,23 @@ module TopologicalInventory
         end
 
         def availability_check(message, severity = :info)
-          log_with_prefix("Source#availability_check", message, severity)
+          send("#{severity}_ext", "Source#availability_check", message)
         end
 
-        def log_with_prefix(prefix, message, severity)
-          send(severity, "#{prefix} - #{message}") if respond_to?(severity)
+        %w[debug info warn error fatal].each do |severity|
+          define_method("#{severity}_ext".to_sym) do |prefix, message|
+            ext_message = [prefix, message].compact.join(' - ')
+            send(severity, ext_message)
+          end
         end
       end
 
       class Logger < ManageIQ::Loggers::CloudWatch
         def self.new(*args)
-          super.tap { |logger| logger.extend(TopologicalInventory::Providers::Common::LoggingFunctions) }
+          super.tap do |logger|
+            logger.extend(TopologicalInventory::Providers::Common::LoggingFunctions)
+            logger.level = ENV['LOG_LEVEL'] if ENV['LOG_LEVEL']
+          end
         end
       end
 

data/lib/topological_inventory/providers/common/mixins/sources_api.rb

@@ -0,0 +1,58 @@
+require "topological_inventory/providers/common/sources_api_client"
+
+module TopologicalInventory
+  module Providers
+    module Common
+      module Mixins
+        module SourcesApi
+          AUTH_NOT_NECESSARY = "n/a".freeze
+
+          def sources_api
+            @sources_api ||= TopologicalInventory::Providers::Common::SourcesApiClient.new(identity)
+          end
+
+          def endpoint
+            @endpoint ||= sources_api.fetch_default_endpoint(source_id)
+          rescue => e
+            logger.error_ext(operation, "Failed to fetch Endpoint for Source #{source_id}: #{e.message}")
+            nil
+          end
+
+          def authentication
+            @authentication ||= if endpoint.receptor_node.present?
+                                  AUTH_NOT_NECESSARY
+                                else
+                                  sources_api.fetch_authentication(source_id, endpoint)
+                                end
+          rescue => e
+            logger.error_ext(operation, "Failed to fetch Authentication for Source #{source_id}: #{e.message}")
+            nil
+          end
+
+          def application
+            @application ||= sources_api.fetch_application(source_id)
+          rescue => e
+            logger.error_ext(operation, "Failed to fetch Application for Source #{source_id}: #{e.message}")
+            nil
+          end
+
+          def on_premise?
+            @on_premise ||= endpoint&.receptor_node.to_s.strip.present?
+          end
+
+          def verify_ssl_mode
+            endpoint&.verify_ssl ? OpenSSL::SSL::VERIFY_PEER : OpenSSL::SSL::VERIFY_NONE
+          end
+
+          def full_hostname(endpoint)
+            if on_premise?
+              "receptor://#{endpoint.receptor_node}"
+            else
+              endpoint.host.tap { |host| host << ":#{endpoint.port}" if endpoint.port }
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/topological_inventory/providers/common/mixins/topology_api.rb

@@ -0,0 +1,26 @@
+require "topological_inventory/providers/common/topology_api_client"
+
+module TopologicalInventory
+  module Providers
+    module Common
+      module Mixins
+        module TopologyApi
+          # @identity attr_reader is expected
+          def topology_api
+            @topology_api ||= TopologicalInventory::Providers::Common::TopologyApiClient.new(identity)
+          end
+
+          def update_task(task_id, source_id: nil, state:, status:, target_type: nil, target_source_ref: nil, context: nil)
+            topology_api.update_task(task_id,
+                                     :source_id         => source_id,
+                                     :state             => state,
+                                     :status            => status,
+                                     :target_type       => target_type,
+                                     :target_source_ref => target_source_ref,
+                                     :context           => context)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/topological_inventory/providers/common/mixins/x_rh_headers.rb

@@ -0,0 +1,24 @@
+module TopologicalInventory
+  module Providers
+    module Common
+      module Mixins
+        module XRhHeaders
+          def account_number_by_identity(identity)
+            return @account_number if @account_number
+            return if identity.try(:[], 'x-rh-identity').nil?
+
+            identity_hash = JSON.parse(Base64.decode64(identity['x-rh-identity']))
+            @account_number = identity_hash.dig('identity', 'account_number')
+          rescue JSON::ParserError => e
+            logger.error_ext(operation, "Failed to parse identity header: #{e.message}")
+            nil
+          end
+
+          def identity_by_account_number(account_number)
+            @identity ||= {"x-rh-identity" => Base64.strict_encode64({"identity" => {"account_number" => account_number, "user" => {"is_org_admin" => true}}}.to_json)}
+          end
+        end
+      end
+    end
+  end
+end

data/lib/topological_inventory/providers/common/operations/async_worker.rb

@@ -0,0 +1,56 @@
+require "topological_inventory/providers/common/logging"
+require "topological_inventory/providers/common/operations/health_check"
+
+module TopologicalInventory
+  module Providers
+    module Common
+      module Operations
+        class AsyncWorker
+          include Logging
+
+          def initialize(processor, queue = nil)
+            @processor = processor
+            @queue = queue || Queue.new
+          end
+
+          def start
+            return if thread.present?
+
+            @thread = Thread.new { listen }
+          end
+
+          def stop
+            thread&.exit
+          end
+
+          def enqueue(msg)
+            queue << msg
+          end
+
+          def listen
+            loop do
+              # the queue thread waits for a message to come during `Queue#pop`
+              msg = queue.pop
+              process_message(msg)
+            end
+          end
+
+          private
+
+          attr_reader :thread, :queue, :processor
+
+          def process_message(msg)
+            processor.process!(msg)
+          rescue => err
+            model, method = msg.message.to_s.split(".")
+            logger.error("#{model}##{method}: async worker failure: #{err.cause}\n#{err}\n#{err.backtrace.join("\n")}")
+          ensure
+            msg.ack
+            TopologicalInventory::Providers::Common::Operations::HealthCheck.touch_file
+            logger.debug("Operations::AsyncWorker queue length: #{queue.length}") if queue.length >= 20 && queue.length % 5 == 0
+          end
+        end
+      end
+    end
+  end
+end

data/lib/topological_inventory/providers/common/operations/health_check.rb

@@ -0,0 +1,15 @@
+module TopologicalInventory
+  module Providers
+    module Common
+      module Operations
+        class HealthCheck
+          HEARTBEAT_FILE = '/tmp/healthy'.freeze
+
+          def self.touch_file
+            FileUtils.touch(HEARTBEAT_FILE)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/topological_inventory/providers/common/operations/source.rb

@@ -1,194 +1,169 @@
 require "topological_inventory/providers/common/logging"
 require "active_support/core_ext/numeric/time"
-require "topological_inventory/providers/common/operations/sources_api_client"
+require "topological_inventory/providers/common/mixins/sources_api"
+require "topological_inventory/providers/common/mixins/x_rh_headers"
 
 module TopologicalInventory
   module Providers
-  module Common
-    module Operations
-      class Source
-        include Logging
+    module Common
+      module Operations
+        class Source
+          include Logging
+          include Mixins::SourcesApi
+          include Mixins::XRhHeaders
 
-        STATUS_AVAILABLE, STATUS_UNAVAILABLE = %w[available unavailable].freeze
+          STATUS_AVAILABLE, STATUS_UNAVAILABLE = %w[available unavailable].freeze
 
-        ERROR_MESSAGES = {
-          :authentication_not_found          => "Authentication not found in Sources API",
-          :endpoint_or_application_not_found => "Endpoint or Application not found in Sources API",
-        }.freeze
+          ERROR_MESSAGES = {
+            :authentication_not_found          => "Authentication not found in Sources API",
+            :endpoint_or_application_not_found => "Endpoint or Application not found in Sources API",
+          }.freeze
 
-        LAST_CHECKED_AT_THRESHOLD = 5.minutes.freeze
-        AUTH_NOT_NECESSARY = "n/a".freeze
+          LAST_CHECKED_AT_THRESHOLD = 5.minutes.freeze
+          AUTH_NOT_NECESSARY        = "n/a".freeze
 
-        attr_accessor :params, :request_context, :source_id, :account_number
+          attr_accessor :identity, :operation, :params, :request_context, :source_id, :account_number
 
-        def initialize(params = {}, request_context = nil)
-          self.params          = params
-          self.request_context = request_context
-          self.source_id       = params['source_id']
-          self.account_number  = params['external_tenant']
-        end
+          def initialize(params = {}, request_context = nil)
+            self.operation       = 'Source'
+            self.params          = params
+            self.request_context = request_context
+            self.source_id       = params['source_id']
 
-        def availability_check
-          return if params_missing?
+            self.account_number  = params['external_tenant']
+            self.identity        = identity_by_account_number(account_number)
+          end
 
-          return if checked_recently?
+          def availability_check
+            self.operation += '#availability_check'
 
-          status, error_message = connection_status
+            return if params_missing?
 
-          update_source_and_subresources(status, error_message)
+            return if checked_recently?
 
-          logger.availability_check("Completed: Source #{source_id} is #{status}")
-        end
+            status, error_message = connection_status
 
-        private
-
-        def required_params
-          %w[source_id]
-        end
+            update_source_and_subresources(status, error_message)
 
-        def params_missing?
-          is_missing = false
-          required_params.each do |attr|
-            if (is_missing = params[attr].blank?)
-              logger.availability_check("Missing #{attr} for the availability_check request [Source ID: #{source_id}]", :error)
-              break
-            end
+            logger.availability_check("Completed: Source #{source_id} is #{status}")
           end
 
-          is_missing
-        end
-
-        def checked_recently?
-          checked_recently = if endpoint.present?
-                               endpoint.last_checked_at.present? && endpoint.last_checked_at >= LAST_CHECKED_AT_THRESHOLD.ago
-                             elsif application.present?
-                               application.last_checked_at.present? && application.last_checked_at >= LAST_CHECKED_AT_THRESHOLD.ago
-                             end
+          private
 
-          logger.availability_check("Skipping, last check at #{endpoint.last_checked_at || application.last_checked_at} [Source ID: #{source_id}] ") if checked_recently
-
-          checked_recently
-        end
+          def required_params
+            %w[source_id]
+          end
 
-        def connection_status
-          # we need either an endpoint or application to check the source.
-          return [STATUS_UNAVAILABLE, ERROR_MESSAGES[:endpoint_or_application_not_found]] unless endpoint || application
+          def params_missing?
+            is_missing = false
+            required_params.each do |attr|
+              if (is_missing = params[attr].blank?)
+                logger.availability_check("Missing #{attr} for the availability_check request [Source ID: #{source_id}]", :error)
+                break
+              end
+            end
 
-          check_time
-          if endpoint
-            endpoint_connection_check
-          elsif application
-            application_connection_check
+            is_missing
           end
-        end
 
-        def endpoint_connection_check
-          return [STATUS_UNAVAILABLE, ERROR_MESSAGES[:authentication_not_found]] unless authentication
+          def checked_recently?
+            checked_recently = if endpoint.present?
+                                 endpoint.last_checked_at.present? && endpoint.last_checked_at >= LAST_CHECKED_AT_THRESHOLD.ago
+                               elsif application.present?
+                                 application.last_checked_at.present? && application.last_checked_at >= LAST_CHECKED_AT_THRESHOLD.ago
+                               end
 
-          # call down into the operations pod implementation of `Source#connection_check`
-          connection_check
-        end
+            logger.availability_check("Skipping, last check at #{endpoint.last_checked_at || application.last_checked_at} [Source ID: #{source_id}] ") if checked_recently
 
-        def application_connection_check
-          case application.availability_status
-          when "available"
-            [STATUS_AVAILABLE, nil]
-          when "unavailable"
-            [STATUS_UNAVAILABLE, "Application id #{application.id} unavailable"]
+            checked_recently
           end
-        end
 
-        # @return [Array<String, String|nil] - STATUS_[UN]AVAILABLE, error message
-        def connection_check
-          raise NotImplementedError, "#{__method__} must be implemented in a subclass"
-        end
-
-        def update_source_and_subresources(status, error_message = nil)
-          logger.availability_check("Updating source [#{source_id}] status [#{status}] message [#{error_message}]")
+          def connection_status
+            # we need either an endpoint or application to check the source.
+            return [STATUS_UNAVAILABLE, ERROR_MESSAGES[:endpoint_or_application_not_found]] unless endpoint || application
 
-          update_source(status)
+            check_time
+            if endpoint
+              endpoint_connection_check
+            elsif application
+              application_connection_check
+            end
+          end
 
-          update_endpoint(status, error_message) if endpoint
-          update_application(status) if application
-        end
+          def endpoint_connection_check
+            return [STATUS_UNAVAILABLE, ERROR_MESSAGES[:authentication_not_found]] unless authentication
 
-        def update_source(status)
-          source = ::SourcesApiClient::Source.new
-          source.availability_status = status
-          source.last_checked_at     = check_time
-          source.last_available_at   = check_time if status == STATUS_AVAILABLE
+            # call down into the operations pod implementation of `Source#connection_check`
+            connection_check
+          end
 
-          api_client.update_source(source_id, source)
-        rescue ::SourcesApiClient::ApiError => e
-          logger.availability_check("Failed to update Source id:#{source_id} - #{e.message}", :error)
-        end
+          def application_connection_check
+            case application.availability_status
+            when "available"
+              [STATUS_AVAILABLE, nil]
+            when "unavailable"
+              [STATUS_UNAVAILABLE, "Application id #{application.id} unavailable"]
+            end
+          end
 
-        def update_endpoint(status, error_message)
-          if endpoint.nil?
-            logger.availability_check("Failed to update Endpoint for Source id:#{source_id}. Endpoint not found", :error)
-            return
+          # @return [Array<String, String|nil] - STATUS_[UN]AVAILABLE, error message
+          def connection_check
+            raise NotImplementedError, "#{__method__} must be implemented in a subclass"
           end
 
-          endpoint_update = ::SourcesApiClient::Endpoint.new
+          def update_source_and_subresources(status, error_message = nil)
+            logger.availability_check("Updating source [#{source_id}] status [#{status}] message [#{error_message}]")
 
-          endpoint_update.availability_status       = status
-          endpoint_update.availability_status_error = error_message.to_s
-          endpoint_update.last_checked_at           = check_time
-          endpoint_update.last_available_at         = check_time if status == STATUS_AVAILABLE
+            update_source(status)
 
-          api_client.update_endpoint(endpoint.id, endpoint_update)
-        rescue ::SourcesApiClient::ApiError => e
-          logger.availability_check("Failed to update Endpoint(ID: #{endpoint.id}) - #{e.message}", :error)
-        end
+            update_endpoint(status, error_message) if endpoint
+            update_application(status) if application
+          end
 
-        def update_application(status)
-          application_update = ::SourcesApiClient::Application.new
-          application_update.last_checked_at     = check_time
-          application_update.last_available_at   = check_time if status == STATUS_AVAILABLE
+          def update_source(status)
+            source                     = ::SourcesApiClient::Source.new
+            source.availability_status = status
+            source.last_checked_at     = check_time
+            source.last_available_at   = check_time if status == STATUS_AVAILABLE
 
-          api_client.update_application(application.id, application_update)
-        rescue ::SourcesApiClient::ApiError => e
-          logger.availability_check("Failed to update Application id: #{application.id} - #{e.message}", :error)
-        end
+            sources_api.update_source(source_id, source)
+          rescue ::SourcesApiClient::ApiError => e
+            logger.availability_check("Failed to update Source id:#{source_id} - #{e.message}", :error)
+          end
 
-        def endpoint
-          @endpoint ||= api_client.fetch_default_endpoint(source_id)
-        rescue => e
-          logger.availability_check("Failed to fetch Endpoint for Source #{source_id}: #{e.message}", :error)
-          nil
-        end
+          def update_endpoint(status, error_message)
+            if endpoint.nil?
+              logger.availability_check("Failed to update Endpoint for Source id:#{source_id}. Endpoint not found", :error)
+              return
+            end
 
-        def authentication
-          @authentication ||= if endpoint.receptor_node.present?
-                                AUTH_NOT_NECESSARY
-                              else
-                                api_client.fetch_authentication(source_id, endpoint)
-                              end
-        rescue => e
-          logger.availability_check("Failed to fetch Authentication for Source #{source_id}: #{e.message}", :error)
-          nil
-        end
+            endpoint_update = ::SourcesApiClient::Endpoint.new
 
-        def application
-          @application ||= api_client.fetch_application(source_id)
-        rescue => e
-          logger.availability_check("Failed to fetch Application for Source #{source_id}: #{e.message}", :error)
-          nil
-        end
+            endpoint_update.availability_status       = status
+            endpoint_update.availability_status_error = error_message.to_s
+            endpoint_update.last_checked_at           = check_time
+            endpoint_update.last_available_at         = check_time if status == STATUS_AVAILABLE
 
-        def check_time
-          @check_time ||= Time.now.utc
-        end
+            sources_api.update_endpoint(endpoint.id, endpoint_update)
+          rescue ::SourcesApiClient::ApiError => e
+            logger.availability_check("Failed to update Endpoint(ID: #{endpoint.id}) - #{e.message}", :error)
+          end
 
-        def identity
-          @identity ||= {"x-rh-identity" => Base64.strict_encode64({"identity" => {"account_number" => account_number, "user" => {"is_org_admin" => true}}}.to_json)}
-        end
+          def update_application(status)
+            application_update                   = ::SourcesApiClient::Application.new
+            application_update.last_checked_at   = check_time
+            application_update.last_available_at = check_time if status == STATUS_AVAILABLE
 
-        def api_client
-          @api_client ||= TopologicalInventory::Providers::Common::Operations::SourcesApiClient.new(identity)
+            sources_api.update_application(application.id, application_update)
+          rescue ::SourcesApiClient::ApiError => e
+            logger.availability_check("Failed to update Application id: #{application.id} - #{e.message}", :error)
+          end
+
+          def check_time
+            @check_time ||= Time.now.utc
+          end
         end
       end
     end
   end
-  end
 end