-
Notifications
You must be signed in to change notification settings - Fork 347
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
T5770 Enable MACsec encryption stanza #2518
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's odd that we did have a node in the CLI definition but not this...
I did check to make sure I didn't accidentally delete some config from my previous PR. I swear I remember it encrypting in 1.4 when I was testing the previous change. Not sure how it ended up not setting that value now. |
Extended the commit and added the appropriate smoketests
|
@Mergifyio backport sagitta |
✅ Backports have been created
|
Change Summary
Enables encryption mode when using MACsec
Types of changes
Related Task(s)
Related PR(s)
Component(s) name
MACSEC
Proposed changes
Enable encryption when creating MACsec interface using "ip link" command
How to test
Smoketest result
Checklist: