Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

install/baseline: use sfdisk, not lsblk #698

Merged
merged 2 commits into from
Jul 17, 2024

Conversation

cgwalters
Copy link
Collaborator

I think this may also help with the install flake we've been
seeing where we somehow get the partitions mixed up.

  • Use sfdisk in the baseline install because it's what
    we're switching to using anyways later on
  • Add some verification at least for the root and EFI partitions
    that we read back the expected type

Copy link
Contributor

@jeckersb jeckersb left a comment

Choose a reason for hiding this comment

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

LGTM, just a few nits that clippy pointed out

lib/src/blockdev.rs Outdated Show resolved Hide resolved
lib/src/install/baseline.rs Outdated Show resolved Hide resolved
lib/src/install/baseline.rs Outdated Show resolved Hide resolved
Signed-off-by: Colin Walters <walters@verbum.org>
I think this may also help with the install flake we've been
seeing where we somehow get the partitions mixed up.

- Use `sfdisk` in the baseline install because it's what
  we're switching to using anyways later on
- Add some verification at least for the root and EFI partitions
  that we read back the expected type

Signed-off-by: Colin Walters <walters@verbum.org>
@jeckersb
Copy link
Contributor

jeckersb commented Jul 17, 2024

Hm not sure how i pushed overtop of you with --force-with-lease 😢

Nevermind it was me, it just shows up weird in the github UI when I push to your branch

@jeckersb jeckersb enabled auto-merge July 17, 2024 18:41
@jeckersb jeckersb merged commit d91c00a into containers:main Jul 17, 2024
10 of 28 checks passed
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.

2 participants