Update KeyChain model to accept hex string for key-id
Closed this issue · 3 comments
m26singhvi commented
As part of #606, Macsec model was updated to use global key chain model. However, this will not work because key-id type in key chain model is U64 :
leaf key-id {
type uint64;
description
"Identifier for the key within the keychain.";
}
whereas prior to the above change, key id in Macsec was hex string as per below :
grouping macsec-mka-key-config {
description
"MKA Key config grouping";
leaf id {
type oc-yang:hex-string {
length "1..64";
}
description
"Key identifier is used as the
Connectivity Association Key name (CKN)";
}
So, currently this is broken and proposal is to make the key-id
in keychain to be a union of hex string and U64.
dplore commented
Thanks @m26singhvi , can you raise a pull request to add this? Please also reference at least two implementations and/or specifications that show the need for a hex-string as macsec key id.
m26singhvi commented