Skip to content
This repository has been archived by the owner on Sep 18, 2020. It is now read-only.

Additional Robustness #792

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Additional Robustness #792

wants to merge 1 commit into from

Conversation

kcwitt
Copy link

@kcwitt kcwitt commented Feb 24, 2018

These changes are focused on restarting without having to re-download files in the event of an error. In some cases, the error may be because the script is running on a host without standard utils such as sha512hash (which under the original script was only detected after downloading the complete image). In other cases, it may be due to an unstable network connection. The existing script requires both virtualbox and a good network connection, so allowing a cache directory allows downloading on a machine with a good network connection and then manually copying to the virtualbox machine.

The intention of these changes is still to maintain 100% of the original crypto-validation (which is why it should be safe to used cached files no matter their origin).

  1. removed trap that deletes temp dir (only delete on success)
  2. added "-c" option to cache files (in case no access to internet or
    retry)
  3. elaborated check for missing utilities (for operating systems such
    as mac an windows that don't include wget :-(
  4. added option to refresh key (but won't die if refresh fails; useful
    for fully disconnected install with old cached files)
  5. changed hash validation to openssl instead of sha1sum and sha512sum
    since some OS such as mac and windows are missing these utils :-(
  6. added some extra error handling to exit on error (specifically to
    prevent deleting the working dir)

2. added "-c" option to cache files (in case no access to internet or
retry)
3. elaborated check for missing utilities (for operating systems such
as mac an windows that don't include wget :-(
4. added option to refresh key (but won't die if refresh fails; useful
for fully disconnected install with old cached files)
5. changed hash validation to openssl instead of sha1sum and sha512sum
since some OS such as mac and windows are missing these utils :-(
6. added some extra error handling to dump out on error (specifically to
prevent deleting the working dir)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant