UniqAI/nvidia-container-toolkit
1
0
Fork 0
mirror of https://github.com/NVIDIA/nvidia-container-toolkit synced 2024-11-25 13:35:00 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge branch 'fix-ld.so.conf-permissions' into 'main'

Browse Source 
Create ld.so.conf file with permissions 644

See merge request nvidia/container-toolkit/container-toolkit!380
This commit is contained in:
Evan Lezar 2023-05-02 10:51:40 +00:00
commit fe8ef9e0bd
2 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGELOG.md
View File

@ -3,6 +3,7 @@
## v1.14.0-rc.1 ## v1.14.0-rc.1
* Add support for updating containerd configs to the `nvidia-ctk runtime configure` command. * Add support for updating containerd configs to the `nvidia-ctk runtime configure` command.
* Create file in `etc/ld.so.conf.d` with permissions `644` to support non-root containers.
## v1.13.1 ## v1.13.1

5
cmd/nvidia-ctk/hook/update-ldcache/update-ldcache.go
View File

@ -135,5 +135,10 @@ func (m command) createConfig(root string, folders []string) error {
configured[folder] = true configured[folder] = true
} }
// The created file needs to be world readable for the cases where the container is run as a non-root user.
if err := os.Chmod(configFile.Name(), 0644); err != nil {
return fmt.Errorf("failed to chmod config file: %v", err)
}
return nil return nil
} }