Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Post-pone guided storage choice #1111

Merged
merged 1 commit into from Sep 8, 2022
Merged

Conversation

jpnurmi
Copy link
Contributor

@jpnurmi jpnurmi commented Sep 6, 2022

A guided storage choice consists of multiple attributes:

  • which target to use
  • whether to use LVM
  • whether to use encryption
  • security key

These inputs are collected from multiple pages:

  • Installation type
  • Select guided storage
  • Install X alongside Y
  • Choose security key

The problem is that Subiquity only allows setting guided storage once.
That is, we cannot "update" the existing guided storage selection as we
go but we need to wait until all attributes have been gathered and then
apply it all in one go. The best place for this is when arriving at the
Write changes to disk page where the storage changes are summarized.

These changes were extracted from #951.

A guided storage choice consists of multiple attributes:

- which disk/partition to use
- whether to use lvm
- whether to use encryption
- security key

These inputs are collected from multiple pages:

- installation type
- select guided storage
- install x alongside y
- choose security key

The problem is that Subiquity only allows setting guided storage once.
That is, we cannot "update" the existing guided storage selection as we
go but we need to wait until all attributes have been gathered and then
apply it all in one go. The best place for this is when arriving at the
"Write changes to disk" page where the storage changes are summarized.
Copy link
Contributor

@CarlosNihelton CarlosNihelton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jpnurmi jpnurmi merged commit 9d91ad2 into canonical:main Sep 8, 2022
@jpnurmi jpnurmi deleted the guided-choice branch September 8, 2022 11:44
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.

None yet

2 participants