Skip to content

Enable gnmi/telemetry user authorization by config_db#21363

Merged
qiluo-msft merged 19 commits intosonic-net:masterfrom
liuh-80:dev/liuh/enable_gnmi_cert_auth
Feb 20, 2025
Merged

Enable gnmi/telemetry user authorization by config_db#21363
qiluo-msft merged 19 commits intosonic-net:masterfrom
liuh-80:dev/liuh/enable_gnmi_cert_auth

Conversation

@liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Jan 9, 2025

Enable gnmi/telemetry user authorization by config_db.

Why I did it

gnmi/telemetry user authorization flag not used in SONiC and can't config by config_db

Work item tracking
  • Microsoft ADO: 30808849

How I did it

Update Yang model and update gnmi/telemetry start script.

How to verify it

Pass all UT.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • []

Description for the changelog

Enable gnmi/telemetry user authorization by config_db.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80 liuh-80 requested a review from ganglyu January 9, 2025 07:23
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Jan 10, 2025

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@qiluo-msft qiluo-msft merged commit 4641562 into sonic-net:master Feb 20, 2025
21 checks passed
miatttao pushed a commit to miatttao/sonic-buildimage that referenced this pull request Mar 5, 2025
Enable gnmi/telemetry user authorization by config_db.

Why I did it
gnmi/telemetry user authorization flag not used in SONiC and can't config by config_db

How I did it
Update Yang model and update gnmi/telemetry start script.

How to verify it
Pass all UT.
@qiluo-msft
Copy link
Collaborator

@bingwang-ms Could you backport 202405?
@kperumalbfn Could you backport 202411?

liuh-80 added a commit to liuh-80/sonic-buildimage that referenced this pull request Jul 8, 2025
Enable gnmi/telemetry user authorization by config_db.

Why I did it
gnmi/telemetry user authorization flag not used in SONiC and can't config by config_db

How I did it
Update Yang model and update gnmi/telemetry start script.

How to verify it
Pass all UT.
@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 8, 2025

PR for 202411: #23248

liuh-80 added a commit to liuh-80/sonic-buildimage that referenced this pull request Jul 8, 2025
Enable gnmi/telemetry user authorization by config_db.

Why I did it
gnmi/telemetry user authorization flag not used in SONiC and can't config by config_db

How I did it
Update Yang model and update gnmi/telemetry start script.

How to verify it
Pass all UT.
liuh-80 added a commit to liuh-80/sonic-buildimage that referenced this pull request Jul 8, 2025
Enable gnmi/telemetry user authorization by config_db.

Why I did it
gnmi/telemetry user authorization flag not used in SONiC and can't config by config_db

How I did it
Update Yang model and update gnmi/telemetry start script.

How to verify it
Pass all UT.
@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 8, 2025

202405 cherry-pick PR: #23249

@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 8, 2025

202311 cherry-pick PR: #23252

@liuh-80 liuh-80 deleted the dev/liuh/enable_gnmi_cert_auth branch July 8, 2025 05:50
kperumalbfn pushed a commit that referenced this pull request Jul 9, 2025
Enable gnmi/telemetry user authorization by config_db
yxieca pushed a commit that referenced this pull request Jul 10, 2025
Enable gnmi/telemetry user authorization by config_db.

Why I did it
gnmi/telemetry user authorization flag not used in SONiC and can't config by config_db

How I did it
Update Yang model and update gnmi/telemetry start script.

How to verify it
Pass all UT.
bingwang-ms pushed a commit that referenced this pull request Jul 10, 2025
Enable gnmi/telemetry user authorization by config_db.

Why I did it
gnmi/telemetry user authorization flag not used in SONiC and can't config by config_db

How I did it
Update Yang model and update gnmi/telemetry start script.

How to verify it
Pass all UT.
patrickmacarthur pushed a commit to patrickmacarthur/sonic-buildimage that referenced this pull request Aug 6, 2025
…05 (sonic-net#1339)

```<br>* f6dfb9d - (HEAD -> 202405) Merge branch '202405' of https://github.com/sonic-net/sonic-buildimage into 202405 (2025-07-11) [Sonic Automation]
* cc8d6de - (head/202405) Enable gnmi/telemetry user authorization by config_db (sonic-net#21363) (sonic-net#23249) (2025-07-10) [Hua Liu]
* c7e79bb - [submodule] Update submodule sonic-linux-kernel to the latest HEAD automatically (sonic-net#23283) (2025-07-11) [mssonicbld]<br>```
tshalvi pushed a commit to tshalvi/sonic-buildimage that referenced this pull request Aug 25, 2025
…12 (sonic-net#1335)

```<br>* 37bad37 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-buildimage into 202412 (2025-07-15) [Sonic Automation]
* 718490f - (head/202411) [submodule] Update submodule sonic-linux-kernel to the latest HEAD automatically (sonic-net#23287) (2025-07-11) [mssonicbld]
* 258389d - Enable gnmi/telemetry user authorization by config_db (sonic-net#21363) (sonic-net#23248) (2025-07-09) [Hua Liu]
* 31fdefd - Including platform_utils in make and removing default initialization of $SMARTSWITCH in TSA, TSB, TSC (sonic-net#23258) (2025-07-09) [mssonicbld]
* f5332b0 - [build] Use package.trafficmanager.net instead of sonicstorage.blob.core.windows.net to Refine Azure resource. (sonic-net#23256) (2025-07-09) [Liu Shilong]
* 07ba952 - [submodule] Update submodule sonic-platform-common to the latest HEAD automatically (sonic-net#23214) (2025-07-07) [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.

5 participants