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

Allow reusing existing "empty" filesystems for / #5416

Conversation

vojtechtrefny
Copy link
Contributor

We currently require that filesystem for / must be newly formatted by Anaconda, with the new storage workflow the filesystems created in Cockpit doesn't need to be reformatted so the check needs to be adjusted to check if the filesystem selected for / is empty and force the reformatting only when it contains some preexisting data.

Depends on: storaged-project/blivet#1188

@pep8speaks
Copy link

pep8speaks commented Jan 22, 2024

Hello @vojtechtrefny! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-02-08 10:00:36 UTC

@github-actions github-actions bot added the f40 label Jan 22, 2024
@vojtechtrefny vojtechtrefny force-pushed the master_anaconda-root-reformat-check-change branch from 8d4843f to fd81dd4 Compare January 22, 2024 10:09
@vojtechtrefny vojtechtrefny added the blocked Don't merge this pull request! label Jan 22, 2024
Copy link
Contributor

@rvykydal rvykydal left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Copy link
Contributor

@M4rtinK M4rtinK left a comment

Choose a reason for hiding this comment

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

Looks good to me as well. :)

@vojtechtrefny vojtechtrefny force-pushed the master_anaconda-root-reformat-check-change branch from fd81dd4 to e5f937f Compare January 26, 2024 09:34
@KKoukiou
Copy link
Contributor

@vojtechtrefny this is blocked on merging till storaged-project/blivet#1188 gets released in rawhide I guess, can you confirm?

@vojtechtrefny vojtechtrefny removed the blocked Don't merge this pull request! label Jan 30, 2024
@vojtechtrefny
Copy link
Contributor Author

vojtechtrefny commented Jan 30, 2024

@vojtechtrefny this is blocked on merging till storaged-project/blivet#1188 gets released in rawhide I guess, can you confirm?

The blivet PR is now merged and built in the daily copr so it should no longer block this. (But we then need to coordinate releases of blivet and anaconda in rawhide to make sure we don't break Fedora.)

@KKoukiou
Copy link
Contributor

@vojtechtrefny this is blocked on merging till storaged-project/blivet#1188 gets released in rawhide I guess, can you confirm?

The blivet PR is now merged and built in the daily copr so it should no longer block this. (But we then need to coordinate releases of blivet and anaconda in rawhide to make sure we don't break Fedora.)

We plan a new release next Tuesday. Do you plan to release this in rawhide before that?

@vojtechtrefny
Copy link
Contributor Author

We plan a new release next Tuesday. Do you plan to release this in rawhide before that?

I actually released 3.9.0 today.

@KKoukiou
Copy link
Contributor

/kickstart-test --testtype smoke

Copy link
Contributor

@KKoukiou KKoukiou left a comment

Choose a reason for hiding this comment

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

It would be correct to introduce the specfile Requires: python-blivet >= 3.9 I think.

@vojtechtrefny
Copy link
Contributor Author

It would be correct to introduce the specfile Requires: python-blivet >= 3.9 I think.

Good point, I've added a new commit with the version bump.

@vojtechtrefny vojtechtrefny force-pushed the master_anaconda-root-reformat-check-change branch from f03bbae to e2a80e6 Compare February 1, 2024 09:22
@KKoukiou
Copy link
Contributor

KKoukiou commented Feb 1, 2024

/kickstart-test --testtype smoke

1 similar comment
@KKoukiou
Copy link
Contributor

KKoukiou commented Feb 8, 2024

/kickstart-test --testtype smoke

We currently require that filesystem for / must be newly formatted
by Anaconda, with the new storage workflow the filesystems created
in Cockpit doesn't need to be reformatted so the check needs to
be adjusted to check if the filesystem selected for / is empty and
force the reformatting only when it contains some preexisting data.
@KKoukiou KKoukiou force-pushed the master_anaconda-root-reformat-check-change branch from e2a80e6 to 6c8876c Compare February 8, 2024 10:00
Copy link
Contributor

@KKoukiou KKoukiou left a comment

Choose a reason for hiding this comment

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

Pushed a clean rebase - to see if tests get auto fixed.

@KKoukiou
Copy link
Contributor

KKoukiou commented Feb 8, 2024

/kickstart-test --testtype smoke

@KKoukiou KKoukiou merged commit 3cae128 into rhinstaller:master Feb 8, 2024
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

5 participants