anton-vinogradov commented on a change in pull request #8294:
URL: https://github.com/apache/ignite/pull/8294#discussion_r500933202



##########
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:
       Rotation should be performed by log4j if necessary. Node restart is not 
a reason to rotate logs.




----------------------------------------------------------------
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