Skip to content

[iface_namingmode] Convert iface_speed to string#2203

Merged
lolyu merged 1 commit intosonic-net:masterfrom
lolyu:fix_config_speed
Sep 11, 2020
Merged

[iface_namingmode] Convert iface_speed to string#2203
lolyu merged 1 commit intosonic-net:masterfrom
lolyu:fix_config_speed

Conversation

@lolyu
Copy link
Collaborator

@lolyu lolyu commented Sep 11, 2020

Description of PR

Convert iface_speed that from Ansible variables to string to align
with the return from port_alias.

Signed-off-by: Longxiang Lyu lolv@microsoft.com

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

test_config_interface_speed failed over 7060.

How did you do it?

How did you verify/test it?

run test_config_interface_speed over 7060.

Any platform specific information?

Arista 7060.

Supported testbed topology if it's a new test case?

Documentation

Convert `iface_speed` that from Ansible variables to string to align
with the return from `port_alias`.

Signed-off-by: Longxiang Lyu <lolv@microsoft.com>
@lolyu lolyu added the Bug 🐛 label Sep 11, 2020
@lolyu lolyu requested a review from a team September 11, 2020 03:16
@lolyu lolyu merged commit 647b57e into sonic-net:master Sep 11, 2020
@lolyu lolyu deleted the fix_config_speed branch September 11, 2020 05:24
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
* [202012] - sonic-swss submodule update to include following commits:

fca407a (HEAD) [VNET]Fixing nexthop group delete during route change (sonic-net#2198)
a9b6b47 [vxlan] Remove tunnel map objects on VNET tunnel removal (sonic-net#2208)
74e9b9f [FdbOrch] SAI_FDB_EVENT_MOVE generates update with empty update.entry.port_name (sonic-net#2201)
0a99445 [202012][BFD]Registering BFD state change callback during session creation (sonic-net#2203)
aebe4a1 [VS test] skip dpb flaky test (sonic-net#2195) (sonic-net#2207)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants