Merge ~liaou3/checkbox-iiotg/+git/checkbox-provider-intliotg:modify_pass_criteria into ~checkbox-dev/checkbox-iiotg/+git/checkbox-provider-intliotg:master

[Vincent Liao]

Fix: Set default pass criteria from 200 ns to 100 ns for time sync test

According to the page 27 in the tsn document [1] provided by intel, the pass criteria should be 100ns instead of 200ns.

[1] https://drive.google.com/file/d/1HsksCfLoUTz_baZU98omYuGBq-LDZKGe/view?usp=drive_link

If the testing environment is two machines connect to each other directly, the pass criteria for time sync test should be under 100 ns.

[Vic Liu (zongminl)]

Can you please share with us the rationale of this change? Any reference document?
If possible, can you please put that in the commit message

review: Needs Information

[Vincent Liao (liaou3)]

> Can you please share with us the rationale of this change? Any reference
> document?
> If possible, can you please put that in the commit message

@Vic, the pass criteria is always 100ns in page 27 in this tsn document [1] provided by intel. I accidentally type the wrong value while doing the job automation in previous commits. But don't worry, the pass criteria for previous tsn tests were 100ns.

[1] https://drive.google.com/file/d/1HsksCfLoUTz_baZU98omYuGBq-LDZKGe/view?usp=drive_link

[Vic Liu (zongminl)]

All good, thank you, but please add that rationale to your commit message

review: Approve

[PeiYao Chang (baconyao)]

LGTM

review: Approve

diff --git a/units/tsn/jobs.pxu b/units/tsn/jobs.pxu
index 719da83..c7b5416 100644
--- a/units/tsn/jobs.pxu
+++ b/units/tsn/jobs.pxu
@@ -172,12 +172,12 @@ command:
     while read -r line
     do
         offset=$(echo "$line" | grep -o 'master offset\s*-\?[0-9]\+ s' | awk '{print $3}')
-        if [ "$offset" -gt "${TSN_MAX_MASTER_OFFSET:-200}" ] || [ "$offset" -lt -"${TSN_MAX_MASTER_OFFSET:-200}" ]; then
-            echo "FAIL: master offset is not within -"${TSN_MAX_MASTER_OFFSET:-200}" to "${TSN_MAX_MASTER_OFFSET:-200}" for the last 10 seconds."
+        if [ "$offset" -gt "${TSN_MAX_MASTER_OFFSET:-100}" ] || [ "$offset" -lt -"${TSN_MAX_MASTER_OFFSET:-100}" ]; then
+            echo "FAIL: master offset is not within -"${TSN_MAX_MASTER_OFFSET:-100}" to "${TSN_MAX_MASTER_OFFSET:-100}" for the last 10 seconds."
             exit 1
         fi
     done <<< "$lines"
-    echo "PASS: master offset is within -"${TSN_MAX_MASTER_OFFSET:-200}" to "${TSN_MAX_MASTER_OFFSET:-200}" for the last 10 seconds."
+    echo "PASS: master offset is within -"${TSN_MAX_MASTER_OFFSET:-100}" to "${TSN_MAX_MASTER_OFFSET:-100}" for the last 10 seconds."
 
 unit: template
 template-resource: tsn/tsn-devices
@@ -209,12 +209,12 @@ command:
     while read -r line
     do
         offset=$(echo "$line" | grep -o 'master offset\s*-\?[0-9]\+ s' | awk '{print $3}')
-        if [ "$offset" -gt "${TSN_MAX_MASTER_OFFSET:-200}" ] || [ "$offset" -lt -"${TSN_MAX_MASTER_OFFSET:-200}" ]; then
-            echo "FAIL: master offset is not within -"${TSN_MAX_MASTER_OFFSET:-200}" to "${TSN_MAX_MASTER_OFFSET:-200}" for the last 10 seconds."
+        if [ "$offset" -gt "${TSN_MAX_MASTER_OFFSET:-100}" ] || [ "$offset" -lt -"${TSN_MAX_MASTER_OFFSET:-100}" ]; then
+            echo "FAIL: master offset is not within -"${TSN_MAX_MASTER_OFFSET:-100}" to "${TSN_MAX_MASTER_OFFSET:-100}" for the last 10 seconds."
             exit 1
         fi
     done <<< "$lines"
-    echo "PASS: master offset is within -"${TSN_MAX_MASTER_OFFSET:-200}" to "${TSN_MAX_MASTER_OFFSET:-200}" for the last 10 seconds."
+    echo "PASS: master offset is within -"${TSN_MAX_MASTER_OFFSET:-100}" to "${TSN_MAX_MASTER_OFFSET:-100}" for the last 10 seconds."
 
 unit: template
 template-resource: tsn/tsn-devices