Skip to content

Conversation

DeanKalla
Copy link
Contributor

First Draft of MACsec SAI API Proposal Document.

@kcudnik
Copy link
Collaborator

kcudnik commented Nov 6, 2019

@DeanKalla
Copy link
Contributor Author

Warnings have been resolved but there is one issue with the build I am not sure how to resolve. I'm new to this codebase and build, could someone help me understand what this metadata build error means?

ERROR: FATAL ERROR === MUST FIX === : Could not open file 'xml/xml/union__sai__macsec__auth__key__t.xml' No such file or directory at xmlutils.pm line 163.

@kcudnik
Copy link
Collaborator

kcudnik commented Nov 21, 2019

  • change SAI_MACSEC_SA_ATTR_DIRECTION to SAI_MACSEC_SA_ATTR_MASCEC_DIRECTION on condition
  • also conditions on stat enums are not supported
  • SAI_OBJECT_TYPE_MAX must have consecutive number after SAI_OBJECT_TYPE_MACSEC_SA
  • fix spelling
  • add SAI_API_MACSEC entry in sai_api_t
  • you introduced 4 new types, you will have to add 4 serialize and 4 deserialize methods (look at meta/saiseralise.[ch] and at compiler errors
  • change SAI_ATTR_VALUE_TYPE_ACL_FIELD_DATA_SCI to SAI_ATTR_VALUE_TYPE_ACL_FIELD_DATA_MACSEC_SCI

i added some enhancements to parser to minimize your impact #1015

please marge and retest and fix errors

@kcudnik
Copy link
Collaborator

kcudnik commented Nov 21, 2019

retest this please

@prafull-brcm
Copy link
Contributor

In sai.h header file one change is required to add MACsec APIs.

SAI_API_MACSEC           = 43, /**< sai_macsec_api_t */
SAI_API_MAX              = 44, /**< total number of APIs */

} sai_api_t;

Copy link

@tfangit tfangit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thx for the work.

@DeanKalla
Copy link
Contributor Author

Can we merge this PR as-is? We can make more PR for other enhancements.

Copy link
Contributor

@SidharajU SidharajU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to revisit if there is a requirement to support upfront SA creation.

@lguohan lguohan merged commit 60e2108 into opencomputeproject:master Dec 30, 2019
@DeanKalla DeanKalla deleted the macsec-sai-spec branch January 3, 2020 17:26
Kalimuthu-Velappan pushed a commit to Kalimuthu-Velappan/SAI that referenced this pull request Feb 6, 2020
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.

9 participants