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

NRI pod can't access the device /dev/isst_interface #101

Open
changzhi1990 opened this issue Aug 4, 2023 · 3 comments
Open

NRI pod can't access the device /dev/isst_interface #101

changzhi1990 opened this issue Aug 4, 2023 · 3 comments

Comments

@changzhi1990
Copy link
Contributor

Hi, all.

Based on my testing about the sst feature in the topology-aware policy. I found that there some problems in the NRI pod.

The NRI pod can't find the /host/dev/isst_interface device.

image

After some research, I add these lines to the NRI daemonset.
image

Then, the NRI pod has no permission to access this device:

W0804 01:56:49.287017       1 system.go:297] failed to get SST info for package 0: failed to read SST PP info: Mbox command failed with failed to open isst device "/host/dev/isst_interface": open /host/dev/isst_interface: operation not permitted

After that, I noticed that there are some securitycontext in the daemonset file and I modified it:
image

I added the privileged: true into it and I commented the next two lines. At last, the NRI can access the sst device:
image

So does my approach was correct?

@changzhi1990 changzhi1990 changed the title NRI pod can't access the /dev/isst_interface NRI pod can't access the device /dev/isst_interface Aug 7, 2023
@marquiz
Copy link
Collaborator

marquiz commented Aug 7, 2023

Yes, looks about right. Running privileged should be an option (default to false) in the Helm chart

@changzhi1990
Copy link
Contributor Author

Yes, looks about right. Running privileged should be an option (default to false) in the Helm chart

Do we need to create a pr to fix it?

@marquiz
Copy link
Collaborator

marquiz commented Aug 16, 2023

Do we need to create a pr to fix it?

Yes, we need that. Add new options to the Helm chart to enable privileged mode and mount the host-dev

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

No branches or pull requests

2 participants