alephant-broker 3.0.1 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 68a9bc33df3994006cc8a45e6c03e7609ad87a85
-  data.tar.gz: b9f7f4f87c2d11c774a4ccaaeed7c7e1ee3b6f9a
+  metadata.gz: a9e0e97812cfc74d3c37cdcdab494a245352a699
+  data.tar.gz: 3b396eee30f30acf3e21317c698e806bcd058c1d
 SHA512:
-  metadata.gz: 02b9c8a85184e32b00019a3401727c32d06445e1afc61248d16e9960b0378a1ed153df9da524c5b8c198aa53792e9dc6f488695edaf52e6ee9369d83d8ebd126
-  data.tar.gz: f5f3d1fee60fa8ce65a8615fcd51d159e5ddb7bd8551fa2916cfd695fc318eb0142aa3b32070e91f250b2520b98962b1994c0cb1dd2d569edcc96f58d43b6e55
+  metadata.gz: deef43f8365dc333a520d02aacad56193f2ecb72ddde29df9d0478827982e7d546f8bc688c31b90dce54538e202d9d80ca2aed2b0b22c439ea9c2f2eb275c56c
+  data.tar.gz: ae2f6611c3b0ce73fe341886464e3d937ae848c76c638d09a1b37d0e9832425defd44e48224688bf928e504a8244b3f499f5cc60d3337f43d68007f877e41678

data/lib/alephant/broker/cache.rb

@@ -15,6 +15,7 @@ module Alephant
             @@client ||= @@elasticache.client
           else
             logger.debug('Broker::Cache::Client#initialize: No config endpoint, NullClient used')
+            logger.metric(:name => "BrokerCacheClientNoConfigEndpoint", :unit => "Count", :value => 1)
             @@client = NullClient.new
           end
         end
@@ -23,6 +24,7 @@ module Alephant
           begin
             result = @@client.get(versioned(key))
             logger.info("Broker::Cache::Client#get key: #{key} - #{result ? 'hit' : 'miss'}")
+            logger.metric(:name => "BrokerCacheClientGetKeyMiss", :unit => "Count", :value => 1) unless result
             result ? result : set(key, block.call)
           rescue StandardError => e
             block.call if block_given?

data/lib/alephant/broker/component.rb

@@ -1,5 +1,4 @@
 require 'crimp'
-require 'alephant/logger'
 require 'alephant/cache'
 require 'alephant/lookup'
 require 'alephant/broker/errors/invalid_cache_key'

data/lib/alephant/broker/component_factory.rb

@@ -20,9 +20,11 @@ module Alephant
         )
       rescue Alephant::Broker::Errors::ContentNotFound => e
         logger.warn 'Broker.ComponentFactory.create: Exception raised (ContentNotFound)'
+        logger.metric(:name => "BrokerComponentFactoryContentNotFound", :unit => "Count", :value => 1)
         ErrorComponent.new(component_meta, 404, e)
       rescue => e
         logger.warn("Broker.ComponentFactory.create: Exception raised (#{e.message}, #{e.backtrace.join('\n')})")
+        logger.metric(:name => "BrokerComponentFactoryExceptionRaised", :unit => "Count", :value => 1)
         ErrorComponent.new(component_meta, 500, e)
       end
     end

data/lib/alephant/broker/environment.rb

@@ -51,6 +51,7 @@ module Alephant
           JSON.parse(json)
         rescue JSON::ParserError => e
           logger.warn("Broker.environment#data: ParserError")
+          logger.metric(:name => "BrokerEnvironmentJSONParserError", :unit => "Count", :value => 1)
           nil
         end
       end

data/lib/alephant/broker/load_strategy/http.rb

@@ -1,11 +1,14 @@
 require 'alephant/broker/cache'
 require 'alephant/broker/errors/content_not_found'
+require 'alephant/logger'
 require 'faraday'
 
 module Alephant
   module Broker
     module LoadStrategy
       class HTTP
+        include Logger
+
         class URL
           def generate
             raise NotImplementedError
@@ -19,6 +22,7 @@ module Alephant
         def load(component_meta)
           cache_object(component_meta)
         rescue
+          logger.metric(:name => "BrokerLoadStrategyHTTPCacheMiss", :unit => "Count", :value => 1)
           cache.set(component_meta.cache_key, content(component_meta))
         end
 
@@ -47,9 +51,17 @@ module Alephant
         end
 
         def request(component_meta)
+          before = Time.new
           component_meta.cached = false
+
           Faraday.get(url_for component_meta).tap do |r|
-            raise Alephant::Broker::Errors::ContentNotFound unless r.success?
+            unless r.success?
+              logger.metric(:name => "BrokerLoadStrategyHTTPContentNotFound", :unit => "Count", :value => 1)
+              raise Alephant::Broker::Errors::ContentNotFound
+            end
+
+            request_time = Time.new - before
+            logger.metric(:name => "BrokerHTTPLoadComponentTime", :unit => "Seconds", :value => request_time)
           end
         end
 

data/lib/alephant/broker/load_strategy/s3.rb

@@ -1,17 +1,21 @@
 require "alephant/broker/cache"
 require 'alephant/broker/errors/content_not_found'
 require 'alephant/broker/errors/invalid_cache_key'
+require 'alephant/logger'
 
 module Alephant
   module Broker
     module LoadStrategy
       class S3
+        include Logger
+
         def load(component_meta)
           add_s3_headers(
             cache_object(component_meta),
             component_meta
           )
         rescue
+          logger.metric(:name => "BrokerLoadStrategyS3CacheMiss", :unit => "Count", :value => 1)
           add_s3_headers(
             cache.set(
               component_meta.cache_key,
@@ -45,6 +49,7 @@ module Alephant
           component_meta.cached = false
           s3.get s3_path(component_meta)
         rescue AWS::S3::Errors::NoSuchKey, InvalidCacheKey
+          logger.metric(:name => "BrokerLoadStrategyS3InvalidCacheKey", :unit => "Count", :value => 1)
           raise Alephant::Broker::Errors::ContentNotFound
         end
 

data/lib/alephant/broker/request/asset.rb

@@ -16,6 +16,9 @@ module Alephant
             nil,
             env.options
           )
+        rescue InvalidAssetId
+          logger.metric(:name => "BrokerRequestAssetInvalidAssetId", :unit => "Count", :value => 1)
+          logger.warn 'Broker.Request.Asset.initialize: Exception raised (InvalidAssetId)'
         end
 
         private

data/lib/alephant/broker/response/base.rb

@@ -1,4 +1,5 @@
 require 'alephant/broker/errors/invalid_cache_key'
+require 'alephant/logger'
 require 'aws-sdk'
 require 'ostruct'
 
@@ -6,6 +7,8 @@ module Alephant
   module Broker
     module Response
       class Base
+        include Logger
+
         attr_reader :content, :headers, :status
 
         STATUS_CODE_MAPPING = {
@@ -19,12 +22,20 @@ module Alephant
           @headers = { "Content-Type" => content_type }
           @status  = status
 
+          log if status !~ /200/
+
           setup
         end
 
         protected
 
         def setup; end
+
+        private
+
+        def log
+          logger.metric(:name => "BrokerResponse#{status}", :unit => "Count", :value => 1)
+        end
       end
     end
   end

data/lib/alephant/broker/version.rb

@@ -1,5 +1,5 @@
 module Alephant
   module Broker
-    VERSION = "3.0.1"
+    VERSION = "3.1.0"
   end
 end

metadata

@@ -1,281 +1,281 @@
 --- !ruby/object:Gem::Specification
 name: alephant-broker
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 3.1.0
 platform: ruby
 authors:
 - Steven Jack
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2014-11-24 00:00:00.000000000 Z
+date: 2014-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rspec
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '>='
-      - !ruby/object:Gem::Version
-        version: '0'
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: rspec
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: rspec-nc
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: rspec-nc
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: guard
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: guard
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: guard-rspec
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: guard-rspec
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: pry
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: pry
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: pry-remote
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: pry-remote
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: pry-nav
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: pry-nav
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: rake-rspec
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.0.2
+        version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: 0.0.2
+  name: rake-rspec
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: bundler
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: 0.0.2
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1.5'
+  name: bundler
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: rake
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.5'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: rake
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: rack-test
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: rack-test
   prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency
-  name: alephant-lookup
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: alephant-lookup
   prerelease: false
   type: :runtime
-- !ruby/object:Gem::Dependency
-  name: alephant-cache
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: alephant-cache
   prerelease: false
   type: :runtime
-- !ruby/object:Gem::Dependency
-  name: alephant-logger
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: alephant-logger
   prerelease: false
   type: :runtime
-- !ruby/object:Gem::Dependency
-  name: alephant-sequencer
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: alephant-sequencer
   prerelease: false
   type: :runtime
-- !ruby/object:Gem::Dependency
-  name: dalli-elasticache
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: dalli-elasticache
   prerelease: false
   type: :runtime
-- !ruby/object:Gem::Dependency
-  name: pmap
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: pmap
   prerelease: false
   type: :runtime
-- !ruby/object:Gem::Dependency
-  name: faraday
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: faraday
   prerelease: false
   type: :runtime
-- !ruby/object:Gem::Dependency
-  name: crimp
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: crimp
   prerelease: false
   type: :runtime
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Brokers requests for alephant components
 email:
 - stevenmajack@gmail.com