puma-cloudwatch 0.3.1 → 0.4.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 597cac59de90ceaab9f7824dc362138439fff16affcd0166fcfac7d286c0c53e
-  data.tar.gz: 714fe4f86113cdf8bdb5ff55e52aea97477edf0880ef72a2ba0d51d159c2cf69
+  metadata.gz: 8e2897967164e24f709d8c976d1fd9a5b741a01d020c2da8dcb191a9d4fe43ab
+  data.tar.gz: bdb6f674505cd71c8423dd343c77a8521715425de02c2a6b418c1b9575e10d9e
 SHA512:
-  metadata.gz: c298cd269d1e8aa81f204c9cb8afea0576c36775ccb4117ee7e967ba06913387bbc1d958d21c30fbf637eab6564e217ed276d6c6256872bc20ad30e29b0bd14d
-  data.tar.gz: '09cd67be3c688102cedca2d80bd0b13d8184340ed16a22388b6f8c5f7a0d7e0c99cad9fb0f9db889e8229e2987819c9c9863e662811edab687d0f33fea8f231e'
+  metadata.gz: '0658266b02fa7ffa618b94bdc956c12b6536cb2eea17dcf2b7a4c39452b7adf1d96bc2e83544568739bc998ee9c76cf90416b9db3d2b174bcf9d759a0b07266a'
+  data.tar.gz: de176da8000aec589ed539b3f0f6538a602344aab633c5dc7216307e21662f31405361525de134f0169288312af414c79b5065b14ff16a71bf257047ab9bc69d

data/CHANGELOG.md

@@ -3,6 +3,21 @@
 All notable changes to this project will be documented in this file.
 This project *loosely tries* to adhere to [Semantic Versioning](http://semver.org/).
 
+## [0.4.4]
+- #4 fix syntax error in older versions of ruby
+
+## [0.4.3]
+- #3 Eliminate the need to add Array#sum
+
+## [0.4.2]
+- #2 keep looper running on adversities
+
+## [0.4.1]
+- #1 empty results breaks cloudwatch put_metric_data
+
+## [0.4.0]
+- improve AWS IAM setup and permission errors so Thread loop doesnt stop
+
 ## [0.3.1]
 - fix PUMA_CLOUDWATCH_MUTE_START_MESSAGE var
 

data/README.md

@@ -2,13 +2,15 @@
 
 [![Gem Version](https://badge.fury.io/rb/puma-cloudwatch.svg)](https://badge.fury.io/rb/puma-cloudwatch)
 
+[![BoltOps Badge](https://img.boltops.com/boltops/badges/boltops-badge.png)](https://www.boltops.com)
+
 A [puma](https://puma.io) plugin that sends puma stats to CloudWatch.
 
 ## Usage
 
 **Important**: To enable the plugin to send metrics to CloudWatch you must set the `PUMA_CLOUDWATCH_ENABLED` env variable. This allows you to send only metrics on configured servers and not unintentionally send them locally.
 
-It also strongly encourage to sent the `PUMA_CLOUDWATCH_DIMENSION_VALUE` env variable to include your application name. For example, if your application is named "demo-web", this would be a good value to use:
+It also strongly encourage to set the `PUMA_CLOUDWATCH_DIMENSION_VALUE` env variable to include your application name. For example, if your application is named "demo-web", this would be a good value to use:
 
     PUMA_CLOUDWATCH_DIMENSION_VALUE=demo-web-puma
 

data/lib/puma_cloudwatch/core_ext.rb

@@ -2,6 +2,5 @@
 major, minor, _ = RUBY_VERSION.split('.')
 if major == '2' && minor == '3'
   require "puma_cloudwatch/core_ext/thread"
-  require "puma_cloudwatch/core_ext/array"
   require "pp"
 end

data/lib/puma_cloudwatch/metrics/looper.rb

@@ -32,10 +32,14 @@ class PumaCloudwatch::Metrics
   private
     def perform
       loop do
-        stats = Fetcher.new(@options).call
-        results = Parser.new(stats).call
-        Sender.new(results).call
-        sleep @frequency
+        begin
+          stats = Fetcher.new(@options).call
+          results = Parser.new(stats).call
+          Sender.new(results).call unless results.empty?
+          sleep @frequency
+        rescue Exception => e
+          puts "Error reached top of looper: #{e.message} (#{e.class})"
+        end
       end
     end
 

data/lib/puma_cloudwatch/metrics/sender.rb

@@ -57,7 +57,7 @@ class PumaCloudwatch::Metrics
             dimensions: dimensions,
             statistic_values: {
               sample_count: values.length,
-              sum: values.sum,
+              sum: values.inject(0) { |sum, el| sum += el },
               minimum: values.min,
               maximum: values.max
             }
@@ -84,7 +84,15 @@ class PumaCloudwatch::Metrics
       end
 
       if enabled?
-        cloudwatch.put_metric_data(params)
+        begin
+          cloudwatch.put_metric_data(params)
+        rescue Aws::CloudWatch::Errors::AccessDenied => e
+          puts "WARN: #{e.class} #{e.message}"
+          puts "Unable to send metrics to CloudWatch"
+        rescue PumaCloudwatch::Error => e
+          puts "WARN: #{e.class} #{e.message}"
+          puts "Unable to send metrics to CloudWatch"
+        end
       end
     end
 
@@ -94,6 +102,12 @@ class PumaCloudwatch::Metrics
 
     def cloudwatch
       @cloudwatch ||= Aws::CloudWatch::Client.new
+    rescue Aws::Errors::MissingRegionError => e
+      # Happens when:
+      #   1. ~/.aws/config is not also setup locally
+      #   2. On EC2 instance when AWS_REGION not set
+      puts "WARN: #{e.class} #{e.message}"
+      raise PumaCloudwatch::Error.new(e.message)
     end
   end
 end

data/lib/puma_cloudwatch/version.rb

@@ -1,3 +1,3 @@
 module PumaCloudwatch
-  VERSION = "0.3.1"
+  VERSION = "0.4.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puma-cloudwatch
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.4
 platform: ruby
 authors:
 - Tung Nguyen
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-09-18 00:00:00.000000000 Z
+date: 2020-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-cloudwatch
@@ -87,7 +87,6 @@ files:
 - lib/puma/plugin/cloudwatch.rb
 - lib/puma_cloudwatch.rb
 - lib/puma_cloudwatch/core_ext.rb
-- lib/puma_cloudwatch/core_ext/array.rb
 - lib/puma_cloudwatch/core_ext/thread.rb
 - lib/puma_cloudwatch/metrics.rb
 - lib/puma_cloudwatch/metrics/fetcher.rb
@@ -115,7 +114,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Puma plugin sends puma stats to CloudWatch

data/lib/puma_cloudwatch/core_ext/array.rb

@@ -1,7 +0,0 @@
-class Array
-  # Note: Array#sum only available in ruby 2.4+
-  # Thanks: http://www.viarails.net/q/How-to-sum-an-array-of-numbers-in-Ruby
-  def sum
-    inject(0) {|sum, i|  sum + i }
-  end
-end