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 <g...@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]