Merge pull request #79 from postgrespro/revert-full-clean

Revert "PBCKP-377 optionaly keep test logs"

Author: asavchkov

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