Merge pull request #55 from kaorimatz/fix-job-not-being-recorded-as-c…

…ompleted Fix job not being recorded as completed when exit status > 127
cookpad · May 9, 2017 · c280969 · c280969
2 parents 0382965 + d9f61cc
commit c280969
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/db/migrate/028_change_exit_status_to_unsigned_tinyint.rb b/db/migrate/028_change_exit_status_to_unsigned_tinyint.rb
@@ -0,0 +1,9 @@
+class ChangeExitStatusToUnsignedTinyint < ActiveRecord::Migration[5.0]
+  def up
+    change_column :executions, :exit_status, :integer, limit: 2
+  end
+
+  def down
+    change_column :executions, :exit_status, :integer, limit: 1
+  end
+end
diff --git a/spec/dummy/db/schema.rb b/spec/dummy/db/schema.rb
@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 27) do
+ActiveRecord::Schema.define(version: 28) do
 
   create_table "admin_assignments", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8mb4" do |t|
     t.integer  "user_id",           null: false
@@ -31,7 +31,7 @@
     t.text     "context",                limit: 65535,                           null: false
     t.integer  "pid"
     t.text     "output",                 limit: 4294967295
-    t.integer  "exit_status",            limit: 1
+    t.integer  "exit_status",            limit: 2
     t.integer  "term_signal",            limit: 1
     t.datetime "started_at"
     t.datetime "finished_at"