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

[gpu] merge install_gpu_driver.sh from custom-images #1251

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

cjac
Copy link
Contributor

@cjac cjac commented Oct 28, 2024

* add is_debuntu to simplify common is_debian || is_ubuntu use case
* prepend clean to execute_with_retries to reduce maximum image size
* downloading assets to ram disk to reduce maximum image size
* added sync to key points of the script to allow accurate measurement
  of maximum disk usage
* moved initialization steps to new function prepare_to_install
* created exit_handler function which gets executed before script exits
* reduce log noise
* enable dkms certificate use anywhere in main function
* this version tested to work with secure boot enabled on all but debian10 and debian11 variants when launched with an image customized with [build-current-images.sh](https://github.com/cjac/custom-images/blob/test-dask-rapids-20241021/examples/secure-boot/build-current-images.sh)

@cjac cjac self-assigned this Oct 28, 2024
@cjac
Copy link
Contributor Author

cjac commented Oct 28, 2024

/gcbrun

@cjac
Copy link
Contributor Author

cjac commented Oct 28, 2024

that machine type is way too small. n1-standard-8 is the correct size for this installer.

@cjac
Copy link
Contributor Author

cjac commented Oct 28, 2024

/gcbrun

@cjac
Copy link
Contributor Author

cjac commented Oct 29, 2024

Everything in the last one worked except for rocky9. for some reason, the installation of screen is causing the internet to break. I've added a retry for failed dnf install commands. I've also increased the timeout from 30m to 90m to make sure rocky9 has enough time to do the installation.

@cjac
Copy link
Contributor Author

cjac commented Oct 29, 2024

/gcbrun

@cjac
Copy link
Contributor Author

cjac commented Oct 29, 2024

/gcbrun

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.

1 participant