seattletestbed / installer-packaging Goto Github PK

seattleinstaller.py creates a file installInfo.old (or .new) in the servicevessel directory where it stores results of the system benchmark, donated percentage, and amount of resources that will be used by Seattle. However, on installs running a V2 nodemanager, files in vessels can only be accessed if their names have lower-case characters (and a few other things) exclusively. As a result, you cannot retrieve or otherwise manipulate installInfo via seash (see below).

This issue is RepyV2-specific. RepyV1 files may contain upper-case characters. We need to be able to see the information contained in the file for debugging purposes on any version of Seattle, so let's make the base filename installinfo (all lower-case) instead.

This requires changes in anything that depends on this exact file name:

• Seattle Installer, https://github.com/SeattleTestbed/installer-packaging/blob/master/all-platforms/seattleinstaller.py#L2146
• Seattle Uninstaller, https://github.com/SeattleTestbed/installer-packaging/blob/master/all-platforms/seattleuninstaller.py#L604
• Android app (1), https://github.com/SensibilityTestbed/sensibility-testbed/blob/e2907df340ec08f6d6c12b241b65bf7de28679b0/SensibilityTestbed/src/com/sensibility_testbed/InstallerService.java#L125-L133
• Android app (2), https://github.com/SensibilityTestbed/sensibility-testbed/blob/359dc6b17ec4d0d8734d86b0f71ce033a0d05412/SensibilityTestbed/src/com/sensibility_testbed/ScriptActivity.java#L311

seash log:

servicevessel@%1 !> cat installInfo.old
An error occurred: Node Manager error 'Filename has disallowed character 'I''
Affected vessels: ee7eb6c1bb4389a62812a9c47b76a0c5f47b0b3e:1224:v9.
servicevessel@%1 !> download installInfo.old
An error occurred: Node Manager error 'Filename has disallowed character 'I''
Affected vessels: ee7eb6c1bb4389a62812a9c47b76a0c5f47b0b3e:1224:v9.
servicevessel@%1 !> delete installInfo.old
An error occurred: Node Manager error 'Filename has disallowed character 'I''
Affected vessels: ee7eb6c1bb4389a62812a9c47b76a0c5f47b0b3e:1224:v9.

The current OpenWrt installer (in the form of @XuefengHuang's .ipk) will install Seattle as the root user. We should provide an option to use a non-privileged account if the user has (or is willing to install) the required tools such as useradd. See the OpenWrt docs for details: http://wiki.openwrt.org/doc/howto/secure.access

rebuild_base_installers.py currently populates and packages (tars, gzips or zips) base installer directories for the Custom Installer Builder. However, it does other things that might be out of scope as they are tied closely to the layout and config of the CIB: It copies tarballs to the deployment dir, backups previous base installers, chowns files, etc.

All of these things are important for deploying a set of new base installers, none of them are actual packaging tasks however. I think we should remove the unrelated parts.

(However, it will probably make sense to still include checking for the softwareupdater URL and key etc., just so that CIB operators don't shoot themselves in the foot).