Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Jan 29, 2020

Fixes "You don't have any Linux partitions." in the rescue mode.

It took me an hour to figure why did the rescue mode was not able to find
a Linux partition.

If we do not want to mount pre-existing partition nor detect the OS in
the installer, then we should only do that in the installer.

Then, it would be much more useful if the function isn't just getting
chopped off like it was but instead a message explaining why it was
chopped off added. Can save hours for people trying to figure why their
rescue mode isn't finding any Linux partition.


Related issue QubesOS/qubes-issues#2835

@ghost ghost requested a review from marmarek January 29, 2020 16:42
@ghost
Copy link
Author

ghost commented Jan 29, 2020

Hum, I've just skimmed through the issue and can see that this patch prevents against the data loss situations. Probably it's better not to merge this, but rather come with a better fix?

@ghost
Copy link
Author

ghost commented Jan 29, 2020

Though, the potential "data loss" issue seem to be was fixed here https://github.com/storaged-project/blivet/pull/526/files#diff-cd99c6e4dd15e4148bb0f4d1a214f303R119 ?

@marmarek
Copy link
Member

It not only do fsck (that's another place and indeed is better now), but also mounts every single partition it finds on the disk, and analyze various files found there (/etc/os-release, /etc/fstab etc). Something that can easily end up badly. Imagine you're reinstalling your system over previous compromised one - the amount of parsing of pre-existing disk content here makes it quite likely that it would be possible to plant files that would re-compromise qubes installer...

@ghost
Copy link
Author

ghost commented Jan 29, 2020

I see the point.
We should still make it working in the rescue mode though.

@ghost ghost closed this Jan 29, 2020
@marmarek
Copy link
Member

We should still make it working in the rescue mode though.

Yes.

@ghost
Copy link
Author

ghost commented Jan 29, 2020

We should still make it working in the rescue mode though.

Yes.

I will open a separate issue so not to forget.

Upd: QubesOS/qubes-issues#5609

Fixes "You don't have any Linux partitions." in the rescue mode.

It took me an hour to figure why did the rescue mode was not able to find
a Linux partition.

If we do not want to mount pre-existing partition nor detect the OS in
the installer, then we should only do that in the installer.

Then, it would be much more useful if the function isn't just getting
chopped off like it was but instead a message explaining why it was
chopped off added. Can save hours for people trying to figure why their
rescue mode isn't finding any Linux partition.
This pull request was closed.
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.

1 participant