You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ga...@apache.org on 2020/05/21 16:49:40 UTC
[flink] branch release-1.11 updated: [FLINK-17794][tests] Tear down
installed software in reverse order
This is an automated email from the ASF dual-hosted git repository.
gary pushed a commit to branch release-1.11
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-1.11 by this push:
new 6a4714f [FLINK-17794][tests] Tear down installed software in reverse order
6a4714f is described below
commit 6a4714fdeff96d54db5fde5fac9b0eb355886b47
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.
---
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]