Skip to content

Commit 7acadc0

Browse files
committed
fix: do not stop udevd before unmounting volumes
As udevd is required by cryptsetup which will timeout if udevd is not working, do not stop it in StopServicesEphemeral, but let StopAllServices handle udev shutdown after cryptsetup close is called Ref: https://bbs.archlinux.org/viewtopic.php?id=162415 Signed-off-by: Dmitry Sharshakov <dmitry.sharshakov@siderolabs.com>
1 parent 6a08105 commit 7acadc0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

internal/app/machined/pkg/runtime/v1alpha1/v1alpha1_sequencer_tasks.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -839,7 +839,7 @@ func StartAllServices(runtime.Sequence, any) (runtime.TaskExecutionFunc, string)
839839
func StopServicesEphemeral(runtime.Sequence, any) (runtime.TaskExecutionFunc, string) {
840840
return func(ctx context.Context, logger *log.Logger, r runtime.Runtime) (err error) {
841841
// stopping 'cri' service stops everything which depends on it (kubelet, etcd, ...)
842-
return system.Services(nil).StopWithRevDepenencies(ctx, "cri", "udevd", "trustd")
842+
return system.Services(nil).StopWithRevDepenencies(ctx, "cri", "trustd")
843843
}, "stopServicesForUpgrade"
844844
}
845845

0 commit comments

Comments
 (0)