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

make gpt flag 59 (growfs) actually work on the root fs #30030

Merged
merged 4 commits into from
Nov 15, 2023

Conversation

poettering
Copy link
Member

@poettering poettering commented Nov 15, 2023

Fixes: #29791
Fixes: #28133

@github-actions github-actions bot added the please-review PR is ready for (re-)review by a maintainer label Nov 15, 2023
Copy link

An -rc1 tag has been created and a release is being prepared, so please note that PRs introducing new features and APIs will be held back until the new version has been released.

@YHNdnzj YHNdnzj added good-to-merge/with-minor-suggestions and removed please-review PR is ready for (re-)review by a maintainer labels Nov 15, 2023
Copy link
Member

@bluca bluca left a comment

Choose a reason for hiding this comment

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

works for me

This call is never called in the initrd, hence we can drop the extra
check, as it is redundant. Let's keep it as an assert() though, as a
form of code-enforced documentation.
We are not invoked in the initrd, and that deserves a comment.
…s_service()

I cannot see a reason why we should ignore this error, so let's not. We
use RET_GATHER() on the returns anyway, i.e. collect errors but
continue, so it makes sense to collect this one too.
@poettering poettering added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed and removed good-to-merge/with-minor-suggestions labels Nov 15, 2023
Copy link

An -rc1 tag has been created and a release is being prepared, so please note that PRs introducing new features and APIs will be held back until the new version has been released.

@bluca bluca merged commit 03745d6 into systemd:main Nov 15, 2023
42 of 44 checks passed
@github-actions github-actions bot removed the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Nov 15, 2023
@septatrix
Copy link
Contributor

Is it realistic to have this backported to v253/v254?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants