Always compare med fix

[donaldsharp]

topotests have a create_route_map function which takes a dict of values, of which some tester has choosen to send med down which means nothing. Switch it to metric as it should be.

[greptile-apps]

Greptile Summary

This PR corrects a parameter naming issue in topotest route map configurations. The create_route_maps function expects metric as the key in the "set" dictionary, but several tests incorrectly used med, which is non-functional. The PR fixes all occurrences across BGP and OSPF topotests.

Additionally, in test_bgp_always_compare_med_topo1.py, the PR corrects next-hop router references from r2 to r3 on lines 505 and 566, ensuring tests verify the correct expected behavior.

• Fixed 4 instances of "med" to "metric" in BGP always compare MED test
• Fixed 9 instances of "med" to "metric" in OSPF route maps test
• Corrected next-hop verification to use r3 instead of r2 in two test functions

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The changes are straightforward test fixes that correct non-functional parameter names to their proper values. All changes align with the create_route_maps function API which expects metric not med. The next-hop corrections also improve test accuracy. No logic changes or production code affected.
• No files require special attention

Important Files Changed

Filename	Overview
tests/topotests/bgp_always_compare_med/test_bgp_always_compare_med_topo1.py	Fixed incorrect med parameter to correct metric parameter in route map configurations and corrected next-hop router reference from r2 to r3
tests/topotests/ospf_basic_functionality/test_ospf_routemaps.py	Fixed incorrect med parameter to correct metric parameter in route map set clauses across multiple test configurations

_{Last reviewed commit: ff9b005}

[Jafaral]

@Mergifyio backport dev/10.6

[mergify]

backport dev/10.6

✅ Backports have been created

Details

• #20916 Always compare med fix (backport #20909) has been created for branch dev/10.6