Re: [devel] [PATCH 1/1] log: fix logtest 4 56 fail because egrep log file fail [#2662]

2017-11-01 Thread Lennart Lund
Hi Canh, Ack with comments. See attached diff file This code does not live up to good standard regarding correct documentation and clarity (and there are other things as well that will not pass static code check). This is however not directly related to the fix as such but it is always a good

[devel] [PATCH 1/1] log: fix logtest 4 56 fail because egrep log file fail [#2662]

2017-10-31 Thread Canh Van Truong
In test case logtest 4 56, it verifies that whether written log file format is reflected correctly. It finds and checks the log file that has been closed (FILENAME_MMDD_HHMMSS_MMDD_HHMMSS.log). But sometimes the log file has not been closed successfully due to timeout reason, this cause