Skip to content

Update port_utils for DellEMC-Z9332f-C32#7661

Merged
lizhijianrd merged 1 commit intosonic-net:masterfrom
lizhijianrd:port-util-z9332f-c32
Mar 7, 2023
Merged

Update port_utils for DellEMC-Z9332f-C32#7661
lizhijianrd merged 1 commit intosonic-net:masterfrom
lizhijianrd:port-util-z9332f-c32

Conversation

@lizhijianrd
Copy link
Contributor

Description of PR

Summary:
Update port_utils.py to support generating name/alias mapping for DellEMC-Z9332f-C32.

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

Update port_utils.py to support generating name/alias mapping for DellEMC-Z9332f-C32.

How did you do it?

Update port_utils.py according to port_config.ini

How did you verify/test it?

Verified on Dell Z9332f switches with 100G cables connected.

Any platform specific information?

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

Documentation

@lizhijianrd lizhijianrd merged commit 782e38d into sonic-net:master Mar 7, 2023
@lizhijianrd lizhijianrd deleted the port-util-z9332f-c32 branch March 7, 2023 10:59
wangxin pushed a commit that referenced this pull request Mar 9, 2023
What is the motivation for this PR?
Update port_utils.py to support generating name/alias mapping for DellEMC-Z9332f-C32.

How did you do it?
Update port_utils.py according to port_config.ini

How did you verify/test it?
Verified on Dell Z9332f switches with 100G cables connected.

Signed-off-by: Zhijian Li <zhijianli@microsoft.com>
wangxin pushed a commit that referenced this pull request Mar 9, 2023
What is the motivation for this PR?
Update port_utils.py to support generating name/alias mapping for DellEMC-Z9332f-C32.

How did you do it?
Update port_utils.py according to port_config.ini

How did you verify/test it?
Verified on Dell Z9332f switches with 100G cables connected.

Signed-off-by: Zhijian Li <zhijianli@microsoft.com>
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.

3 participants