[GitHub] [ignite] Sega76 commented on a change in pull request #8575: IGNITE-13492: Added snapshot test to ducktests

[GitBox <gi...@apache.org>]

Sega76 commented on a change in pull request #8575:
URL: https://github.com/apache/ignite/pull/8575#discussion_r543281753



##########
File path: modules/ducktests/tests/ignitetest/services/utils/ignite_aware.py
##########
@@ -351,3 +351,25 @@ def __dump_netfilter_settings(node):
         Reads current netfilter settings on the node for debugging purposes.
         """
         return str(node.account.ssh_client.exec_command("sudo iptables -L -n")[1].read(), sys.getdefaultencoding())
+
+    def restart(self):
+        """
+        Restart ignite cluster without cleaning.
+        """
+        self.stop()
+        self.rotate_log()
+        self.start(clean=False)
+
+    def rotate_log(self):
+        """
+        Rotate log file.
+        """
+        new_log_file = self.STDOUT_STDERR_CAPTURE.replace('.log', '_$N.log')
+
+        for node in self.nodes:
+            node.account.ssh(f'if [ -e {self.STDOUT_STDERR_CAPTURE} ];'
+                             f'then '
+                             f'cd {self.LOGS_DIR};'
+                             f'N=`ls | grep log | wc -l`;'

Review comment:
       when starting different services on the same nodes (for example, when testing the PDS), we will not be able to use counter the service.
   So I think the counter is a bad idea.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org