The support for using encrypted file systems with advanced disk
configuration needs a lot more work.
Tracking new GitHub Issue in #512
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
The GUI need to be updated to ensure that the user defined state
was updated so that we store the user's answer in the (temp) YAML.
Make the GUI be consistent with the TUI and with the other
two selection labels for the Disk configuration.
Change advanced to warning
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Use new size checking for advanced partitioning.
Fix logic bug in checking for validation errors before
checking for encryption passphrase.
Code review fixes.
Make advanced description consistent with GUI
New min size and adv labels
Change advanced to warning
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
New label for labeled
New translation for passphrase encryption
Based on user feedback switched the tense of installation steps.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Since we have code which assumes we want autoUpdate to be true
unless it is added to the YAML file, we need to be sure it is
write to the YAML when false.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
We've added new functionality to storage which is only
exercised on media when running as root, so dropping
the basic coverage value for storage.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
The UUID is not set until after the file system is created.
We need to update the block device in memory with the new
UUID to properly create a fstab with UUID instead of device.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Write Partition table is setting the partition type
using the partition number, ensure it is set first.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Since we depend upon systemd to mount well-known partitions,
we need to ensure that the GUID is set correctly in the gpt
table for each file system. This is true even if we are
re-using an existing partition.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Check the exit status and collect the results. If we had any failures,
then save a copy of the log file and do not remove the temp files.
Correct how we are enabling root access to the DISPLAY
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
We should only store the telemetry answer when it was
user selected. This fix was needed due to the new advanced
configuration which save the temporary state.
Need to update the ister model for the new change.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
No easy way to test all of the error conditions of
Marshaling and Unmarshaling, so dropping the coverage by 1%
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Add translation for advanced partitions as well as new strings
related to the media changes which will be done in confirmation.
Add new tests.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
- model: Store precheck status in model
- Refactor sudo user into reusable function
- Enable the partitioning tool button
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Show the user the changes that will happen to media before
confirming the start of the installation.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Change InstallSelected to be a map with the key being
the name of the TargetMedias from the model to support
installing across multiple disks.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
The disk device name is show in the configured information, but
was missing in the chooser list. User had request this to be able
to differentiate between two drives of the same model.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
The graphical tool won't usually show the echo statements
in a terminal, so the sleep are an undesired delay.
Added code to enable proper Xauth in the shell script for
running the graphical code as super user in the non-admin
user's desktop.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Change InstallSelected to be a map with the key being
the name of the TargetMedias from the model to support
installing across multiple disks.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Change InstallSelected to be a map with the key being
the name of the TargetMedias from the model to support
installing across multiple disks.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Remove the code which was attempting to do the work of many
existing, more reliable disk partitioning tools. The code was
getting overly complex to handle the very high number of options
and corner cases.
The installer will rely on external tools which do a better job.
This will parse the media for advanced partition labels.
Show the user the details of what will change and be used
for the installation.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
If the telemetry value is added to the YAML will set the default
as if the user had selected it in the UI.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
This is needed to ensure proper locking while running the external
tool for disk partitioning for Advanced disk configuration during
interactive installations.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Verify advanced labels to ensure all required are present and
no conflicts.
Add checker for the use of encryption for advanced labels.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Detect the details of the changes to the media to be presented
to the user during the final confirmation step.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Add partition labels to the lsblk parsed data to enable the
advanced media configuration option. An external tool will be
used to set the partition labels to identify media for use.
Check if a file system is present on each label partition, and if
non is found, use the default file system and flag for formatting.
Signed-off-by: Mark D Horn <mark.d.horn@intel.com>
Mark D Horn