Skip to content

[action] [PR:12851] Fix m0-2vlan topo prefix_v6 value that causes route_check fail#12935

Merged
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12851
May 22, 2024
Merged

[action] [PR:12851] Fix m0-2vlan topo prefix_v6 value that causes route_check fail#12935
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12851

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary: Explicit zeros in prefix_v6 value causes issue in route_check
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

The prefix_v6 value fc02:1000:0:0::1/64 is not simplified and at some point there is logic that populates APPL_DB INTF_TABLE with fc02:1000:0:0::1/128 based on this prefix_v6 value. However, for some reason the equivalent, simplified form of the APPL_DB entry is what appears in the ASIC_DB ROUTE_ENTRY_TABLE: fc02:1000::1/128. The route_check is not smart enough to recognize that fc02:1000:0:0::1/128 and fc02:1000::1/128 are equivalent and this causes the error.

How did you do it?

Changed prefix_v6 value in the topo file to the simplified form

How did you verify/test it?

Verified route_check passes on Arista 720DT-48S

Any platform specific information?

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

Documentation

…-net#12851)

Explicit zeros in prefix_v6 value causes issue in route_check. Need to use simplified form instead
@mssonicbld
Copy link
Collaborator Author

Original PR: #12851

@mssonicbld mssonicbld merged commit bdf7b09 into sonic-net:202311 May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants