Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add cert revocation list support to GNMI service. #19536

Merged
merged 7 commits into from
Nov 22, 2024

Conversation

liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Jul 11, 2024

Add GNMI CRL support.

Why I did it

Allow gnmi service revoke client cert with CRL

Work item tracking
  • Microsoft ADO: 27146924

How I did it

Add GNMI CRL config to yang model.

How to verify it

Pass all UT.
Verified with new sonic-mgmt test case.

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)

will updated with this PR image later.

  • [] SONiC.master-16482.360728-2c8b4066f

Description for the changelog

Add GNMI CRL support.

Link to config_db schema for YANG module changes

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

@liuh-80 liuh-80 changed the title Add GNMI CRL support [POC] Add GNMI CRL support. Jul 11, 2024
@liuh-80 liuh-80 force-pushed the dev/liuh/enable-crl-check branch from acce004 to 45203eb Compare September 24, 2024 07:09
@liuh-80 liuh-80 changed the title [POC] Add GNMI CRL support. Add GNMI CRL support. Nov 20, 2024
@liuh-80
Copy link
Contributor Author

liuh-80 commented Nov 20, 2024

GNMI side change is merged sonic-net/sonic-gnmi#269, after submodule update, will update this PR and publish for review.

@liuh-80 liuh-80 marked this pull request as ready for review November 22, 2024 07:09
@liuh-80 liuh-80 requested a review from ganglyu November 22, 2024 07:09
@liuh-80 liuh-80 changed the title Add GNMI CRL support. Add cert revocation list support to GNMI service. Nov 22, 2024
@qiluo-msft qiluo-msft merged commit 886500a into sonic-net:master Nov 22, 2024
22 checks passed
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