Skip to content

[action] [PR:3250] T2-VOQ-VS: Modified exception handling due to new sonic_platform package support for VS (#3250)#3310

Merged
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/3250
May 9, 2024
Merged

[action] [PR:3250] T2-VOQ-VS: Modified exception handling due to new sonic_platform package support for VS (#3250)#3310
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/3250

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

What I did

For T2-Chassis VS support, we are adding new sonic_platform package for vs platforms. Please refer sonic-net/sonic-buildimage#18512 for more details.
Due to this new platform package, need to modify excpetion handling as now the Module would be found, but the metadata file will not be found for pizzabox vs platforms.

How I did it

Modified the exception handling logic.
MSFT ADO: 27414904

How to verify it

Bring up vms-kvm-t0 topology. ran show interface status. The output is proper.

PS: the Main PR(sonic-net/sonic-buildimage#18512) is dependent on this PR to be merged in first.

…age support for VS (sonic-net#3250)

### What I did
For T2-Chassis VS support, we are adding new sonic_platform package for vs platforms. Please refer sonic-net/sonic-buildimage#18512 for more details.
Due to this new platform package, need to modify excpetion handling as now the Module would be found, but the metadata file will not be found for pizzabox vs platforms.

#### How I did it
Modified the exception handling logic.
MSFT ADO: 27414904

#### How to verify it
Bring up vms-kvm-t0 topology. ran show interface status. The output is proper.

PS: the Main PR(sonic-net/sonic-buildimage#18512) is dependent on this PR to be merged in first.
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #3250

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