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

[secure-boot] merge changes from init-actions #106

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

Conversation

cjac
Copy link
Contributor

@cjac cjac commented Oct 30, 2024

  • include metadata when performing compute instance create
  • --trusted-cert is only used to disable shielded-secure-boot
  • reduce noise in installer ; also put long job timer and log redirection in execute_with_retries
  • installer dir is pointed to by ${tmpdir}
  • require 10.5G of free memory before using the ramdisk acceleration

@cjac cjac self-assigned this Oct 30, 2024
README.md
* clarified that passing an empty string to --trusted-cert will
  disable secure-boot
custom_image_utils/shell_script_generator.py
* execute_with_retries reduces log noise and emits timing information
* inject secret metadata unless --trusted-cert="" is passed
examples/secure-boot/build-current-images.sh
* using ceil to round up in disk usage analysis perl script
* example of calling perl script cleaned up a little
examples/secure-boot/install_gpu_driver.sh
examples/secure-boot/dask.sh
examples/secure-boot/rapids.sh
* matched exit_handler implementations between the three of the above
* collect disk usage information
* disk sizes are collected in 1k pages instead of human-readable
* mounting ram disk at /tmp to reduce disk usage further
examples/secure-boot/pre-init.sh
* removed metadata from arguments, since it will be injected in the
  generated script
* passing creating-image metadata to zero disk before and after run
* updated disk sizes
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