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

Update openconfig-keychain-types.yang #1173

Merged
merged 1 commit into from
Aug 27, 2024
Merged

Conversation

tengyiG
Copy link
Contributor

@tengyiG tengyiG commented Aug 27, 2024

[Note: Please fill out the following template for your pull request. lines

tagged with "Note" can be removed from the template.]

[Note: Before this PR can be reviewed please agree to the CLA covering this

repo. Please also review the contribution guide -

https://github.com/openconfig/public/blob/master/doc/external-contributions-guide.md]

Change Scope

  • This change adds new identity definitions for AES-CMAC algorithms to the OpenConfig crypto-types model. Specifically, it adds AES_128_CMAC, AES_256_CMAC identities.

  • This change is backwards compatible as it only adds new identities and does not modify existing structures or semantics.

Platform Implementations

@tengyiG tengyiG requested a review from a team as a code owner August 27, 2024 18:37
Copy link

google-cla bot commented Aug 27, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@dplore
Copy link
Member

dplore commented Aug 27, 2024

/gcbrun

@OpenConfigBot
Copy link

No major YANG version changes in commit f36290f

@dplore dplore merged commit abba929 into openconfig:master Aug 27, 2024
14 checks passed
sallylsy pushed a commit to sallylsy/OC_public that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants