Skip to content

Conversation

@rtanaka
Copy link
Contributor

@rtanaka rtanaka commented Oct 31, 2018

you should now be able to run the configure.sh repeatedly to update scripts provided the environment variables are set properly. setup does more of the one time heavy lifting... i.e. create the partitions, create the usb drive and edit fstab.

i tested this twice from a clean sd card build .. once before and after the merge from master.

rtgoodwin and others added 30 commits October 26, 2018 06:58
I hate this whole unbound variable protection mode, but it sure does catch bugs. :)
More logging & partition start point
…n argument instead of a environment variable
- using new ip lookup function
take out rclone specific logic out of here and move into rclone_archive
…/ installation. this will allow a user to run configuration multiple times.
@rtgoodwin
Copy link
Contributor

Is configure designed to run in headless mode? Or for manual setup only?

@cimryan cimryan merged commit 1012c63 into cimryan:u/rtanaka/organizing Oct 31, 2018
@rtanaka
Copy link
Contributor Author

rtanaka commented Oct 31, 2018

hey there. i split this up because as you know, setting up the partitions is more of a one time thing and failing to mount your archive currently breaks the whole process. the split allows you to run configure multiple times to get (and eventually change/update) your archive server without a full reinstall.

from a functional perspective if you add configure immediately after setup-teslausb it will result in the same outcome. i haven't added your headless specific logging items into the new configure script. along those lines i believe we should split out some common functions (your headless logging, led blinking, get_script, check_variable) into a separate lib so both scripts can leverage them.

@rtgoodwin - would you mind making the rc.local change for headless? also let's move this discussion to #81 since this is merged.

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.

3 participants