Skip to content

Commit

Permalink
[FLINK-17794][tests] Tear down installed software in reverse order
Browse files Browse the repository at this point in the history
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 apache#12249.
  • Loading branch information
GJL committed May 21, 2020
1 parent 4dbc18d commit 2b2c574
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion flink-jepsen/src/jepsen/flink/db.clj
Original file line number Diff line number Diff line change
Expand Up @@ -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]
Expand Down

0 comments on commit 2b2c574

Please sign in to comment.