shipit-engine 0.16.0 → 0.17.0

data/test/models/commits_test.rb

@@ -6,6 +6,7 @@ module Shipit
       @stack = shipit_stacks(:shipit)
       @pr = @stack.commits.new
       @pr.message = "Merge pull request #31 from Shopify/improve-polling\n\nSeveral improvements to polling"
+      @stack.reload
       @commit = shipit_commits(:first)
     end
 
@@ -209,6 +210,21 @@ module Shipit
       end
     end
 
+    test "#creating a commit for new stack updates last_deployed_at to nil" do
+      walrus = shipit_users(:walrus)
+      stack = shipit_stacks(:undeployed_stack)
+      stack.commits.create!(
+        author: walrus,
+        committer: walrus,
+        sha: "ab12",
+        authored_at: DateTime.now,
+        committed_at: DateTime.now,
+        message: "more fish!",
+      )
+      stack.reload
+      assert_nil stack.last_deployed_at
+    end
+
     test ".by_sha! raises if the sha prefix matches multiple commits" do
       clone = Commit.new(@commit.attributes.except('id'))
       clone.sha[8..-1] = 'abc12'
@@ -278,6 +294,12 @@ module Shipit
       refute_predicate shipit_commits(:fifth), :deployable?
     end
 
+    test "#deployable? is false if commit is locked" do
+      commit = shipit_commits(:cyclimse_first)
+      commit.update!(locked: true)
+      refute_predicate commit, :deployable?
+    end
+
     test "#deployable? is false if a required status is missing" do
       commit = shipit_commits(:cyclimse_first)
       commit.stack.stubs(:required_statuses).returns(%w(ci/very-important))
@@ -480,6 +502,55 @@ module Shipit
       assert_nil commit.pull_request
     end
 
+    test "#revert? returns false if the message doesn't follow the revert convention" do
+      commit = Commit.new(message: "Revert stuff")
+      refute_predicate commit, :revert?
+    end
+
+    test "#revert? returns true for commits reverted by GitHub" do
+      commit = Commit.new(
+        message: "Merge pull request #17 from Shopify/revert-16\n\nRevert \"Create README.md\"",
+      )
+      assert_predicate commit, :revert?
+    end
+
+    test "#revert? returns true for commits reverted from CLI" do
+      commit = Commit.new(
+        message: "Revert \"Super Feature\"\n\nThis reverts commit 49430d5091abc34f2c576c23ebf369ec7094d8aa.",
+      )
+      assert_predicate commit, :revert?
+    end
+
+    test "#revert_of? works with pull requests reverted on GitHub" do
+      commit = Commit.new(
+        message: "Merge pull request #16 from byroot/casperisfine-patch-1\n\nCreate README.md",
+      )
+      revert = Commit.new(
+        message: "Merge pull request #17 from Shopify/revert-16\n\nRevert \"Create README.md\"",
+      )
+      assert revert.revert_of?(commit)
+    end
+
+    test "#revert_of? works with commits reverted from CLI" do
+      commit = Commit.new(
+        message: "Create README.md",
+      )
+      revert = Commit.new(
+        message: "Revert \"Create README.md\"\n\nThis reverts commit 49430d5091abc34f2c576c23ebf369ec7094d8aa.",
+      )
+      assert revert.revert_of?(commit)
+    end
+
+    test "#revert_of? works with pull requests reverted from CLI" do
+      commit = Commit.new(
+        message: "Merge pull request #19 from byroot/casperisfine-patch-1\n\nUpdate README.md",
+      )
+      revert = Commit.new(
+        message: "Revert \"Merge pull request #19 from byroot/casperisfine-patch-1\"\n\nThis reverts commit fa3722ef8372b47160f5d96010d3c54743d192f9, reversing\nchanges made to 868b6f65f759d003c04d056f2f928f18d6813c7e.",
+      )
+      assert revert.revert_of?(commit)
+    end
+
     private
 
     def expect_event(stack)

data/test/models/deploy_spec_test.rb

@@ -346,6 +346,21 @@ module Shipit
       assert_equal ['bundle exec foo'], definition.steps
     end
 
+    test "#task_definitions returns kubernetes commands as well as comands from the config" do
+      @spec.stubs(:load_config).returns(
+        'tasks' => {'another_task' => {'steps' => %w(foo)}},
+        'kubernetes' => {
+          'namespace' => 'foo',
+          'context' => 'bar',
+        },
+      )
+      tasks = @spec.task_definitions
+      assert_equal 2, tasks.size
+
+      restart_task = tasks.find { |t| t.id == "restart" }
+      assert_equal ["kubernetes-restart foo bar"], restart_task.steps
+    end
+
     test "task definitions returns an array of VariableDefinition instances" do
       @spec.expects(:load_config).returns('tasks' =>
         {'restart' =>

data/test/models/deploys_test.rb

@@ -225,6 +225,13 @@ module Shipit
       end
     end
 
+    test "transitions to any state updates last deploy time to stack record" do
+      @deploy = shipit_deploys(:shipit_running)
+      @deploy.complete!
+      @stack.reload
+      assert_in_delta @deploy.ended_at, @stack.last_deployed_at, 1
+    end
+
     test "transitioning to success schedule a MergePullRequests job" do
       @deploy = shipit_deploys(:shipit_running)
       assert_enqueued_with(job: MergePullRequestsJob, args: [@deploy.stack]) do

data/test/models/pull_request_test.rb

@@ -18,7 +18,7 @@ module Shipit
 
     test ".request_merge! only track pull requests once" do
       assert_difference -> { PullRequest.count }, +1 do
-        5.times { PullRequest.request_merge!(@stack, 65, @user) }
+        5.times { PullRequest.request_merge!(@stack, 999, @user) }
       end
     end
 
@@ -77,6 +77,7 @@ module Shipit
           mergeable: true,
           additions: 24,
           deletions: 5,
+          merged_at: nil,
           head: stub(
             ref: 'super-branch',
             sha: head_sha,
@@ -193,5 +194,22 @@ module Shipit
       assert_json 'pull_request.rejection_reason', 'merge_conflict', document: params['payload']
       assert_json 'pull_request.number', @pr.number, document: params['payload']
     end
+
+    test "#merge! doesnt delete the branch if there are open PRs against it" do
+      Shipit.github_api.expects(:merge_pull_request).once.returns(true)
+      Shipit.github_api.expects(:pull_requests).once.with(@stack.github_repo_name, base: @pr.branch).returns([1])
+      Shipit.github_api.expects(:delete_branch).never.returns(false)
+      assert_equal true, @pr.merge!
+    end
+
+    test "#merge! increments undeployed_commits_count" do
+      Shipit.github_api.expects(:merge_pull_request).once.returns(true)
+      Shipit.github_api.expects(:pull_requests).once.returns([])
+      Shipit.github_api.expects(:delete_branch).once.returns(true)
+      assert_difference '@stack.undeployed_commits_count' do
+        @pr.merge!
+        @stack.reload
+      end
+    end
   end
 end

data/test/models/task_definitions_test.rb

@@ -12,6 +12,8 @@ module Shipit
         'variables' => [
           {'name' => 'FOO', 'title' => 'Set to 0 to foo', 'default' => '1'},
           {'name' => 'BAR', 'title' => 'Set to 1 to bar', 'default' => '0'},
+          {'name' => 'WALRUS', 'title' => 'Use with caution', 'default' => ' '},
+          {'name' => 'NODEFAULT', 'title' => 'Variable without default'},
         ],
       )
     end
@@ -25,6 +27,11 @@ module Shipit
       assert_nil TaskDefinition.dump(nil)
     end
 
+    test "#variables" do
+      assert_equal 4, @definition.variables.size
+      assert_equal 3, @definition.variables_with_defaults.size
+    end
+
     test "serialization works" do
       as_json = {
         id: 'restart',
@@ -36,6 +43,8 @@ module Shipit
         variables: [
           {'name' => 'FOO', 'title' => 'Set to 0 to foo', 'default' => '1', 'select' => nil},
           {'name' => 'BAR', 'title' => 'Set to 1 to bar', 'default' => '0', 'select' => nil},
+          {'name' => 'WALRUS', 'title' => 'Use with caution', 'default' => ' ', 'select' => nil},
+          {'name' => 'NODEFAULT', 'title' => 'Variable without default', 'default' => '', 'select' => nil},
         ],
       }
       assert_equal as_json, TaskDefinition.load(TaskDefinition.dump(@definition)).as_json

data/test/models/undeployed_commits_test.rb

@@ -43,8 +43,8 @@ module Shipit
       assert_equal 'allowed', @commit.deploy_state
     end
 
-    test "#deploy_state returns `locked` if the stack is locked" do
-      @stack.update!(lock_reason: "Let's eat some chips!")
+    test "#deploy_state returns `locked` if the commit is locked" do
+      @commit.update!(locked: true)
       assert_equal 'locked', @commit.deploy_state
     end
 
@@ -72,11 +72,6 @@ module Shipit
       assert_equal 'allowed', @commit.redeploy_state
     end
 
-    test "#redeploy_state returns `locked` if the stack is locked" do
-      @stack.update!(lock_reason: "Let's eat some chips!")
-      assert_equal 'locked', @commit.redeploy_state
-    end
-
     test "#redeploy_state returns `allowed` if the stack is locked but the safeties are ignored" do
       @stack.update!(lock_reason: "Let's eat some chips!")
       assert_equal 'allowed', @commit.redeploy_state(true)

data/test/unit/deploy_commands_test.rb

@@ -134,6 +134,13 @@ module Shipit
       assert_equal @deploy.stack.last_deployed_commit.sha, command.env['LAST_DEPLOYED_SHA']
     end
 
+    test "#perform calls cap $environment deploy with the TASK_ID in the environment" do
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal @deploy.id.to_s, command.env['TASK_ID']
+    end
+
     test "#perform transliterates the user name" do
       @deploy.user = User.new(login: 'Sirupsen', name: "Simon Hørup Eskildsen")
       commands = @commands.perform

data/test/unit/variable_definition_test.rb

@@ -18,6 +18,7 @@ module Shipit
       assert_equal "Variable title", subject.title
       assert_equal "Variable default", subject.default
       assert_nil subject.select
+      assert subject.default_provided?
     end
 
     test "#initialize name is required" do
@@ -48,6 +49,15 @@ module Shipit
       assert_nil subject.select
     end
 
+    test "#default_provided?" do
+      attributes = {
+        "name" => "Variable name",
+        "title" => "Variable title",
+      }
+      subject = Shipit::VariableDefinition.new(attributes)
+      refute subject.default_provided?
+    end
+
     test "#to_h returns hash version" do
       assert_equal @attributes.merge("select" => nil), Shipit::VariableDefinition.new(@attributes).to_h
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: shipit-engine
 version: !ruby/object:Gem::Version
-  version: 0.16.0
+  version: 0.17.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-22 00:00:00.000000000 Z
+date: 2017-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -499,6 +499,7 @@ files:
 - app/assets/images/aborted.svg
 - app/assets/images/aborting.svg
 - app/assets/images/anchor.svg
+- app/assets/images/caret-down.svg
 - app/assets/images/default.svg
 - app/assets/images/deploy_error.jpg
 - app/assets/images/deploy_failed.jpg
@@ -511,15 +512,12 @@ files:
 - app/assets/images/failure.svg
 - app/assets/images/favicon.ico
 - app/assets/images/flapping.svg
-- app/assets/images/github.svg
 - app/assets/images/lock.svg
 - app/assets/images/logo.svg
 - app/assets/images/pending-small.svg
 - app/assets/images/pending.gif
-- app/assets/images/refresh.svg
 - app/assets/images/search-active.svg
 - app/assets/images/search.svg
-- app/assets/images/settings.svg
 - app/assets/images/ship.png
 - app/assets/images/success-small.svg
 - app/assets/images/success.svg
@@ -548,7 +546,9 @@ files:
 - app/assets/stylesheets/_base/_buttons.scss
 - app/assets/stylesheets/_base/_colors.scss
 - app/assets/stylesheets/_base/_forms.scss
+- app/assets/stylesheets/_base/_icons.scss
 - app/assets/stylesheets/_base/_media-queries.scss
+- app/assets/stylesheets/_base/_spacing.scss
 - app/assets/stylesheets/_base/_status-items.scss
 - app/assets/stylesheets/_base/_utility.scss
 - app/assets/stylesheets/_pages/_commits.scss
@@ -557,6 +557,7 @@ files:
 - app/assets/stylesheets/_pages/_stacks.scss
 - app/assets/stylesheets/_structure/_layout.scss
 - app/assets/stylesheets/_structure/_main.scss
+- app/assets/stylesheets/_structure/_navigation.scss
 - app/assets/stylesheets/shipit.scss
 - app/assets/stylesheets/shipit_bs.scss
 - app/assets/webfonts/CheckoutSymbols-Regular.eot
@@ -580,6 +581,7 @@ files:
 - app/controllers/shipit/api/tasks_controller.rb
 - app/controllers/shipit/ccmenu_url_controller.rb
 - app/controllers/shipit/commit_checks_controller.rb
+- app/controllers/shipit/commits_controller.rb
 - app/controllers/shipit/deploys_controller.rb
 - app/controllers/shipit/github_authentication_controller.rb
 - app/controllers/shipit/pull_requests_controller.rb
@@ -750,6 +752,9 @@ files:
 - db/migrate/20170220152410_improve_users_indexing.rb
 - db/migrate/20170221102128_improve_tasks_indexing.rb
 - db/migrate/20170221130336_add_last_revalidated_at_on_pull_requests.rb
+- db/migrate/20170310164315_add_merged_at_on_pull_requests.rb
+- db/migrate/20170314145604_add_last_deployed_at_to_stack.rb
+- db/migrate/20170320124156_add_locked_to_commits.rb
 - lib/shipit-engine.rb
 - lib/shipit.rb
 - lib/shipit/command.rb
@@ -773,7 +778,6 @@ files:
 - lib/snippets/assert-egg-version-tag
 - lib/snippets/assert-gem-version-tag
 - lib/snippets/assert-npm-version-tag
-- lib/snippets/deploy-to-gke
 - lib/snippets/extract-egg-version
 - lib/snippets/extract-gem-version
 - lib/snippets/fetch-gem-version
@@ -795,6 +799,7 @@ files:
 - test/controllers/api/tasks_controller_test.rb
 - test/controllers/ccmenu_controller_test.rb
 - test/controllers/commit_checks_controller_test.rb
+- test/controllers/commits_controller_test.rb
 - test/controllers/deploys_controller_test.rb
 - test/controllers/github_authentication_controller_test.rb
 - test/controllers/pull_requests_controller_test.rb
@@ -855,6 +860,7 @@ files:
 - test/dummy/db/schema.rb
 - test/dummy/db/seeds.rb
 - test/dummy/db/test.sqlite3
+- test/dummy/db/test.sqlite3-journal
 - test/dummy/public/404.html
 - test/dummy/public/422.html
 - test/dummy/public/500.html
@@ -957,7 +963,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.6
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Application deployment software
@@ -974,6 +980,7 @@ test_files:
 - test/controllers/api/tasks_controller_test.rb
 - test/controllers/ccmenu_controller_test.rb
 - test/controllers/commit_checks_controller_test.rb
+- test/controllers/commits_controller_test.rb
 - test/controllers/deploys_controller_test.rb
 - test/controllers/github_authentication_controller_test.rb
 - test/controllers/pull_requests_controller_test.rb
@@ -1033,6 +1040,7 @@ test_files:
 - test/dummy/db/schema.rb
 - test/dummy/db/seeds.rb
 - test/dummy/db/test.sqlite3
+- test/dummy/db/test.sqlite3-journal
 - test/dummy/public/404.html
 - test/dummy/public/422.html
 - test/dummy/public/500.html

data/app/assets/images/github.svg

@@ -1,9 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!-- Generator: Adobe Illustrator 17.0.2, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
-<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
-<svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px" width="18px"
-	 height="18px" viewBox="0 0 42 42" enable-background="new 0 0 18 18" xml:space="preserve">
-<g transform="matrix(1,0,0,-1,20,5)">
-<path d="m 0,0 c -8.995,0 -16.288,-7.293 -16.288,-16.29 0,-7.197 4.667,-13.302 11.14,-15.457 0.815,-0.149 1.112,0.354 1.112,0.786 0,0.386 -0.014,1.411 -0.022,2.77 -4.531,-0.984 -5.487,2.184 -5.487,2.184 -0.741,1.882 -1.809,2.383 -1.809,2.383 -1.479,1.01 0.112,0.99 0.112,0.99 1.635,-0.115 2.495,-1.679 2.495,-1.679 1.453,-2.489 3.813,-1.77 4.741,-1.353 0.148,1.052 0.568,1.77 1.034,2.177 -3.617,0.411 -7.42,1.809 -7.42,8.051 0,1.778 0.635,3.232 1.677,4.371 -0.168,0.412 -0.727,2.068 0.159,4.311 0,0 1.368,0.438 4.48,-1.67 1.299,0.362 2.693,0.542 4.078,0.548 1.383,-0.006 2.777,-0.186 4.078,-0.548 3.11,2.108 4.475,1.67 4.475,1.67 0.889,-2.243 0.33,-3.899 0.162,-4.311 1.044,-1.139 1.675,-2.593 1.675,-4.371 0,-6.258 -3.809,-7.635 -7.438,-8.038 0.585,-0.503 1.106,-1.497 1.106,-3.017 0,-2.177 -0.02,-3.934 -0.02,-4.468 0,-0.436 0.293,-0.943 1.12,-0.784 6.468,2.159 11.131,8.26 11.131,15.455 C 16.291,-7.293 8.997,0 0,0" style="fill:#FFF;fill-opacity:1;fill-rule:evenodd;stroke:none"/>
-</g>
-</svg>

data/app/assets/images/refresh.svg

@@ -1,8 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1 Tiny//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11-tiny.dtd">
-<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" version="1.1" baseProfile="tiny" x="0px" y="0px" width="480px" height="480px" viewBox="0 0 480 480" xml:space="preserve">
-  <g id="refresh" fill="#FFF">
-    <path d="M383.434,172.502c-25.336-58.241-81.998-95.648-145.861-95.648c-65.309,0-125,40.928-148.514,101.827l49.5,19.117   c15.672-40.617,55.469-67.894,99.014-67.894c42.02,0,79.197,24.386,96.408,62.332l-36.117,14.428l92.352,53.279l27.01-100.933   L383.434,172.502z"/>
-    <path d="M237.573,342.101c-41.639,0-79.615-25.115-96.592-62.819l35.604-13.763l-91.387-52.119l-27.975,98.249l34.08-13.172   c24.852,58.018,82.859,96.671,146.27,96.671c65.551,0,123.598-39.336,147.871-100.196l-49.268-19.652   C319.981,315.877,281.288,342.101,237.573,342.101z"/>
-  </g>
-</svg>

data/app/assets/images/settings.svg

@@ -1,33 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!-- Generator: Adobe Illustrator 17.0.2, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
-<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
-<svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px" width="18px"
-	 height="18px" viewBox="0 0 18 18" enable-background="new 0 0 18 18" xml:space="preserve">
-<g id="Layer_1">
-	<g>
-		<path fill="#FFFFFF" d="M15.667,10.042l1.729,1.5c0.139,0.098,0.174,0.229,0.104,0.396c-0.348,1.07-0.917,2.056-1.708,2.959
-			c-0.111,0.139-0.243,0.174-0.396,0.104l-2.166-0.75c-0.556,0.445-1.153,0.792-1.792,1.042L11,17.542
-			c-0.028,0.167-0.125,0.264-0.292,0.292C10.111,17.945,9.542,18,9,18s-1.111-0.055-1.708-0.166C7.125,17.806,7.027,17.709,7,17.542
-			l-0.438-2.25c-0.681-0.264-1.278-0.611-1.792-1.042L2.604,15c-0.153,0.07-0.285,0.035-0.396-0.104
-			c-0.792-0.903-1.361-1.889-1.708-2.959c-0.07-0.166-0.035-0.298,0.104-0.396l1.729-1.5C2.278,9.75,2.25,9.403,2.25,9
-			s0.028-0.75,0.083-1.042l-1.729-1.5C0.465,6.362,0.43,6.229,0.5,6.063c0.347-1.069,0.917-2.055,1.708-2.958
-			C2.319,2.966,2.451,2.931,2.604,3l2.167,0.75c0.514-0.43,1.111-0.778,1.792-1.042L7,0.459c0.027-0.167,0.125-0.264,0.292-0.292
-			c1.139-0.236,2.277-0.236,3.416,0C10.875,0.195,10.972,0.292,11,0.459l0.438,2.25c0.639,0.25,1.236,0.597,1.792,1.042L15.396,3
-			c0.153-0.069,0.285-0.035,0.396,0.104c0.791,0.903,1.36,1.889,1.708,2.958c0.069,0.167,0.034,0.299-0.104,0.396l-1.729,1.5
-			C15.722,8.25,15.75,8.598,15.75,9S15.722,9.75,15.667,10.042z M9,12.375c0.931,0,1.726-0.329,2.386-0.989
-			c0.659-0.659,0.989-1.455,0.989-2.385c0-0.93-0.33-1.726-0.989-2.385c-0.66-0.66-1.455-0.99-2.386-0.99s-1.726,0.33-2.385,0.99
-			C5.955,7.275,5.625,8.07,5.625,9c0,0.945,0.326,1.743,0.979,2.396C7.257,12.049,8.056,12.375,9,12.375z"/>
-	</g>
-</g>
-<g id="Layer_1_copy" display="none">
-	<g display="inline">
-		<path fill="#FFFFFF" d="M13.872,2.629h-1.479c0.229,0.521,0.354,1.125,0.354,1.791c0,2.479-1.771,4.124-3.958,4.208
-			c-0.646,1.021-0.479,1.624,0.459,2.249c2.582,0.104,4.624,1.604,4.624,3.562c0,2.083-2.188,3.562-4.874,3.562
-			c-2.687,0-4.874-1.499-4.874-3.562c0-1.333,0.917-2.437,2.312-3.041c-1.312-1.291-0.854-2.582,0.062-3.332
-			c-1.208-0.688-2-1.979-2-3.645c0-2.541,1.854-4.228,4.124-4.228c0.833,0,1.604,0.229,2.249,0.625c2.229-0.146,3-0.812,3-0.812
-			V2.629z M6.749,14.438c0,0.729,0.875,1.312,2.25,1.312c1.374,0,2.249-0.583,2.249-1.312c0-0.729-0.875-1.312-2.249-1.312
-			C7.623,13.126,6.749,13.71,6.749,14.438z M10.122,4.316c0-1.312-0.666-2.062-1.499-2.062s-1.5,0.75-1.5,2.062
-			s0.667,2.062,1.5,2.062S10.122,5.628,10.122,4.316z"/>
-	</g>
-</g>
-</svg>

data/lib/snippets/deploy-to-gke

@@ -1,161 +0,0 @@
-#!/usr/bin/env ruby
-
-# Usage: deploy-to-gke <app's namespace> <gcloud deployment> <gcloud project>
-
-# Requires the following to be available on Shipit machine:
-#  - gcloud binary available on the shipit machine's path
-#  - kubectl binary available in the shipit machine's path
-#  - $GCLOUD_CREDENTIALS_DIR/my-gcloud-project-name.json must exist
-
-# Optionally, the following variables can be used to override script defaults:
-#  - K8S_TEMPLATE_FOLDER: location of Kubernetes files to deploy. Default is config/deploy/#{environment}.
-
-require 'open3'
-require 'securerandom'
-require 'erb'
-require 'json'
-require 'yaml'
-require 'shellwords'
-require 'tempfile'
-
-class GKEDeployment
-  class FatalDeploymentError < StandardError; end
-
-  def initialize(namespace:, gcloud_deployment:, gcloud_project:, environment:, current_sha:, key_file:, template_folder: nil)
-    @namespace = namespace
-    @gcloud_deployment = gcloud_deployment
-    @gcloud_project = gcloud_project
-    @current_sha = current_sha
-    @key_file = key_file
-    @template_path = './' + (template_folder || "config/deploy/#{environment}")
-
-    # Validate params + check existance of auth key and template(s)
-    enforce_required_params
-    preliminary_check
-
-    # Max length of podname is only 63chars so try to save some room by truncating sha to 8 chars
-    @id = current_sha[0...8] + "-#{SecureRandom.hex(4)}"
-  end
-
-  def run
-    authorize_gcloud
-    fetch_clusters.each do |cluster|
-      set_kubectl_cluster(cluster)
-      apply_all_templates
-    end
-  rescue FatalDeploymentError => error
-    print_error(error.message)
-    exit 1
-  end
-
-  def template_variables
-    {
-      'current_sha' => @current_sha,
-      'deployment_id' => @id,
-    }
-  end
-
-  private
-
-  def enforce_required_params
-    [@namespace, @gcloud_project, @gcloud_deployment, @current_sha, @key_file].each do |required_param|
-      raise ArgumentError, "#{required_param} is required" unless required_param && !required_param.empty?
-    end
-  end
-
-  def preliminary_check
-    raise FatalDeploymentError, "Project config missing at #{@key_file}" unless File.file?(@key_file)
-    raise FatalDeploymentError, "#{@template_path} doesn't exist" unless File.directory?(@template_path)
-    raise FatalDeploymentError, "#{@template_path} doesn't have files with postfix .yml or .yml.erb" unless Dir.entries(@template_path).select {|file| file =~ /\.yml(.erb)?$/}.size > 0
-  end
-
-  def apply_all_templates
-    found = 0
-    Dir.foreach(@template_path) do |file|
-      file_path = "#{@template_path}/#{file}"
-      if File.extname(file) == '.yml'
-        found += 1
-        apply_template(file_path)
-      elsif File.extname(file) == '.erb'
-        found += 1
-        render_and_apply_template(file_path)
-      end
-    end
-    raise FatalDeploymentError, "No templates found in #{@template_path}" if found.zero?
-  end
-
-  def render_and_apply_template(file_path)
-    erb_template = ERB.new(File.read(file_path))
-    erb_binding = binding
-    template_variables.each do |var_name, value|
-      erb_binding.local_variable_set(var_name, value)
-    end
-    content = erb_template.result(erb_binding)
-
-    f = Tempfile.new(['kube_template', '.yml'])
-    f.write(content)
-    f.close
-    apply_template(f.path, original_path: file_path)
-  ensure
-    f.unlink if f
-  end
-
-  def apply_template(path, original_path: nil)
-    status = run_command('kubectl', 'apply', '-f', path, "--namespace=#{@namespace}")
-    raise FatalDeploymentError, "Failed to apply template #{original_path || path}" unless status
-  end
-
-  def authorize_gcloud
-    status = run_command('gcloud', '-q', 'auth', 'activate-service-account', '--key-file', @key_file)
-    status = run_command('gcloud', '-q', 'config', 'set', 'project', @gcloud_project) if status
-    raise FatalDeploymentError, "Failed to set gcloud project #{@gcloud_project}" unless status
-  end
-
-  def set_kubectl_cluster(cluster)
-    cluster_name = cluster[0]
-    cluster_zone = cluster[1]
-    status = run_command('gcloud', '-q', 'container', 'clusters', 'get-credentials', cluster_name, '--zone', cluster_zone)
-    raise FatalDeploymentError, "Failed to set cluster #{cluster_name}/#{cluster_zone}" unless status
-  end
-
-  def fetch_clusters
-    result = query_deployments
-    result['resources'].each_with_object([]) do |resource, data|
-      next unless resource['type'] == 'container.v1.cluster'
-      properties = YAML.load(resource['finalProperties'])
-      data << [resource['name'], properties['zone']]
-    end
-  end
-
-  def run_command(*args)
-    puts Shellwords.join(args)
-    out, err, st = Open3.capture3(*args)
-    puts out
-    print_error(err) unless st.success?
-    st.success?
-  end
-
-  def print_error(msg)
-    puts "\033[0;31m#{msg}\033[0m"
-  end
-
-  def query_deployments
-    out, err, st = Open3.capture3('gcloud', '-q', 'deployment-manager', 'deployments', 'describe', @gcloud_deployment, '--format=json')
-    unless st.success?
-      print_error(err)
-      raise FatalDeploymentError, "Failed to fetch cluster with deployment #{@gcloud_deployment}"
-    end
-    JSON.parse(out)
-  end
-end
-
-deployment = GKEDeployment.new(
-  namespace: ARGV[0],
-  gcloud_deployment: ARGV[1],
-  gcloud_project: ARGV[2],
-  environment: ENV['ENVIRONMENT'],
-  template_folder: ENV['K8S_TEMPLATE_FOLDER'],
-  current_sha: ENV['REVISION'],
-  key_file: ENV['GOOGLE_APPLICATION_CREDENTIALS']
-)
-deployment.run