ivandasch commented on a change in pull request #8294:
URL: https://github.com/apache/ignite/pull/8294#discussion_r500905113



##########
File path: modules/ducktests/tests/ignitetest/services/ignite.py
##########
@@ -46,13 +46,42 @@ def __init__(self, context, config, num_nodes, 
jvm_opts=None, modules=None):
 
     # pylint: disable=W0221
     def start(self, timeout_sec=180):
+        self._rotate_log()
+
         super().start()
 
         self.logger.info("Waiting for Ignite(s) to start...")
 
         for node in self.nodes:
             self.await_node_started(node, timeout_sec)
 
+    def restart(self, timeout_sec=180):
+        """
+        Restart ignite cluster without cleaning.
+        """
+        self.stop()
+
+        self._rotate_log()
+
+        for node in self.nodes:
+            super().start_node(node)
+
+        self.logger.info("Waiting for Ignite(s) to start...")
+
+        for node in self.nodes:
+            self.await_node_started(node, timeout_sec)
+
+    def _rotate_log(self):

Review comment:
       Because we do not append to stream, so log file will be broken.
   But yes, we can do fix this just adding 1 character




----------------------------------------------------------------
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:
[email protected]


Reply via email to