Signed-off-by: Frédéric Danis <frederic.danis@collabora.com>

Docker should work out-of-box, and previously used images should be
available through the persistent disk.

Signed-off-by: Frédéric Danis <frederic.danis@collabora.com>

Back in the long past we had to have separate partitions due to how the
application framework worked. This is no more requested and the system now
use a single partition.

Signed-off-by: Frédéric Danis <frederic.danis@collabora.com>

Signed-off-by: Frédéric Danis <frederic.danis@collabora.com>

psdk uses `sfdisk` to partition the future persistent disk.

Signed-off-by: Frédéric Danis <frederic.danis@collabora.com>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

This patch removes code that was not used, simplify the Progressbar
class, and add comments to document corner cases.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

psdk is now interactive and does not require the user to discover disk
nodes before using the persistent disk. This patch updates the README.md
file to reflect the new usage.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Signed-off-by: Ritesh Raj Sarraf <ritesh.sarraf@collabora.com>

Signed-off-by: Ritesh Raj Sarraf <ritesh.sarraf@collabora.com>

Signed-off-by: Ritesh Raj Sarraf <ritesh.sarraf@collabora.com>

Peter Senna Tschudin authored 5 years ago and Emanuele Aina committed 5 years ago

This patch:
 - changes the package name to 'psdk'
 - adds a systemd service to call psdk -s on startup
 - fixes issues with debian files

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Peter Senna Tschudin authored 5 years ago and Emanuele Aina committed 5 years ago

Move the icon from the desktop to the System menu with other file system
related tools. Also change the icon to make it more visible with the
gray menu back ground.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Peter Senna Tschudin authored 5 years ago and Emanuele Aina committed 5 years ago

This patch adds ETC_FILES_AUTO that contains a list of configuration
files that are moved to persistent storage during disk initialization.

It also adds a systemd service file to scan the persistent storage for
configuration files during boot. Minor changes were necessary so that
psdk exits cleanly when no persistent disks are found and when no
changes are required.

On first boot after configuring a persistent disk, systemd will show:

 user@apertis:~$ sudo systemctl status psdk-conf-scan.service
 ● psdk-conf-scan.service - Scan for configuration files on persistent storage
    Loaded: loaded (/etc/systemd/system/psdk-conf-scan.service; enabled; vendor preset: enabled)
    Active: active (exited) since Mon 2019-06-17 09:06:39 UTC; 19s ago
   Process: 473 ExecStart=/usr/bin/psdk -s (code=exited, status=0/SUCCESS)
  Main PID: 473 (code=exited, status=0/SUCCESS)

 Jun 17 09:06:39 apertis sudo[538]: pam_unix(sudo:session): session opened for user root by (uid=0)
 Jun 17 09:06:39 apertis sudo[538]: pam_unix(sudo:session): session closed for user root
 Jun 17 09:06:39 apertis sudo[547]:     root : TTY=unknown ; PWD=/ ; USER=root ; COMMAND=/bin/mv -f /etc/cntlm.conf /etc/cntlm
 Jun 17 09:06:39 apertis sudo[547]: pam_unix(sudo:session): session opened for user root by (uid=0)
 Jun 17 09:06:39 apertis sudo[547]: pam_unix(sudo:session): session closed for user root
 Jun 17 09:06:39 apertis sudo[551]:     root : TTY=unknown ; PWD=/ ; USER=root ; COMMAND=/bin/ln -sf /home/etc/cntlm.conf /etc
 Jun 17 09:06:39 apertis sudo[551]: pam_unix(sudo:session): session opened for user root by (uid=0)
 Jun 17 09:06:39 apertis sudo[551]: pam_unix(sudo:session): session closed for user root
 Jun 17 09:06:39 apertis psdk[473]: Using configuration file found on persistent storage. From now on, changes made to /etc/cn
 Jun 17 09:06:39 apertis systemd[1]: Started Scan for configuration files on persistent storage.

And then on following boots:

 user@apertis:~$ sudo systemctl status psdk-conf-scan.service
 ● psdk-conf-scan.service - Scan for configuration files on persistent storage
    Loaded: loaded (/etc/systemd/system/psdk-conf-scan.service; enabled; vendor preset: enabled)
    Active: active (exited) since Mon 2019-06-17 09:07:59 UTC; 5s ago
   Process: 471 ExecStart=/usr/bin/psdk -s (code=exited, status=0/SUCCESS)
  Main PID: 471 (code=exited, status=0/SUCCESS)

 Jun 17 09:07:59 apertis systemd[1]: Starting Scan for configuration files on persistent storage...
 Jun 17 09:07:59 apertis sudo[545]:     root : TTY=unknown ; PWD=/ ; USER=root ; COMMAND=/usr/bin/find /home/etc -type f
 Jun 17 09:07:59 apertis sudo[545]: pam_unix(sudo:session): session opened for user root by (uid=0)
 Jun 17 09:07:59 apertis sudo[545]: pam_unix(sudo:session): session closed for user root
 Jun 17 09:07:59 apertis psdk[471]: /etc/cntlm.conf is already on persistent storage. No action needed.
 Jun 17 09:07:59 apertis systemd[1]: Started Scan for configuration files on persistent storage.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Peter Senna Tschudin authored 5 years ago and Emanuele Aina committed 5 years ago

This patch adds a new zenity based menu and a zenity based progress bar.
The progress bar was required as copying up to 20GB of data can be very
slow depending on the hardware configuration.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Peter Senna Tschudin authored 5 years ago and Emanuele Aina committed 5 years ago

The user no longer needs to point to disk device nodes when initializing
a new disk or when configuring the SDK to use an existing persistent
disk. When using the command line, the only option that accepts a path
is --etc to add a configuration file to persistent storage.

A class Disks was added to handle disk detection based on the output of
lsblk. This made it possible to replace various other methods and
functions that were overlaping in functionality.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Using a label on the partition instead of on the file system has the
advantage of decoupling the label information from the file system which
will persist even to formatting the file system for example. Using
PARTLABEL is in general a more robust approach.

This patch also replaces `e2label` by `lsblk` as the former does not
require root permissions and also produces json output making it more
reliable to parse.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Using findmnt simplifies the code to find out if persistent disk is in
use.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Add a new line before the messages about reboot tom improve readability.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Add instructions on how to use psdk interactive mode to the REAME.md
file.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Calling psdk without arguments will now trigger the interactive mode.

There are a few pre-conditions:
 - empty disk available: asks if the user wants to initialize the disk
   and configure the SDK to use it.
 - persistent disk available, but not in use: asks if the user wants to
   configure the SDK to use the persistent disk.
 - persistent disk in use: no actions allowed

This patch also includes the file data/psdk.desktop which creates a
desktop icon when copied to ~/Desktop.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Functionality of Disk.isvalid() and Disk.isempty() are useful outside
the Disk class. Make them functions.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Add capture_output=True to subprocess.run() so that stderr messages are
not shown on screen.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

This patch creates a function persistent_disk_in_use() moving it's
content from main().

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Use ext4 instead of btrfs for the persistent partition. This patch makes
the changes to psdk and README to reflect the new file system.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Fix typo SKD to SDK.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

An empty line is required before a block text formated as code.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

An empty line is required before a block text formated as code.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Mention which version of VirtualBox was used for creating the
instructions.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Remove ':' from the end of title lines.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

README is formatted using Markdown, and the .md extension is needed for
proper rendering on the web interface.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

The occrect directory is /opt and not /etc.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Add use instructions to the README file.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Previous message was confusing as if -s would overwrite the
configuration file found on the persistent storage. This patch improves
the message.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Change the license to MPL-2.0 to comply with Apertis licensing
guidelines.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

--scan will look for configuration files on the persistent storage and
will use them on the SDK. This patch makes psdk print a message on each
file successfully configured.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Add the debian directory and the basic set of files for creating a
Debian package.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

This patch remove debug messages, add informative messages, and improve
existing messages for a better usability.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

Improve the help text for --scan making it more verbose.

Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>