Skip to content

Commit 3b2fa09

Browse files
Issue #21208: Enable sFlow for VS tests (#21209)
* Issue #21208: Enable sFlow for VS tests Currently, sflow/test_sflow.py is disabled for virtual devices and is not being tested. However, the test runs and passes just fine on virtual devices when the sflow feature is enabled. Remove the skip condition and enable sflow during the test when run on virtual devices. * Remove hard-coded vs topology clause This clause is not needed, since sflow should be enabled by default on vs topos.
1 parent 649e487 commit 3b2fa09

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

tests/common/plugins/conditional_mark/tests_mark_conditions.yaml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4371,14 +4371,6 @@ scripts:
43714371
#######################################
43724372
##### sflow #####
43734373
#######################################
4374-
sflow/test_sflow.py:
4375-
skip:
4376-
reason: "Sflow feature is default disabled on vs platform and it is not tested for now"
4377-
conditions_logical_operator: or
4378-
conditions:
4379-
- "asic_type in ['vs']"
4380-
- "True"
4381-
43824374
sflow/test_sflow.py::TestReboot::testFastreboot:
43834375
skip:
43844376
reason: "Dualtor topology doesn't support advanced-reboot"

0 commit comments

Comments
 (0)