mamiya 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5d26f86a476093f14e8d738041bae5acd60d9ad0
-  data.tar.gz: 09dd771694cec321f0b8c50a414671de465291e4
+  metadata.gz: 36bd9c34a4fd00eeaa86b5cf5beee9fb61f73a4a
+  data.tar.gz: 94961d874eed9777d6f281a0e154c15807990c20
 SHA512:
-  metadata.gz: ae5b6693750dfbff6a45d149b6ea16b1e79acae51a0c98e70b89a0217db3061eff318242a8d9637a7420eaefa4ddd41673e19d2fe6855a07a7cc2c9879610e75
-  data.tar.gz: 9d41bc38c61954f5c65609ae5e79fa6b87c402b50c88d71ad6d2577adfe7bdc6de49aac6acf49b842c17296e1a203455ef4e84fca3a376cceba674c70712d059
+  metadata.gz: f9c8f23fc8874eb97f98c2f8591aa4d3f6c48d7d5c9b8593dc383280bda9401a2c5151e2785ccbeac3defbcabd12061c4d73d3fcff47747dac6bd7aa88cd02f5
+  data.tar.gz: bc1b2048ec6f901d3cc98f08c0b02a5e84784c7e98374db56b4d736ced5fb59d64be8c2736df82b610ce8b3c94e3e1501b1b628a29916bb769a5ff9f23f7877d

data/.gitignore

@@ -1,3 +1,4 @@
+.DS_Store
 *.gem
 *.rbc
 .bundle

data/README.md

@@ -16,7 +16,9 @@ Mamiya allows you to deploy using without ssh -- using tarballs, some storages (
 4. Write deploy script for your own
   - how to build package, how to prepare release, how to release, etc.
 5. Build then push package
-6. Time to deploy
+  - `mamiya build`
+  - `mamiya push path/to/package.tar.gz`
+6. Time to deploy: `mamiya client deploy -a app package`
 
 ### Example configuration
 
@@ -33,6 +35,10 @@ This solves such problem by using [Serf](http://www.serfdom.io/) and tarball on
 Also, I'm planning to allow to distribute files before the deploy command. I guess this can skip or shorten
 file transferring phase in deploy.
 
+## In the production
+
+- [Cookpad](https://info.cookpad.com/en) is using Mamiya in production of [cookpad.com](http://cookpad.com).
+
 ## Misc.
 
 - [Scalable Deployments - How we deploy Rails app to 150+ hosts in a minute // Speaker Deck](https://speakerdeck.com/sorah/scalable-deployments-how-we-deploy-rails-app-to-150-plus-hosts-in-a-minute)

data/docs/internal/serf_events.md

@@ -0,0 +1,22 @@
+## Serf Events used in Mamiya
+
+## Master sends
+
+- `mamiya:task`
+  - Enqueue `Task` to task queue.
+  - Payload is in JSON. `payload['task']` is a task name. Payload will be passed in `Agent:;TaskQueue#enqueue` directly.
+
+## Agents send
+
+- `mamiya:task:start`
+- `mamiya:task:finish`
+- `mamiya:task:error`
+
+  - `payload['task']` is a started/finished/errored task.
+  - `payload['error']` is a class name of exception occured.
+
+----
+
+- `mamiya:pkg:remove`
+- `mamiya:prerelease:remove`
+- `mamiya:release:remove`

data/docs/internal/serf_queries.md

@@ -0,0 +1,6 @@
+## Serf queries used in Mamiya
+
+- `mamiya:status`
+  - Returns result of `Mamiya::Agent#status(packages: false)` in JSON.
+- `mamiya:packages`
+  - Returns result of `Mamiya::Agent#existing_packages, existing_prereleases, releases, currents` in JSON.

data/docs/internal/tasks.md

@@ -0,0 +1,36 @@
+## Task
+
+Task is a some procedures that ordered remotely.
+It'll be executed via `mamiya:task` serf event, and `TaskQueue`.
+
+Task implementations are placed in `lib/mamiya/agent/tasks`.
+
+Each task run has a job specification, such as target application and package.
+
+### Base classes
+
+- `abstract` - base
+- `notifyable`
+
+### Tasks
+
+- `fetch` - Fetch specified package from storage
+- `prepare` - Unpack fetched package then prepare it
+  - if specified package hasn't fetched, enqueues `fetch` task with chain
+- `switch`
+
+- `clean`
+- `ping`
+
+### Chaining
+
+- task can have chain in `_chain` key
+- `_chain` should be an Array (if specified,) like `['prepare', 'switch']`.
+- When task run with `_chain` finished, Task named the first element of `_chain` will be enqueued into `TaskQueue`.
+
+#### Example
+
+1. User enqueue `{"_chain": ["bar"], "task": "foo", "test": 42}`
+2. Task `{"_chain": ["bar"], "task": "foo", "test": 42}` run
+3. Task `{"_chain": ["bar"], "task": "foo", "test": 42}` enqueues `{"task": "bar", "test": 42}`
+4. Task `{"task": "bar", "test": 42}` run

data/lib/mamiya/agent.rb

@@ -15,6 +15,7 @@ require 'mamiya/agent/tasks/fetch'
 require 'mamiya/agent/tasks/prepare'
 require 'mamiya/agent/tasks/clean'
 require 'mamiya/agent/tasks/switch'
+require 'mamiya/agent/tasks/ping'
 
 require 'mamiya/agent/handlers/task'
 require 'mamiya/agent/actions'
@@ -44,6 +45,7 @@ module Mamiya
         Mamiya::Agent::Tasks::Prepare,
         Mamiya::Agent::Tasks::Clean,
         Mamiya::Agent::Tasks::Switch,
+        Mamiya::Agent::Tasks::Ping,
       ])
     end
 

data/lib/mamiya/agent/actions.rb

@@ -18,8 +18,12 @@ module Mamiya
         order_task('prepare', app: application, pkg: package, labels: labels)
       end
 
-      def switch(application, package, labels: nil, no_release: false)
-        order_task('switch', app: application, pkg: package, labels: labels, no_release: no_release)
+      def switch(application, package, labels: nil, no_release: false, do_release: false)
+        order_task('switch', app: application, pkg: package, labels: labels, no_release: no_release, do_release: do_release)
+      end
+
+      def ping
+        order_task('ping')
       end
     end
   end

data/lib/mamiya/agent/tasks/ping.rb

@@ -0,0 +1,19 @@
+require 'mamiya/agent'
+require 'mamiya/agent/tasks/abstract'
+
+module Mamiya
+  class Agent
+    module Tasks
+      class Ping < Abstract
+        def run
+          logger.info "Responding ping: #{task.inspect}"
+
+          agent.trigger('pong', coalesce: false,
+            at: Time.now.to_i,
+            id: self.task_id,
+          )
+        end
+      end
+    end
+  end
+end

data/lib/mamiya/agent/tasks/switch.rb

@@ -15,7 +15,8 @@ module Mamiya
         end
 
         def check
-          return true if prerelease_path.exist?
+          return true if ignore_incompletion? && (prerelease_path.exist? || release_path.exist?)
+          return true if prerelease_prepared?
           return true if release_prepared?
 
           unless package_path.exist?
@@ -28,7 +29,7 @@ module Mamiya
             return false
           end
 
-          unless prerelease_path.exist?
+          unless prerelease_prepared?
             new_chain = ['switch'] + (task['_chain'] || [])
             logger.info "Package not prepared, enqueueing prepare task with #{new_chain.inspect}"
             task_queue.enqueue(
@@ -43,10 +44,12 @@ module Mamiya
 
         def run
           case
-          when prerelease_path.exist? && release_path.exist? && !release_path.join('.mamiya.prepared').exist?
+          when prerelease_prepared? && release_path.exist? && !release_path.join('.mamiya.prepared').exist?
             logger.info "Removing existing release (not prepared)"
             FileUtils.remove_entry_secure release_path
-          when !prerelease_path.exist? && prelease_path.exist? && !release_path.join('.mamiya.prepared').exist?
+          when ignore_incompletion? && (prerelease_path.exist? || release_path.exist?)
+            logger.warn "Using incomplete release or prereleases"
+          when !prerelease_prepared? && prerelease_path.exist? && !release_path.join('.mamiya.prepared').exist?
             # this condition may be a bug
             logger.error "Existing release is not prepared but prerelease doesn't exist"
             raise PrereleaseMissing, "Existing release is not prepared but prerelease doesn't exist"
@@ -73,6 +76,10 @@ module Mamiya
           task['pkg']
         end
 
+        def ignore_incompletion?
+          task['allow_incomplete'] || task['allow_incompletion'] || task['incomplete']
+        end
+
         def release_name
           task['release'] || package
         end
@@ -105,6 +112,10 @@ module Mamiya
           release_path.exist? && release_path.join('.mamiya.prepared').exist?
         end
 
+        def prerelease_prepared?
+          prerelease_path.exist? && prerelease_path.join('.mamiya.prepared').exist?
+        end
+
         def labels
           @labels ||= agent.labels
         end
@@ -116,7 +127,8 @@ module Mamiya
             logger: logger,
             config: config,
             labels: agent.labels,
-            no_release: !!task['no_release']
+            no_release: !!task['no_release'],
+            do_release: !!task['do_release'],
           )
         end
       end

data/lib/mamiya/cli/client.rb

@@ -163,12 +163,7 @@ not distributed: #{dist['not_distributed_count']} agents
       method_option :labels, type: :string
       method_option :no_release, type: :boolean, default: false
       def switch(package)
-        params = {no_release: options[:no_release]}
-        if options[:labels]
-          params[:labels] = Mamiya::Util::LabelMatcher.parse_string_expr(options[:labels])
-        end
-
-        p master_post("/packages/#{application}/#{package}/switch", params.merge(type: :json))
+        switch_(package, no_release: options[:no_release])
       end
 
       desc "refresh", "order refreshing agent status"
@@ -181,11 +176,15 @@ not distributed: #{dist['not_distributed_count']} agents
       method_option :no_release, type: :boolean, default: false
       method_option :config, aliases: '-C', type: :string
       method_option :no_switch, type: :boolean, default: false
+      method_option :synced_release, type: :boolean, default: false
       def deploy(package)
         @deploy_exception = nil
+        synced_release = options[:synced_release] || (config && config.synced_release)
+
         # TODO: move this run on master node side
         puts "=> Deploying #{application}/#{package}"
         puts " * onto agents which labeled: #{options[:labels].inspect}" if options[:labels] && !options[:labels].empty?
+        puts " * releasing will be synced in all agents" if synced_release
 
         show_package(package)
 
@@ -220,10 +219,11 @@ not distributed: #{dist['not_distributed_count']} agents
         end
 
         ###
+        #
 
         unless options[:no_switch]
           puts "=> Switching..."
-          switch(package)
+          switch_(package, no_release: synced_release)
 
           puts " * Wait until switch"
           puts ""
@@ -233,6 +233,13 @@ not distributed: #{dist['not_distributed_count']} agents
             break if s['participants_count'] == s['switch']['done'].size
             sleep 2
           end
+
+          if synced_release
+            puts "=> Releasing..."
+            switch_(package, do_release: true)
+
+            puts " * due to current implementation's limitation, releasing will be untracked."
+          end
         end
       rescue Exception => e
         @deploy_exception = e
@@ -349,6 +356,15 @@ not distributed: #{dist['not_distributed_count']} agents
         raise e
       end
 
+      def switch_(package, no_release: false, do_release: false)
+        params = {no_release: no_release, do_release: do_release}
+        if options[:labels]
+          params[:labels] = Mamiya::Util::LabelMatcher.parse_string_expr(options[:labels])
+        end
+
+        p master_post("/packages/#{application}/#{package}/switch", params.merge(type: :json))
+      end
+
       def master_http
         url = master_url
         Net::HTTP.new(url.host, url.port).tap do |http|

data/lib/mamiya/configuration.rb

@@ -26,6 +26,7 @@ module Mamiya
     # master
     set_default :master, {monitor: {refresh_interval: nil}} # TODO: don't nest
     set_default :web, {port: 7761, bind: '0.0.0.0', environment: :development} # TODO: IPv6
+    set_default :synced_release, false
 
     add_hook :labels, chain: true
 

data/lib/mamiya/master/agent_monitor.rb

@@ -112,7 +112,6 @@ module Mamiya
       end
 
       def refresh(**kwargs)
-        # TODO: lock
         logger.debug "Refreshing..."
 
         new_agents = {}

data/lib/mamiya/master/web.rb

@@ -110,7 +110,7 @@ module Mamiya
       post '/packages/:application/:package/switch' do
         if storage(params[:application]).meta(params[:package])
           status 204
-          master.switch(params[:application], params[:package], labels: params['labels'], no_release: !!(params['no_release'] || params[:no_release]))
+          master.switch(params[:application], params[:package], labels: params['labels'], no_release: !!(params['no_release'] || params[:no_release]), do_release: !!(params['do_release'] || params[:do_release]))
         else
           status 404
           content_type :json

data/lib/mamiya/steps/build.rb

@@ -20,8 +20,6 @@ module Mamiya
           raise ApplicationNotSpecified, ":application should be specified in your script file"
         end
 
-        logger.info "Initiating package build"
-
         run_before_build
         run_prepare_build
         run_build
@@ -48,7 +46,9 @@ module Mamiya
         logger.info "Running script.after_build"
         script.after_build[@exception]
 
-        logger.info "DONE!" unless @exception
+        unless @exception
+          logger.info "DONE: #{package_name} built at #{package.path}"
+        end
       end
 
       private
@@ -72,7 +72,7 @@ module Mamiya
         begin
           # Using without block because chdir in block shows warning
           Dir.chdir(script.build_from)
-          logger.info "Running script.build ..."
+          logger.info "Running script.build"
           logger.debug "pwd=#{Dir.pwd}"
           script.build[]
         ensure
@@ -82,7 +82,7 @@ module Mamiya
 
       def copy_deploy_scripts
         # XXX: TODO: move to another class?
-        logger.info "Copying script files..."
+        logger.info "Copying script files"
 
         if script_dest.exist?
           logger.warn "Removing existing .mamiya.script"
@@ -90,7 +90,7 @@ module Mamiya
         end
         script_dest.mkdir
 
-        logger.info "- #{script_file} -> #{script_dest}"
+        logger.debug "- #{script_file} -> #{script_dest}"
         FileUtils.cp script_file, script_dest
 
         if script.script_additionals
@@ -98,7 +98,7 @@ module Mamiya
           script.script_additionals.each do |additional|
             src = script_dir.join(additional)
             dst = script_dest.join(additional)
-            logger.info "- #{src} -> #{dst}"
+            logger.debug "- #{src} -> #{dst}"
             FileUtils.mkdir_p dst.dirname
             FileUtils.cp_r src, dst
           end
@@ -114,7 +114,7 @@ module Mamiya
       end
 
       def build_package
-        logger.info "Packaging to: #{package.path}"
+        logger.debug "Packaging to: #{package.path}"
         logger.debug "meta=#{package.meta.inspect}"
         package.build!(script.build_from,
            exclude_from_package: script.exclude_from_package || [],
@@ -131,7 +131,7 @@ module Mamiya
               [Time.now.strftime("%Y%m%d%H%M%S"), script.application]
             ].join('-')
           }
-          logger.info "Package name determined: #{name}"
+          logger.debug "Package name determined: #{name}"
           name
         end
       end

data/lib/mamiya/version.rb

@@ -1,3 +1,3 @@
 module Mamiya
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/spec/agent/actions_spec.rb

@@ -57,26 +57,41 @@ describe Mamiya::Agent::Actions do
 
   describe "#switch" do
     it "sends switch request" do
-      expect(agent).to receive(:trigger).with('task', task: 'switch', app: 'myapp', pkg: 'mypkg', coalesce: false, no_release: false)
+      expect(agent).to receive(:trigger).with('task', task: 'switch', app: 'myapp', pkg: 'mypkg', coalesce: false, no_release: false, do_release: false)
 
       agent.switch('myapp', 'mypkg')
     end
 
     context "with no_release" do
       it "sends switch request" do
-        expect(agent).to receive(:trigger).with('task', task: 'switch', app: 'myapp', pkg: 'mypkg', coalesce: false, no_release: true)
+        expect(agent).to receive(:trigger).with('task', task: 'switch', app: 'myapp', pkg: 'mypkg', coalesce: false, no_release: true, do_release: false)
 
         agent.switch('myapp', 'mypkg', no_release: true)
       end
+    end
 
+    context "with no_release" do
+      it "sends switch request" do
+        expect(agent).to receive(:trigger).with('task', task: 'switch', app: 'myapp', pkg: 'mypkg', coalesce: false, no_release: false, do_release: true)
+
+        agent.switch('myapp', 'mypkg', do_release: true)
+      end
     end
 
     context "with labels" do
       it "adds _labels on task" do
-        expect(agent).to receive(:trigger).with('task', task: 'switch', app: 'myapp', pkg: 'mypkg', _labels: ['foo'], coalesce: false, no_release: false)
+        expect(agent).to receive(:trigger).with('task', task: 'switch', app: 'myapp', pkg: 'mypkg', _labels: ['foo'], coalesce: false, no_release: false, do_release: false)
 
         agent.switch('myapp', 'mypkg', labels: ['foo'])
       end
     end
   end
+
+  describe "#ping" do
+    it "pings cluster" do
+      expect(agent).to receive(:trigger).with('task', task: 'ping', coalesce: false)
+
+      agent.ping
+    end
+  end
 end

data/spec/agent/tasks/ping_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+require 'mamiya/agent/tasks/abstract'
+require 'mamiya/agent/tasks/ping'
+require 'mamiya/configuration'
+
+describe Mamiya::Agent::Tasks::Ping do
+  let(:config) do
+    Mamiya::Configuration.new
+  end
+
+  let(:agent) { double('agent', config: config) }
+  let(:task_queue) { double('task_queue') }
+
+  subject(:task) { described_class.new(task_queue, {}, agent: agent, raise_error: true) }
+
+  it 'inherits abstract task' do
+    expect(described_class.ancestors).to include(Mamiya::Agent::Tasks::Abstract)
+  end
+
+  describe "#execute" do
+    let(:now) { Time.now }
+
+    before do
+      allow(Time).to receive(:now).and_return(now)
+    end
+
+    it "responds with it's task_id" do
+      expect(agent).to receive(:trigger).with('pong', at: now.to_i, id: task.task_id, coalesce: false)
+
+      task.execute
+    end
+  end
+end

data/spec/agent/tasks/switch_spec.rb

@@ -94,6 +94,65 @@ describe Mamiya::Agent::Tasks::Switch do
       end
     end
 
+    context "when package fetched but incompletely prepared" do
+      before do
+        File.write packages_app_dir.join('mypkg.tar.gz'), "\n"
+        File.write packages_app_dir.join('mypkg.json'), "{}\n"
+
+        # no .mamiya.prepare
+        prerelease = prereleases_app_dir.join('mypkg').tap(&:mkpath)
+        File.write prerelease.join('hello'), "hola\n"
+      end
+
+      context "without task.incomplete" do
+        before do
+          expect(Mamiya::Steps::Switch).not_to receive(:new)
+        end
+
+        it "enqueues prepare task and finish" do
+          expect(task_queue).to receive(:enqueue).with(
+            :prepare, job.merge('task' => 'switch', '_chain' => ['switch'])
+          )
+
+          task.execute
+        end
+
+        context "with _chain-ed job" do
+          let(:job) { {'app' => 'myapp', 'pkg' => 'mypkg', '_chain' => ['next']} }
+
+          it "enqueues prepare task and finish" do
+            expect(task_queue).to receive(:enqueue).with(
+              :prepare, job.merge('task' => 'switch', '_chain' => ['switch', 'next'])
+            )
+
+            task.execute
+          end
+        end
+      end
+
+      context "with task.incomplete=true" do
+        let(:job) { {'app' => 'myapp', 'pkg' => 'mypkg', 'incomplete' => true} }
+
+        before do
+          allow(Mamiya::Steps::Switch).to receive(:new).with(
+            target: deploy_to.join('releases', 'mypkg'),
+            labels: [:foo, :bar],
+            no_release: false,
+            do_release: false,
+            config: config,
+            logger: task.logger,
+          ).and_return(switch_step)
+        end
+
+        it "calls switch step" do
+          expect(switch_step).to receive(:run!) do
+            expect(deploy_to.join('releases', 'mypkg', 'hello')).to be_exist
+          end
+          task.execute
+        end
+      end
+    end
+
     context "when package prepared" do
       before do
         File.write packages_app_dir.join('mypkg.tar.gz'), "\n"
@@ -107,6 +166,7 @@ describe Mamiya::Agent::Tasks::Switch do
           target: deploy_to.join('releases', 'mypkg'),
           labels: [:foo, :bar],
           no_release: false,
+          do_release: false,
           config: config,
           logger: task.logger,
         ).and_return(switch_step)
@@ -162,6 +222,26 @@ describe Mamiya::Agent::Tasks::Switch do
             target: deploy_to.join('releases', 'mypkg'),
             labels: [:foo, :bar],
             no_release: true,
+            do_release: false,
+            config: config,
+            logger: task.logger,
+          ).and_return(switch_step)
+
+          expect(switch_step).to receive(:run!)
+
+          task.execute
+        end
+      end
+
+      context "with do_release" do
+        let(:job) { {'app' => 'myapp', 'pkg' => 'mypkg', 'do_release' => true} }
+
+        it "calls switch step with no_release" do
+          expect(Mamiya::Steps::Switch).to receive(:new).with(
+            target: deploy_to.join('releases', 'mypkg'),
+            labels: [:foo, :bar],
+            no_release: false,
+            do_release: true,
             config: config,
             logger: task.logger,
           ).and_return(switch_step)

data/spec/master/web_spec.rb

@@ -219,7 +219,7 @@ describe Mamiya::Master::Web do
 
   describe "POST /packages/:application/:package/switch" do
     it "dispatchs switch request" do
-      expect(master).to receive(:switch).with('myapp', 'mypackage', labels: nil, no_release: false)
+      expect(master).to receive(:switch).with('myapp', 'mypackage', labels: nil, no_release: false, do_release: false)
 
       post '/packages/myapp/mypackage/switch'
 
@@ -228,7 +228,7 @@ describe Mamiya::Master::Web do
 
     context "with no_release" do
       it "dispatchs switch request" do
-        expect(master).to receive(:switch).with('myapp', 'mypackage', labels: nil, no_release: true)
+        expect(master).to receive(:switch).with('myapp', 'mypackage', labels: nil, no_release: true, do_release: false)
 
         post '/packages/myapp/mypackage/switch',
           {'no_release' => true}.to_json,
@@ -240,7 +240,7 @@ describe Mamiya::Master::Web do
 
     context "with labels" do
       it "dispatchs prepare request with labels" do
-        expect(master).to receive(:switch).with('myapp', 'mypackage', labels: ['foo'], no_release: false)
+        expect(master).to receive(:switch).with('myapp', 'mypackage', labels: ['foo'], no_release: false, do_release: false)
 
         post '/packages/myapp/mypackage/switch',
           {'labels' =>  ["foo"]}.to_json,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mamiya
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Shota Fukumori (sora_h)
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-26 00:00:00.000000000 Z
+date: 2014-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -180,6 +180,9 @@ files:
 - README.md
 - Rakefile
 - bin/mamiya
+- docs/internal/serf_events.md
+- docs/internal/serf_queries.md
+- docs/internal/tasks.md
 - docs/upgrading.md
 - example/.gitignore
 - example/Procfile
@@ -198,6 +201,7 @@ files:
 - lib/mamiya/agent/tasks/clean.rb
 - lib/mamiya/agent/tasks/fetch.rb
 - lib/mamiya/agent/tasks/notifyable.rb
+- lib/mamiya/agent/tasks/ping.rb
 - lib/mamiya/agent/tasks/prepare.rb
 - lib/mamiya/agent/tasks/switch.rb
 - lib/mamiya/cli.rb
@@ -238,6 +242,7 @@ files:
 - spec/agent/tasks/clean_spec.rb
 - spec/agent/tasks/fetch_spec.rb
 - spec/agent/tasks/notifyable_spec.rb
+- spec/agent/tasks/ping_spec.rb
 - spec/agent/tasks/prepare_spec.rb
 - spec/agent/tasks/switch_spec.rb
 - spec/agent_spec.rb
@@ -292,7 +297,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.1
 signing_key: 
 specification_version: 4
 summary: Fast deploy tool using tarballs and serf
@@ -304,6 +309,7 @@ test_files:
 - spec/agent/tasks/clean_spec.rb
 - spec/agent/tasks/fetch_spec.rb
 - spec/agent/tasks/notifyable_spec.rb
+- spec/agent/tasks/ping_spec.rb
 - spec/agent/tasks/prepare_spec.rb
 - spec/agent/tasks/switch_spec.rb
 - spec/agent_spec.rb