[ARVADOS] updated: 6aa0477266f3250f7e1a5ad0915cee308b661b49

[Git user]

Summary of changes:
 sdk/python/arvados/commands/ws.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

       via  6aa0477266f3250f7e1a5ad0915cee308b661b49 (commit)
      from  a68b0ba89ab00dddbc1e59872ed81c08a6113295 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 6aa0477266f3250f7e1a5ad0915cee308b661b49
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Thu Aug 17 15:38:17 2017 -0400

    12123: Subscribe before unsubscribing to avoid losing messages.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/sdk/python/arvados/commands/ws.py b/sdk/python/arvados/commands/ws.py
index 5707063..37dab55 100644
--- a/sdk/python/arvados/commands/ws.py
+++ b/sdk/python/arvados/commands/ws.py
@@ -50,9 +50,10 @@ def main(arguments=None):
             if "job" in components[c]:
                 pipeline_jobs.add(components[c]["job"]["uuid"])
         if known_component_jobs != pipeline_jobs:
+            new_filters = [['object_uuid', 'in', [args.pipeline] + list(pipeline_jobs)]]
+            ws.subscribe(new_filters)
             ws.unsubscribe(filters)
-            filters = [['object_uuid', 'in', [args.pipeline] + list(pipeline_jobs)]]
-            ws.subscribe([['object_uuid', 'in', [args.pipeline] + list(pipeline_jobs)]])
+            filters = new_filters
             known_component_jobs = pipeline_jobs
 
     api = arvados.api('v1')

-----------------------------------------------------------------------


hooks/post-receive
--