[ARVADOS] updated: 1.3.0-527-g6582c5afa

Git user git at public.curoverse.com
Mon Mar 18 13:42:51 UTC 2019


Summary of changes:
 lib/dispatchcloud/container/queue.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  6582c5afa53258fcb36d682fb690203930b7b2f6 (commit)
      from  2a748e79c3a72454d70e40f39fcad9dabf4943cc (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 6582c5afa53258fcb36d682fb690203930b7b2f6
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Mon Mar 18 09:42:27 2019 -0400

    14977: Re-order statements for clarity.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/lib/dispatchcloud/container/queue.go b/lib/dispatchcloud/container/queue.go
index 9cb210d0e..4e807a12a 100644
--- a/lib/dispatchcloud/container/queue.go
+++ b/lib/dispatchcloud/container/queue.go
@@ -428,6 +428,7 @@ func (cq *Queue) poll() (map[string]*arvados.Container, error) {
 		if err != nil {
 			return nil, err
 		}
+		apply(ended)
 		if len(ended) == 0 {
 			// This is the only case where we can conclude
 			// a container has been deleted from the
@@ -443,7 +444,6 @@ func (cq *Queue) poll() (map[string]*arvados.Container, error) {
 			}
 			continue
 		}
-		apply(ended)
 		for _, ctr := range ended {
 			if _, ok := missing[ctr.UUID]; !ok {
 				msg := "BUG? server response did not match requested filters, erroring out rather than risk deadlock"

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list