Skip to content

Commit 4cdfee0

Browse files
authored
Merge pull request #1638 from elezar/relax-refresh-service-restart
Remove restart logic from nvidia-cdi-refresh.service
2 parents 322454a + 5fe6b42 commit 4cdfee0

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# NVIDIA Container Toolkit Changelog
22

33
## v1.19.0-rc.3
4+
- Remove restart logic from nvidia-cdi-refresh.service.
45
- fix: Set device node GID in CDI specs
56
- fix: Log actual CDI spec version
67
- Make internal/system packages public

deployments/systemd/nvidia-cdi-refresh.service

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,7 @@ Description=Refresh NVIDIA CDI specification file
1717
ConditionPathExists=|/usr/bin/nvidia-smi
1818
ConditionPathExists=|/usr/sbin/nvidia-smi
1919
ConditionPathExists=/usr/bin/nvidia-ctk
20-
# Limit the number of successive restarts to 5 in 10 seconds.
21-
StartLimitBurst=5
22-
StartLimitIntervalSec=10s
20+
After=multi-user.target
2321

2422
[Service]
2523
Type=oneshot
@@ -29,10 +27,6 @@ EnvironmentFile=-/etc/nvidia-container-toolkit/nvidia-cdi-refresh.env
2927
ExecCondition=/usr/bin/grep -qE '/(nvidia|nvidia-current)\\.ko' /lib/modules/%v/modules.dep
3028
ExecStart=/usr/bin/nvidia-ctk cdi generate
3129
CapabilityBoundingSet=CAP_SYS_MODULE CAP_SYS_ADMIN CAP_MKNOD
32-
# We set the service to restart on failure to ensure that a CDI spec is
33-
# eventually generated.
34-
Restart=on-failure
35-
RestartSec=1s
3630

3731
[Install]
3832
WantedBy=multi-user.target

0 commit comments

Comments
 (0)