Skip to content

Commit

Permalink
Update nvidia-gpu-reset.target setup
Browse files Browse the repository at this point in the history
This PR fixes an incorrect entry in the
nvidia-mig-manager.service. The "WantedBy" needs
to point to multi-uer.targer otherwise unless
something "Requires" or "Wants" nvidia-mig-manager
it will not ever be started.

Also the original patch that added the target did not
update the debian Makefile and RPM spec file. This
resulted in the target not being installed.

Signed-off-by: KeithMnemonic <[email protected]>
  • Loading branch information
KeithMnemonic committed Sep 5, 2024
1 parent 95697ca commit 5fece20
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion deployments/systemd/nvidia-mig-manager.service
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,4 @@ Type=oneshot
ExecStart=-/bin/bash /etc/nvidia-mig-manager/service.sh

[Install]
WantedBy=nvidia-gpu-reset.target
WantedBy=multi-user.target
2 changes: 2 additions & 0 deletions deployments/systemd/packages/debian/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ SOURCE7 = hooks.sh
SOURCE8 = hooks-default.yaml
SOURCE9 = hooks-minimal.yaml
SOURCE10 = config-default.yaml
SOURCE11 = nvidia-gpu-reset.target

build:

Expand All @@ -45,3 +46,4 @@ install:
install -m 644 -t $(DESTDIR)/etc/nvidia-mig-manager $(SOURCE8)
install -m 644 -t $(DESTDIR)/etc/nvidia-mig-manager $(SOURCE9)
install -m 644 -t $(DESTDIR)/etc/nvidia-mig-manager $(SOURCE10)
install -m 644 -t $(DESTDIR)/lib/systemd/system $(SOURCE11)
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ Source7: hooks.sh
Source8: hooks-default.yaml
Source9: hooks-minimal.yaml
Source10: config-default.yaml
Source11: nvidia-gpu-reset.target

%description
The NVIDIA MIG Partition Editor allows administrators to declaratively define a
Expand All @@ -38,7 +39,7 @@ cp %{SOURCE0} %{SOURCE1} \
%{SOURCE4} %{SOURCE5} \
%{SOURCE6} %{SOURCE7} \
%{SOURCE8} %{SOURCE9} \
%{SOURCE10} \
%{SOURCE10} %{SOURCE11} \
.

%install
Expand All @@ -59,6 +60,7 @@ install -m 644 -t %{buildroot}/etc/nvidia-mig-manager %{SOURCE7}
install -m 644 -t %{buildroot}/etc/nvidia-mig-manager %{SOURCE8}
install -m 644 -t %{buildroot}/etc/nvidia-mig-manager %{SOURCE9}
install -m 644 -t %{buildroot}/etc/nvidia-mig-manager %{SOURCE10}
install -m 644 -t %{buildroot}/usr/lib/systemd/system %{SOURCE11}

%files
%license LICENSE
Expand All @@ -75,6 +77,7 @@ install -m 644 -t %{buildroot}/etc/nvidia-mig-manager %{SOURCE10}
%dir /etc/systemd/system/nvidia-mig-manager.service.d
%dir /etc/nvidia-mig-manager/
%dir /var/lib/nvidia-mig-manager
/usr/lib/systemd/system/nvidia-gpu-reset.target

%post
systemctl daemon-reload
Expand Down

0 comments on commit 5fece20

Please sign in to comment.