Skip to content

Add hwsku Nokia-7215 support in port_utils.py#3028

Merged
yxieca merged 1 commit intosonic-net:masterfrom
rawal01:nokia-7215
Feb 26, 2021
Merged

Add hwsku Nokia-7215 support in port_utils.py#3028
yxieca merged 1 commit intosonic-net:masterfrom
rawal01:nokia-7215

Conversation

@rawal01
Copy link
Copy Markdown
Contributor

@rawal01 rawal01 commented Feb 23, 2021

Description of PR

Add hwsku Nokia-7215 to port_utils,py

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

Add support for Nokia-7215 hwsku in port_utils.py used to get port alias map

How did you do it?

Use existing mapping for et6448m as Nokia-7215 uses same mapping:

How did you verify/test it?

verified config facts and minigraph facts are correct for hwsku Nokia-7215

Any platform specific information?

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

N/A

Documentation

@rawal01 rawal01 requested a review from a team as a code owner February 23, 2021 17:05
@rawal01
Copy link
Copy Markdown
Contributor Author

rawal01 commented Feb 23, 2021

@sumukhatv can you please review this?

@yxieca yxieca merged commit 50c74bb into sonic-net:master Feb 26, 2021
@rawal01 rawal01 deleted the nokia-7215 branch March 19, 2021 17:50
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…atically (sonic-net#17173)

#### Why I did it
src/sonic-utilities
```
* 3609e417 - (HEAD -> 202305, origin/202305) [sonic-package-manager] do not modify config_db.json (sonic-net#3032) (2 hours ago) [Stepan Blyshchak]
* 354dfe80 - [sonic_installer]: Improve exception handling: introduce notes. (sonic-net#3028) (3 hours ago) [Nazarii Hnydyn]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants