Skip to content

--no-restart option for env-add env-del #50

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,10 @@ MIT license, please see the LICENSE file. All rights reserved._

Application Environment
env <appname> List application environment variables
env-add <appname> <variable[=]value> Add an environment variable to an application
env-del <appname> <variable> Delete an environment variable to an application
env-add <appname> <variable[=]value>
[--no-restart] Add an environment variable to an application
env-del <appname> <variable>
[--no-restart] Delete an environment variable to an application

Services
services Lists of services available and provisioned
Expand Down
5 changes: 3 additions & 2 deletions lib/cli/commands/apps.rb
Original file line number Diff line number Diff line change
Expand Up @@ -429,6 +429,7 @@ def environment(appname)
end

def environment_add(appname, k, v=nil)
no_restart = @options[:norestart]
app = client.app_info(appname)
env = app[:env] || []
k,v = k.split('=', 2) unless v
Expand All @@ -437,7 +438,7 @@ def environment_add(appname, k, v=nil)
app[:env] = env
client.update_app(appname, app)
display 'OK'.green
restart appname if app[:state] == 'STARTED'
restart appname if app[:state] == 'STARTED' unless no_restart
end

def environment_del(appname, variable)
Expand All @@ -457,7 +458,7 @@ def environment_del(appname, variable)
app[:env] = env
client.update_app(appname, app)
display 'OK'.green
restart appname if app[:state] == 'STARTED'
restart appname if app[:state] == 'STARTED' unless no_restart
else
display 'OK'.green
end
Expand Down
2 changes: 2 additions & 0 deletions lib/cli/runner.rb
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@ def parse_options!
opts.on('--path PATH') { |path| @options[:path] = path }
opts.on('--no-start') { @options[:nostart] = true }
opts.on('--nostart') { @options[:nostart] = true }
opts.on('--no-restart') { @options[:norestart] = true }
opts.on('--norestart') { @options[:norestart] = true }
opts.on('--force') { @options[:force] = true }
opts.on('--all') { @options[:all] = true }

Expand Down