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

(RHEL-6210) cryptsetup: try to detach the device multiple times before giving up #428

Closed
wants to merge 1 commit into from

Conversation

lnykryn
Copy link
Member

@lnykryn lnykryn commented Feb 14, 2024

Terrible hack for RHEL8. We are missing the [email protected], so we can't correctly express some dependencies. That means when systemd-cryptsetup detach can fail, because for example a filesystem is still mounted on top of the device. This can cause shutdown to hang forever. See https://issues.redhat.com/browse/RHEL-6210 So as a workaround, let's try to detach the device multiple times before giving up. */

RHEL-only
Resolves: RHEL-6210

@github-actions github-actions bot changed the title cryptsetup: try to detach the device multiple times before giving up (RHEL-6210) cryptsetup: try to detach the device multiple times before giving up Feb 14, 2024
@github-actions github-actions bot added tracker/unapproved Formerly needs-acks pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Feb 14, 2024
Copy link

github-actions bot commented Feb 14, 2024

Commit validation

Tracker - RHEL-6210

The following commits meet all requirements

commit upstream
cdf70e6 - cryptsetup: try to detach the device multiple times before giving up RHEL-only

Tracker validation

Failed

🔴 Tracker RHEL-6210 has not been approved

Success

🟢 Tracker RHEL-6210 has set desired product: rhel-8.8.0
🟢 Tracker RHEL-6210 has set desired component: systemd


Pull Request validation

Failed

🔴 Failed or pending statuses - CentOS CI (CentOS Stream 8)[pending]
🔴 Review - Missing review from a member.

@lnykryn lnykryn marked this pull request as draft February 14, 2024 14:29
@lnykryn
Copy link
Member Author

lnykryn commented Feb 14, 2024

Marking as a draft so it is not automatically merge. Maybe we will be able to backport [email protected]

Terrible hack for RHEL8. We are missing the [email protected], so we
can't correctly express some dependencies. That means when
systemd-cryptsetup detach can fail, because for example a filesystem is
still mounted on top of the device. This can cause shutdown to hang
forever. See https://issues.redhat.com/browse/RHEL-6210
So as a workaround, let's try to detach the device multiple times
before giving up.

Proper fix is
systemd/systemd@44b0d1f

RHEL-only
Resolves: RHEL-6210
@lnykryn
Copy link
Member Author

lnykryn commented Feb 14, 2024

Hmm with encrypted root, there is now 10 seconds delay in shutdown every time 😿

@github-actions github-actions bot removed the pr/needs-ci Formerly needs-ci label Feb 14, 2024
@jamacku jamacku changed the base branch from main to rhel-8.10.0 March 11, 2024 15:10
@github-actions github-actions bot added rhel-8.10.0 pr/needs-ci Formerly needs-ci labels Mar 11, 2024
@jamacku
Copy link
Member

jamacku commented Mar 11, 2024

Replaced by #429

@jamacku jamacku closed this Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dont-merge pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review rhel-8.10.0 tracker/unapproved Formerly needs-acks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants