Skip to content

[action] [PR:4041] [smartswitch] Update get_gnmi_port() based on smartswitch config updates#4043

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/4041
Aug 26, 2025
Merged

[action] [PR:4041] [smartswitch] Update get_gnmi_port() based on smartswitch config updates#4043
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/4041

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

What I did

This PR updates the get_gnmi_port() function in the smartswitch reboot helper script to align with recent YANG model changes for DPU configuration. The function now retrieves the GNMI port from the updated CONFIG_DB schema.

https://github.com/sonic-net/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md#dpu-configuration

How I did it

  • Modified database key pattern from DPU_PORT|$DPU_NAME to DPU|*$DPU_NAME
  • Added default parameter value for DPU_NAME
  • Updated query method to iterate through matching keys

How to verify it

Run the reboot command with latest configuration changes.

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

<!--
    Please make sure you've read and understood our contributing guidelines:
    https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

    ** Make sure all your commits include a signature generated with `git commit -s` **

    If this is a bug fix, make sure your description includes "closes #xxxx",
    "fixes #xxxx" or "resolves #xxxx" so that GitHub automatically closes the related
    issue when the PR is merged.

    If you are adding/modifying/removing any command or utility script, please also
    make sure to add/modify/remove any unit tests from the tests
    directory as appropriate.

    If you are modifying or removing an existing 'show', 'config' or 'sonic-clear'
    subcommand, or you are adding a new subcommand, please make sure you also
    update the Command Line Reference Guide (doc/Command-Reference.md) to reflect
    your changes.

    Please provide the following information:
-->

#### What I did
This PR updates the get_gnmi_port() function in the smartswitch reboot helper script to align with recent YANG model changes for DPU configuration. The function now retrieves the GNMI port from the updated CONFIG_DB schema.

https://github.com/sonic-net/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md#dpu-configuration

#### How I did it
- Modified database key pattern from DPU_PORT|$DPU_NAME to DPU|*$DPU_NAME
- Added default parameter value for DPU_NAME
- Updated query method to iterate through matching keys

#### How to verify it
Run the reboot command with latest configuration changes.

#### Previous command output (if the output of a command-line utility has changed)

#### New command output (if the output of a command-line utility has changed)
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #4041

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 5546fe2 into sonic-net:202505 Aug 26, 2025
7 checks passed
kktheballer pushed a commit to kktheballer/sonic-utilities that referenced this pull request Jan 14, 2026
```<br>* a4d1eb9 - (HEAD -> 202506) Merge branch '202505' of https://github.com/sonic-net/sonic-utilities into 202506 (2025-08-27) [Sonic Automation]
* 5546fe2 - (origin/202505) [smartswitch] Update get_gnmi_port() based on smartswitch config updates (sonic-net#4043) (2025-08-26) [mssonicbld]<br>```
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.

1 participant