[pr check] [masic] Add sanity check and tests, also mark multi-asic P…

…R check as required (sonic-net#6310) * add loganalyzer and tests to masic pr check * Mark multi-asic PR check as required
Azarack · Oct 17, 2022 · 3799052 · 3799052
1 parent 96f42ba
commit 3799052
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/azure-pipelines.yml b/azure-pipelines.yml
@@ -194,7 +194,7 @@ stages:
     pool: sonictest-ma
     displayName: "kvmtest-multi-asic-t1-lag"
     timeoutInMinutes: 400
-    continueOnError: true
+    continueOnError: false
     steps:
       - template: .azure-pipelines/run-test-template.yml
         parameters:

diff --git a/tests/kvmtest.sh b/tests/kvmtest.sh
@@ -277,11 +277,19 @@ test_multi_asic_t1_lag() {
 test_multi_asic_t1_lag_pr() {
     tgname=multi_asic_t1_lag
     tests="\
-    bgp/test_bgp_fact.py"
+    bgp/test_bgp_fact.py \
+    snmp/test_snmp_default_route.py \
+    snmp/test_snmp_loopback.py \
+    snmp/test_snmp_pfc_counters.py \
+    snmp/test_snmp_queue.py \
+    tacacs/test_accounting.py \
+    tacacs/test_authorization.py \
+    tacacs/test_jit_user.py \
+    tacacs/test_ro_user.py \
+    tacacs/test_rw_user.py"
 
     pushd $SONIC_MGMT_DIR/tests
-    # TODO: Remove disable of loganaler and sanity check once multi-asic testbed is stable.
-    ./run_tests.sh $RUNTEST_CLI_COMMON_OPTS -c "$tests" -p logs/$tgname -e --disable_loganalyzer -u
+    ./run_tests.sh $RUNTEST_CLI_COMMON_OPTS -c "$tests" -p logs/$tgname -u
     popd
 }