[flink] branch master updated: [FLINK-17794][tests] Tear down installed software in reverse order

[ga...@apache.org]

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

gary pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/master by this push:
     new 2b2c574  [FLINK-17794][tests] Tear down installed software in reverse order
2b2c574 is described below

commit 2b2c574f102689b3cde9deac0bd1bcf78ad7ebc7
Author: Gary Yao <ga...@apache.org>
AuthorDate: Mon May 18 10:48:34 2020 +0200

    [FLINK-17794][tests] Tear down installed software in reverse order
    
    Tear down installed software in reverse order in Jepsen tests. This
    mitigates the issue that sometimes YARN's NodeManager directories cannot
    be removed using 'rm -rf' because Flink processes keep running and
    generate files after the YARN NodeManager is shut down. rm -r removes
    files recursively but if files are created in the background
    concurrently, the command can still fail with a non-zero exit code.
    
    This closes #12249.
---
 flink-jepsen/src/jepsen/flink/db.clj | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/flink-jepsen/src/jepsen/flink/db.clj b/flink-jepsen/src/jepsen/flink/db.clj
index 61743b1..71c6716 100644
--- a/flink-jepsen/src/jepsen/flink/db.clj
+++ b/flink-jepsen/src/jepsen/flink/db.clj
@@ -109,7 +109,7 @@
     (teardown! [_ test node]
       (c/su
         (try
-          (doseq [db dbs] (db/teardown! db test node))
+          (doseq [db (reverse dbs)] (db/teardown! db test node))
           (finally (fu/stop-all-supervised-services!)))))
     db/LogFiles
     (log-files [_ test node]