Skip to content

homed: allow per-user additional LUKS mount options #21331

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.

Sign up for GitHub

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

Jump to bottom
Merged
poettering merged 3 commits into from
Nov 13, 2021
Merged

homed: allow per-user additional LUKS mount options #21331

poettering merged 3 commits into from
Nov 13, 2021

Conversation

@poettering
Copy link
Member

@poettering poettering commented Nov 12, 2021

Fixes: #15120

@poettering poettering added the homed homed, homectl, pam_homed label Nov 12, 2021
@poettering poettering mentioned this pull request Nov 12, 2021
Merged
bluca
bluca approved these changes Nov 12, 2021
View reviewed changes
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.

just one typo in the docs

@poettering poettering force-pushed the luks-extra-mount-options branch from 318a60f to 5dd57a0 Compare November 12, 2021 21:25
@poettering
Copy link
Member Author

poettering commented Nov 12, 2021

Force pushed new version with the typo fixed. Upgrading green label

@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 12, 2021
yuwata
yuwata approved these changes Nov 13, 2021
View reviewed changes
Copy link
Member

@yuwata yuwata left a comment

Choose a reason for hiding this comment

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

LGTM.

@poettering poettering merged commit b1beb00 into systemd:main Nov 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bluca bluca bluca approved these changes

@yuwata yuwata yuwata approved these changes

Assignees

No one assigned

Labels

documentation 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 homed homed, homectl, pam_homed

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

homectl support for additional or arbitrary mount options

3 participants

@poettering @bluca @yuwata