From 003d334a103b392084893e7b77ee0c8a86fcaa76 Mon Sep 17 00:00:00 2001 From: Alexey Savchkov Date: Fri, 5 May 2023 11:25:55 +0700 Subject: [PATCH] Revert "PBCKP-377 optionaly keep test logs" This reverts commit b3777b2c4bedd4435befb924a2e7b96bd5f00c76. --- testgres/node.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/testgres/node.py b/testgres/node.py index de6a5032..e6ac44b0 100644 --- a/testgres/node.py +++ b/testgres/node.py @@ -875,15 +875,13 @@ def free_port(self): self._should_free_port = False release_port(self.port) - def cleanup(self, max_attempts=3, full_clean=testgres_config.node_cleanup_full): + def cleanup(self, max_attempts=3): """ Stop node if needed and remove its data/logs directory. NOTE: take a look at TestgresConfig.node_cleanup_full. Args: max_attempts: how many times should we try to stop()? - full_clean: True - clean all node data - False - clean only data dir, save logs Returns: This instance of :class:`.PostgresNode`. @@ -892,7 +890,7 @@ def cleanup(self, max_attempts=3, full_clean=testgres_config.node_cleanup_full): self._try_shutdown(max_attempts) # choose directory to be removed - if full_clean: + if testgres_config.node_cleanup_full: rm_dir = self.base_dir # everything else: rm_dir = self.data_dir # just data, save logs