Page MenuHomeFreeBSD

freebsd-update: do not restart jailed sshd
ClosedPublic

Authored by emaste on Sep 16 2023, 8:54 PM.
Tags
None
Referenced Files
F150446319: D41890.diff
Wed, Apr 1, 6:52 AM
F150405868: D41890.diff
Tue, Mar 31, 10:58 PM
Unknown Object (File)
Wed, Mar 25, 8:55 AM
Unknown Object (File)
Wed, Mar 18, 2:55 PM
Unknown Object (File)
Tue, Mar 17, 8:57 PM
Unknown Object (File)
Tue, Mar 3, 12:25 AM
Unknown Object (File)
Feb 16 2026, 6:36 PM
Unknown Object (File)
Dec 15 2025, 6:09 AM
Subscribers

Details

Summary

In 6cd1bc531609 for PR 263489 I changed freebsd-update to restart sshd after upgrade, to avoid an upgrade-related incompatibility that made it impossible to login.

This is intended to avoid losing access to remote hosts, and ought not apply to upgrading a jail.

PR: 263489, 272282

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste created this revision.

Test $BASEDIR instead of $JAIL; we should not do the sshd restart dance for -b either.

I see inconsistent use of quotes around variable dereference (sometimes ${BASEDIR, sometimes "${BASEDIR}"). Other than that LGTM.

I'll test it on some jails I have left to update.

Tested, working, LGTM.

This revision is now accepted and ready to land.Sep 17 2023, 10:26 AM

I was going to complain from the description that I could only have access to the jail and be updating from within, but I reckon BASEDIR would be / in that case

I was going to complain from the description that I could only have access to the jail and be updating from within

Will update the commit message to make this clear