This is an automated email from the ASF dual-hosted git repository.

aviemzur pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-liminal.git


The following commit(s) were added to refs/heads/master by this push:
     new f8d7589  add detached mode for liminal start
f8d7589 is described below

commit f8d7589cc7a75acf0a14efb6c591da388dff1a39
Author: lidor ettinger <[email protected]>
AuthorDate: Sat Jan 2 20:19:47 2021 +0200

    add detached mode for liminal start
    
    Signed-off-by: lidor ettinger <[email protected]>
---
 scripts/liminal | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/scripts/liminal b/scripts/liminal
index ff4767e..1317562 100755
--- a/scripts/liminal
+++ b/scripts/liminal
@@ -95,6 +95,7 @@ def deploy_liminal_core_internal(clean):
 
 
 def docker_compose_command(command_name, args):
+    detached_mode = False
     docker_compose_path, project_dir = get_docker_compose_paths()
     concated_args = ' '.join(args)
     run_command = [
@@ -104,14 +105,16 @@ def docker_compose_command(command_name, args):
         f'{project_dir} {command_name} {concated_args}'
     ]
     logging.info(run_command[0])
-    if 'follow' in str(args):
-        subprocess.call(run_command, env=os.environ, shell=True)
-        return '', ''
-    else:
+    if ('tail' in str(args) or '-d' in str(args) or command_name in ['ps', 
'down']):
+        detached_mode = True
+
+    if detached_mode:
         output = subprocess.run(run_command, env=os.environ, shell=True, 
stdout=subprocess.PIPE,
                                 stderr=subprocess.PIPE, 
universal_newlines=True)
         return output.stdout, output.stderr
-
+    else:
+        subprocess.call(run_command, env=os.environ, shell=True)
+        return '', ''
 
 @cli.command("deploy", short_help="deploys your liminal.yaml files to 
$LIMINAL_HOME folder")
 @click.option('--path', default=os.getcwd(), help="folder containing 
liminal.yaml files")
@@ -165,13 +168,17 @@ def logs(follow, tail):
 @cli.command("start",
              short_help="starts a local airflow in docker compose. should be 
run after deploy. " +
                         "Make sure docker is running on your machine")
-def start():
[email protected]('--detached-mode', '-d', is_flag=True, default=False, 
help="Start liminal in detached mode.")
+def start(detached_mode):
     liminal_version = environment.get_liminal_version()
     logging.info(f'starting liminal version: {liminal_version}')
     if docker_is_running():
         # initialize liminal home by default
         environment.get_liminal_home()
-        docker_compose_command('up', ['-d'])
+        if detached_mode:
+            docker_compose_command('up', ['-d'])
+        else:
+            docker_compose_command('up', [])
 
 
 def get_docker_compose_paths():

Reply via email to