AcceptedPublic
Actions

Authored by lwhsu on Fri, Jan 2, 8:37 AM.

Details

Reviewers

antranigv_freebsd.am

Group Reviewers

Jails

Summary

jail: install a default /etc/jail.conf which honors /etc/jail.conf.d/

If just putting all jail settings in /etc/jail.conf.d, /etc/rc.d/jail gives:

Starting jails:jail: /etc/jail.conf: No such file or directory

Sponsored by:	The FreeBSD Foundation

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 69567
Build 66450: arc lint + arc unit

Event Timeline

lwhsu created this revision.Fri, Jan 2, 8:37 AM

Herald added a subscriber: imp. · View Herald TranscriptFri, Jan 2, 8:37 AM

lwhsu requested review of this revision.Fri, Jan 2, 8:37 AM

Harbormaster completed remote builds in B69567: Diff 168870.Fri, Jan 2, 8:37 AM

lwhsu added a reviewer: Jails.Fri, Jan 2, 8:40 AM

as a matter of fact, this is a good motivation to update the jail.conf examples as well.

This revision is now accepted and ready to land.Fri, Jan 2, 8:46 AM

In D54450#1244273, @antranigv_freebsd.am wrote:

as a matter of fact, this is a good motivation to update the jail.conf examples as well.

Any suggestion? It seems this already in the EXAMPLES section?

The EXAMPLES section in the manual pages are actually pretty good, but I was thinking about the examples in the examples directory: /usr/share/examples/jails || https://cgit.freebsd.org/src/tree/share/examples/jails

Since 14.0, multiple features have been added which are under demonstrated. We should also update the README and remove the VIMAGE file (has been merged into GENERIC since 11.0).

Thank you for this little change, it's a good motivation to move other low hanging fruits as well.