Skip to content

Commit

Permalink
[sonic-mgmt] Remove incorrect quotes around issue url (#15204)
Browse files Browse the repository at this point in the history
PR#14912 has introduced single quotes around github issue url and this
is causing conditional mark evaluation to fail. Thus conditional mark
config is not honored and couple of tests are getting scheduled even
though they are supposed to be skipped.
  • Loading branch information
vkjammala-arista authored and mssonicbld committed Oct 31, 2024
1 parent 4a3295a commit d16eb08
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ bgp/test_bgp_slb.py::test_bgp_slb_neighbor_persistence_across_advanced_reboot:
reason: "Skip it on dual tor since it got stuck during warm reboot due to known issue on master and internal image
or over topologies which doesn't support slb."
conditions:
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56'] and 'https://github.com/sonic-net/sonic-mgmt/issues/9201'"
- "topo_name in ['dualtor', 'dualtor-56', 'dualtor-120', 'dualtor-aa', 'dualtor-aa-56'] and https://github.com/sonic-net/sonic-mgmt/issues/9201"
- "'backend' in topo_name or 'mgmttor' in topo_name"

bgp/test_bgp_speaker.py:
Expand Down Expand Up @@ -764,7 +764,7 @@ generic_config_updater/test_eth_interface.py::test_replace_fec:
reason: 'Skipping test on 7260/3800 platform due to bug of https://github.com/sonic-net/sonic-mgmt/issues/11237 / generic_config_updater is not a supported feature for T2'
conditions_logical_operator: "OR"
conditions:
- "hwsku in ['Arista-7260CX3-D108C8', 'Arista-7260CX3-Q64', 'Mellanox-SN3800-D112C8'] and 'https://github.com/sonic-net/sonic-mgmt/issues/11237'"
- "hwsku in ['Arista-7260CX3-D108C8', 'Arista-7260CX3-Q64', 'Mellanox-SN3800-D112C8'] and https://github.com/sonic-net/sonic-mgmt/issues/11237"
- "'t2' in topo_name"

generic_config_updater/test_eth_interface.py::test_toggle_pfc_asym:
Expand Down

0 comments on commit d16eb08

Please sign in to comment.