Skip to content

[submodule][master] Update submodule sonic-gnmi to the latest HEAD automatically#28135

Merged
mssonicbld merged 1 commit into
sonic-net:masterfrom
mssonicbld:submodule-master-sonic-gnmi
Jun 29, 2026
Merged

[submodule][master] Update submodule sonic-gnmi to the latest HEAD automatically#28135
mssonicbld merged 1 commit into
sonic-net:masterfrom
mssonicbld:submodule-master-sonic-gnmi

Conversation

@mssonicbld

Copy link
Copy Markdown
Collaborator

Why I did it

src/sonic-gnmi

* 48b9821 - (HEAD -> master, origin/master, origin/HEAD) telemetry: enforce --noTLS requires loopback --bind_address (#712) (23 hours ago) [xq9mend]

How I did it

How to verify it

Description for the changelog

Signed-off-by: mssonicbld <sonicbld@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator Author

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
mssonicbld merged commit 8a5a673 into sonic-net:master Jun 29, 2026
27 checks passed
xq9mend added a commit to xq9mend/sonic-buildimage that referenced this pull request Jun 29, 2026
Bumps src/sonic-gnmi to 67890d7 which re-adds the enforcement that
--noTLS requires --bind_address to be a loopback address.

The telemetry.sh and gnmi-native.sh scripts already pass
--bind_address 127.0.0.1 in the --noTLS fallback (since sonic-net#28110),
so existing behavior is unchanged. This commit makes the binary
enforce the constraint so future usages cannot accidentally expose
cleartext gRPC over the network.

Note: sonic-net#28135 auto-bumped sonic-gnmi to
the sonic-net canonical fork. This PR tracks the xq9mend fork SHA
for reference.

Signed-off-by: xq9mend <xq9mend@users.noreply.github.com>
roger-nexthop pushed a commit to nexthop-ai/sonic-buildimage that referenced this pull request Jul 12, 2026
…lly (sonic-net#28135)

#### Why I did it
src/sonic-gnmi
```
* 48b9821 - (HEAD -> master, origin/master, origin/HEAD) telemetry: enforce --noTLS requires loopback --bind_address (sonic-net#712) (23 hours ago) [xq9mend]
```
#### 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant