[GitHub] [ignite] Sega76 commented on a change in pull request #8557: IGNITE-13829: Added log rotation to ducktape-tests

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

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



##########
File path: modules/ducktests/tests/ignitetest/services/utils/ignite_spec.py
##########
@@ -102,12 +102,13 @@ class IgniteNodeSpec(IgniteSpec, IgnitePersistenceAware):
     """
     @property
     def command(self):
-        cmd = "%s %s %s %s 2>&1 | tee -a %s &" % \
+        cmd = "%s %s %s %s 2>&1 | tee %s >> %s &" % \
               (self._envs(),
                self.path.script("ignite.sh"),
                self._jvm_opts(),
                self.CONFIG_FILE,
-               self.STDOUT_STDERR_CAPTURE)
+               self.STDOUT_STDERR_CAPTURE,
+               self.STDOUT_STDERR_ALL)

Review comment:
       console.log is used as a monitor, when the service starts, we use it to determine that it has started.
   It appears earlier than ignite.log. on restart, if we look at the old log, we will see all the events there, and we will not wait for them. it is not right
   
   console.log contains stdout the last start
   console_all.log contains stdout all starts




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