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

Enable NVIDIA Persistence Daemon by default #147

Merged
merged 1 commit into from
Sep 26, 2023
Merged

Enable NVIDIA Persistence Daemon by default #147

merged 1 commit into from
Sep 26, 2023

Conversation

SreeeS
Copy link
Contributor

@SreeeS SreeeS commented Sep 26, 2023

Summary

This PR is to enable NVIDIA Persistence Daemon by default

Implementation details

Added sudo systemctl enable nvidia-persistenced to scripts/enable-ecs-agent-gpu-support.sh

Testing

  • REGION=us-west-2 make al2gpu
  • Launch instance with test AMI and verify nvidia-persistenced.service is enabled, active (running)
systemctl status nvidia-persistenced
● nvidia-persistenced.service - NVIDIA Persistence Daemon
   Loaded: loaded (/usr/lib/systemd/system/nvidia-persistenced.service; enabled; vendor preset: disabled)
   Active: active (running) since Tue 2023-09-26 04:07:02 UTC; 1min 7s ago
 Main PID: 4705 (nvidia-persiste)
    Tasks: 1
   Memory: 28.1M
   CGroup: /system.slice/nvidia-persistenced.service
           └─4705 /usr/bin/nvidia-persistenced --verbose
  • Run functional set suite against the AMIs created and verify exec tests succeed

New tests cover the changes: No new tests added

Description for the changelog

Enable NVIDIA Persistence Daemon by default

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@SreeeS SreeeS marked this pull request as ready for review September 26, 2023 15:35
singholt
singholt previously approved these changes Sep 26, 2023
scripts/enable-ecs-agent-gpu-support.sh Outdated Show resolved Hide resolved
mythri-garaga
mythri-garaga previously approved these changes Sep 26, 2023
@SreeeS SreeeS merged commit f219269 into aws:main Sep 26, 2023
@prateekchaudhry prateekchaudhry mentioned this pull request Sep 29, 2023
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.

4 participants