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

nfd: Update NFD CR's to support SGX EPC #252

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

vbedida79
Copy link
Contributor

@vbedida79 vbedida79 commented Jun 7, 2024

Fixes #255
Removed SGX extraLabelNs and resourceLabels from NodeFeatureDiscovery CR. Added SGX EPC as

extendedResources:
        sgx.intel.com/epc: "@cpu.security.sgx.epc" 

in the NodeFeatureRule CR.
With these changes, initContainer in SGXDevicePlugin CR wont be used (in Intel device plugins 0.29.0)
Signed-off-by: vbedida79 [email protected]

Copy link
Contributor

@uMartinXu uMartinXu left a comment

Choose a reason for hiding this comment

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

It is a good job.
Could you add more technical detail about this PR in the commit log?
It is great if we have a github issue in the commit log for detail about this fixing.
Thanks!

@vbedida79
Copy link
Contributor Author

It is a good job. Could you add more technical detail about this PR in the commit log? It is great if we have a github issue in the commit log for detail about this fixing. Thanks!

Created #255, added in commit log

@vbedida79 vbedida79 closed this Jun 10, 2024
@vbedida79 vbedida79 reopened this Jun 10, 2024
@uMartinXu uMartinXu merged commit fe4df14 into intel:main Jun 10, 2024
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.

NFD SGX EPC extended resource without using SGX device plugin init container
3 participants