Skip to content

Commit

Permalink
Remove kill_after_multiple_restarts column from apps table
Browse files Browse the repository at this point in the history
[#88352892]
  • Loading branch information
zaksoup authored and James Myers and Zak Auerbach committed Feb 20, 2015
1 parent ea2dfe1 commit 49b2a20
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 14 deletions.
1 change: 0 additions & 1 deletion app/models/runtime/app.rb
Expand Up @@ -72,7 +72,6 @@ def after_initialize
# Last staging response which will contain streaming log url
attr_accessor :last_stager_response

alias_method :kill_after_multiple_restarts?, :kill_after_multiple_restarts
alias_method :diego?, :diego

def copy_buildpack_errors
Expand Down
@@ -0,0 +1,13 @@
Sequel.migration do
up do
alter_table :apps do
drop_column :kill_after_multiple_restarts
end
end

down do
alter_table :apps do
add_column :kill_after_multiple_restarts, :boolean, default: false
end
end
end
13 changes: 0 additions & 13 deletions spec/unit/models/runtime/app_spec.rb
Expand Up @@ -1342,19 +1342,6 @@ def self.it_does_not_mark_for_re_staging
end
end

describe 'kill_after_multiple_restarts?' do
let(:app) { AppFactory.make }

it 'defaults to false' do
expect(app.kill_after_multiple_restarts?).to eq false
end

it 'can be set to true' do
app.kill_after_multiple_restarts = true
expect(app.kill_after_multiple_restarts?).to eq true
end
end

describe 'version' do
let(:app) { AppFactory.make(package_hash: 'abc', package_state: 'STAGED') }

Expand Down

0 comments on commit 49b2a20

Please sign in to comment.