perf_check 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f8cdfcd06c215cfa05e4267afa28abf9f8d8efba
-  data.tar.gz: a895cac67e5622ea31c997239f7371aac08fce8a
+  metadata.gz: ad9c18545aaee3bdcba0a6bb28eb307b341ba78a
+  data.tar.gz: 89809449598f93009f78b0c353ab432f7da341f5
 SHA512:
-  metadata.gz: a9c4c040e1d5e27d2f21b59f527efe287ebdfdacd794c447ada57c08c85c725406695f6759c2a305474875688fb7f6c6f13c4f0e10db4fc2094cc9e984564ea6
-  data.tar.gz: eba0557a4d74c324892cf01a9a4c2a5abbf6f6b0c7cb78df3be9289f1d7fa0342900c43569b101d55d8b9e09b31ca661856275817794ac0daa06588377bb6dbc
+  metadata.gz: 51b79c492435781ae275d452045cc34ce6d7468b0ce756a43b1ead8692a3c15a42ca6f718854718aea189c4f9aafaf089da281767d44f34eb89a441ffddb1e41
+  data.tar.gz: 6963d6cf9bf01766168a87fc3962cbdd85b9a42b46b63128c9f9defb18d2c206219d0eb359c0da8c6dc3ef066c268023ceb2a3dba7f810f245824cf4367caac8

data/lib/perf_check.rb

@@ -59,10 +59,10 @@ class PerfCheck
       test.cookie = options.cookie
 
       if options.diff
-        logger.info("Issuing #{test.resource}")
+        PerfCheck.logger.info("Issuing #{test.resource}")
       else
-        logger.info ''
-        logger.info("Benchmarking #{test.resource}:")
+        PerfCheck.logger.info ''
+        PerfCheck.logger.info("Benchmarking #{test.resource}:")
       end
 
       test.run(server, options)

data/lib/perf_check/callbacks.rb

@@ -17,9 +17,9 @@ class PerfCheck
   def self.before_start_callbacks
     (@before_start_callbacks || []) + [
       proc {
-        logger.info("=" * 77)
-        logger.info("PERRRRF CHERRRK! Grab a ☕️  and don't touch your working tree (we automate git)")
-        logger.info("=" * 77)
+        PerfCheck.logger.info("=" * 77)
+        PerfCheck.logger.info("PERRRRF CHERRRK! Grab a ☕️  and don't touch your working tree (we automate git)")
+        PerfCheck.logger.info("=" * 77)
       }
     ]
   end

data/lib/perf_check/config.rb

@@ -3,7 +3,7 @@ require 'optparse'
 class PerfCheck
   def self.config
     @config ||= OpenStruct.new(
-      number_of_requests: 10,
+      number_of_requests: 20,
       reference: 'master',
       cookie: nil,
       headers: {},
@@ -25,7 +25,7 @@ class PerfCheck
 
     opts.separator "\nBenchmark options:"
     opts.on('--requests N', '-n',
-            'Use N requests in benchmark, defaults to 10') do |n|
+            'Use N requests in benchmark, defaults to 20') do |n|
       config.number_of_requests = n.to_i
     end
 
@@ -35,8 +35,7 @@ class PerfCheck
     end
 
     opts.on('--quick', '-q',
-            'Fire off 5 requests just on this branch, no comparison with master') do
-      config.number_of_requests = 5
+            'Fire off 20 requests just on this branch (no comparison with master)') do
       config.reference = nil
     end
 

data/lib/perf_check/git.rb

@@ -11,7 +11,7 @@ class PerfCheck
     def self.checkout_reference(reference='master')
       checkout(reference)
       at_exit do
-        logger.info ''
+        PerfCheck.logger.info ''
         Git.checkout_current_branch(false)
       end
     end
@@ -21,11 +21,11 @@ class PerfCheck
     end
 
     def self.checkout(branch, bundle=true)
-      logger.info("Checking out #{branch} and bundling... ")
+      PerfCheck.logger.info("Checking out #{branch} and bundling... ")
       `git checkout #{branch} --quiet`
 
       unless $?.success?
-        logger.fatal("Problem with git checkout! Bailing...") && abort
+        PerfCheck.logger.fatal("Problem with git checkout! Bailing...") && abort
       end
 
       `git submodule update --quiet`
@@ -33,18 +33,18 @@ class PerfCheck
       if bundle
         Bundler.with_clean_env{ `bundle` }
         unless $?.success?
-          logger.fatal("Problem bundling! Bailing...") && abort
+          PerfCheck.logger.fatal("Problem bundling! Bailing...") && abort
         end
       end
     end
 
     def self.stash_if_needed
       if anything_to_stash?
-        logger.info("Stashing your changes... ")
+        PerfCheck.logger.info("Stashing your changes... ")
         system('git stash -q >/dev/null')
 
         unless $?.success?
-          logger.fatal("Problem with git stash! Bailing...") && abort
+          PerfCheck.logger.fatal("Problem with git stash! Bailing...") && abort
         end
 
         at_exit do
@@ -60,11 +60,11 @@ class PerfCheck
     end
 
     def self.pop
-      logger.info("Git stash applying...")
+      PerfCheck.logger.info("Git stash applying...")
       system('git stash pop -q')
 
       unless $?.success?
-        logger.fatal("Problem with git stash! Bailing...") && abort
+        PerfCheck.logger.fatal("Problem with git stash! Bailing...") && abort
       end
     end
 

data/lib/perf_check/logger.rb

@@ -12,8 +12,3 @@ class PerfCheck
 
   def logger; self.class.logger; end
 end
-
-class Object
-  def self.logger; PerfCheck.logger; end
-  def logger; PerfCheck.logger; end
-end

data/lib/perf_check/server.rb

@@ -108,10 +108,10 @@ class PerfCheck
 
     def restart
       if !@running
-        logger.info("starting rails...")
+        PerfCheck.logger.info("starting rails...")
         start
       else
-        logger.info("re-starting rails...")
+        PerfCheck.logger.info("re-starting rails...")
         exit
         start
       end

data/lib/perf_check/test_case.rb

@@ -20,7 +20,7 @@ class PerfCheck
 
     def run(server, options)
       unless options.diff
-        logger.info("\t"+['request', 'latency', 'server rss', 'status', 'queries', 'profiler data'].map(&:underline).join("   "))
+        PerfCheck.logger.info("\t"+['request', 'latency', 'server rss', 'status', 'queries', 'profiler data'].map(&:underline).join("   "))
       end
 
       profiles = (@context == :reference) ? reference_profiles : this_profiles
@@ -40,9 +40,9 @@ class PerfCheck
               error_dump.write(profile.response_body)
             end
             error = sprintf("\t%2i:\tFAILED! (HTTP %d)", i, profile.response_code)
-            logger.fatal(error.red.bold)
-            logger.fatal("\t   The server responded with a non-2xx status for this request.")
-            logger.fatal("\t   The response has been written to tmp/perf_check/failed_request.html")
+            PerfCheck.logger.fatal(error.red.bold)
+            PerfCheck.logger.fatal("\t   The server responded with a non-2xx status for this request.")
+            PerfCheck.logger.fatal("\t   The response has been written to tmp/perf_check/failed_request.html")
             abort
           end
         end
@@ -65,13 +65,13 @@ class PerfCheck
           row = sprintf("\t%2i:\t  %.1fms   %4dMB\t  %s\t   %s\t   %s",
                         i, profile.latency, profile.server_memory,
                         profile.response_code, profile.query_count, profile.profile_url)
-          logger.info(row)
+          PerfCheck.logger.info(row)
         end
 
         profiles << profile
       end
 
-      logger.info '' unless options.diff # pretty!
+      PerfCheck.logger.info '' unless options.diff # pretty!
     end
 
     def this_latency

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: perf_check
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - rubytune
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-03 00:00:00.000000000 Z
+date: 2016-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize