[logAnalyzer] Avoid logAnalyzer logs being overwritten (#822)

In logAnalyzer related scripts, ansible_date_time is used for generating
testname_unique which will be used as folder name for storing
logAnalyzer results. The drawback is that ansible_date_time is fixed
in single ansible playbook execution. If logAnalyzer is called
multiple times, the earlier results could be overwritten by the latest
results.

The fix is to replace ansible_date_time with:
{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}

There are two differences:
1. Each time the lookup plugin is called, the returned date and time
   would be different (interval of two calls longer than 1 second)
2. The lookup plugin gets date and time of the sonic-mgmt container,
   not date and time of DUT.

Signed-off-by: Xin Wang <xinw@mellanox.com>

Author: wangxin

ansible/roles/test/files/tools/loganalyzer/loganalyzer_init.yml

@@ -18,7 +18,7 @@
   when: expect_file is not defined
 
 - set_fact:
-    testname_unique: "{{ testname }}.{{ ansible_date_time.date }}.{{ ansible_date_time.time }}"
+    testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
   when: testname_unique is not defined
 
 - set_fact:

ansible/roles/test/tasks/acltb_ranges_test.yml

@@ -27,7 +27,7 @@
 
 # Separate set_fact is required to be able to use 'testname' fact.
 - set_fact:
-    testname_unique: "{{ testname }}.{{ ansible_date_time.date}}.{{ ansible_date_time.hour}}-{{ ansible_date_time.minute}}-{{ ansible_date_time.second}}"
+    testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
 
 # Separate set_fact is required to be able to use 'testname_unique' fact.
 - set_fact:

ansible/roles/test/tasks/decap.yml

@@ -103,7 +103,7 @@
 
   # Separate set_fact is required to be able to use 'testname' fact.
   - set_fact:
-      testname_unique: "{{ testname }}.{{ ansible_date_time.date}}.{{ ansible_date_time.hour}}-{{ ansible_date_time.minute}}-{{ ansible_date_time.second}}"
+      testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
 
   - debug: msg="generated run id:{{testname_unique}}"
 

ansible/roles/test/tasks/lag.yml

@@ -33,7 +33,7 @@
 
 # Separate set_fact is required to be able to use 'testname' fact.
 - set_fact:
-    testname_unique: "{{ testname }}.{{ ansible_date_time.date}}.{{ ansible_date_time.hour}}-{{ ansible_date_time.minute}}-{{ ansible_date_time.second}}"
+    testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
 
 # Separate set_fact is required to be able to use 'testname_unique' fact.
 - set_fact:

ansible/vars/run_config_test_vars.yml

@@ -1,6 +1,6 @@
 ---
 
-testname_unique: "{{ testname }}.{{ ansible_date_time.date }}.{{ ansible_date_time.time }}"
+testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
 
 test_out_dir: "{{ out_dir }}/{{ testname_unique }}"
 loganalyzer_init: roles/test/files/tools/loganalyzer/loganalyzer_init.yml

ansible/vars/run_loganalyzer_vars.yml

@@ -1,6 +1,6 @@
 ---
 
-testname_unique: "{{ testname }}.{{ ansible_date_time.date }}.{{ ansible_date_time.time }}"
+testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
 
 test_out_dir: "{{ out_dir }}/{{ testname_unique }}"
 loganalyzer_init: roles/test/files/tools/loganalyzer/loganalyzer_init.yml

ansible/vars/run_ping_test_vars.yml

@@ -1,6 +1,6 @@
 ---
 
-testname_unique: "{{ testname }}.{{ ansible_date_time.date }}.{{ ansible_date_time.time }}"
+testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
 
 test_out_dir: "{{ out_dir }}/{{ testname_unique }}"
 summary_file: "summary.loganalysis.{{ testname_unique }}.log"

ansible/vars/run_ptf_test_vars.yml

@@ -1,6 +1,6 @@
 ---
 
-testname_unique: "{{ testname }}.{{ ansible_date_time.date }}.{{ ansible_date_time.time }}"
+testname_unique: "{{ testname }}.{{lookup('pipe','date +%Y-%m-%d-%H:%M:%S')}}"
 
 test_out_dir: "{{ out_dir }}/{{ testname_unique }}"
 summary_file: "summary.loganalysis.{{ testname_unique }}.log"