Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-12 Thread Garrett Cooper
On Feb 9, 2015, at 18:51, James Gritton ja...@freebsd.org wrote: On 2015-02-06 22:23, Garrett Cooper wrote: On Feb 6, 2015, at 18:38, James Gritton ja...@freebsd.org wrote: On 2015-02-06 19:23, Garrett Cooper wrote: I think you broke the Jenkins tests runs, and potentially jail support in

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-12 Thread Garrett Cooper
On Feb 12, 2015, at 13:12, Garrett Cooper yaneurab...@gmail.com wrote: On Feb 9, 2015, at 18:51, James Gritton ja...@freebsd.org wrote: On 2015-02-06 22:23, Garrett Cooper wrote: On Feb 6, 2015, at 18:38, James Gritton ja...@freebsd.org wrote: On 2015-02-06 19:23, Garrett Cooper wrote: I

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-09 Thread NGie Cooper
On Sat, Feb 7, 2015 at 12:10 AM, Konstantin Belousov kostik...@gmail.com wrote: On Fri, Feb 06, 2015 at 09:23:47PM -0800, Garrett Cooper wrote: pgrep uses /proc to determine whether or not a process is running. If it?s not properly mounted or the jail isn?t started properly, that could cause

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-09 Thread James Gritton
On 2015-02-06 22:23, Garrett Cooper wrote: On Feb 6, 2015, at 18:38, James Gritton ja...@freebsd.org wrote: On 2015-02-06 19:23, Garrett Cooper wrote: I think you broke the Jenkins tests runs, and potentially jail support in some edgecases:

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-07 Thread Konstantin Belousov
On Fri, Feb 06, 2015 at 09:23:47PM -0800, Garrett Cooper wrote: pgrep uses /proc to determine whether or not a process is running. If it?s not properly mounted or the jail isn?t started properly, that could cause the issues seen here. I know because I?ve tried running these tests before in an

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread James Gritton
On 2015-02-06 22:31, Garrett Cooper wrote: On Feb 6, 2015, at 21:27, James Gritton ja...@freebsd.org wrote: On 2015-02-06 22:18, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Modified: head/usr.sbin/jail/command.c

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread Garrett Cooper
On Feb 6, 2015, at 21:31, Garrett Cooper yaneurab...@gmail.com wrote: On Feb 6, 2015, at 21:27, James Gritton ja...@freebsd.org wrote: On 2015-02-06 22:18, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Modified: head/usr.sbin/jail/command.c

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread Garrett Cooper
On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Author: jamie Date: Fri Feb 6 17:54:53 2015 New Revision: 278323 URL: https://svnweb.freebsd.org/changeset/base/278323 Log: Add mount.procfs jail parameter, so procfs can be mounted when a prison's root is in its fstab.

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread James Gritton
On 2015-02-06 22:23, Garrett Cooper wrote: On Feb 6, 2015, at 18:38, James Gritton ja...@freebsd.org wrote: On 2015-02-06 19:23, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Author: jamie Date: Fri Feb 6 17:54:53 2015 New Revision: 278323 URL:

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread Garrett Cooper
On Feb 6, 2015, at 21:27, James Gritton ja...@freebsd.org wrote: On 2015-02-06 22:18, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Modified: head/usr.sbin/jail/command.c ==

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread Garrett Cooper
On Feb 6, 2015, at 18:38, James Gritton ja...@freebsd.org wrote: On 2015-02-06 19:23, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Author: jamie Date: Fri Feb 6 17:54:53 2015 New Revision: 278323 URL: https://svnweb.freebsd.org/changeset/base/278323

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread Garrett Cooper
On Feb 6, 2015, at 21:23, Garrett Cooper yaneurab...@gmail.com wrote: On Feb 6, 2015, at 18:38, James Gritton ja...@freebsd.org wrote: On 2015-02-06 19:23, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Author: jamie Date: Fri Feb 6 17:54:53 2015

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread James Gritton
On 2015-02-06 22:18, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Modified: head/usr.sbin/jail/command.c == --- head/usr.sbin/jail/command.cFri Feb 6 17:43:13 2015

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread Garrett Cooper
On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Author: jamie Date: Fri Feb 6 17:54:53 2015 New Revision: 278323 URL: https://svnweb.freebsd.org/changeset/base/278323 Log: Add mount.procfs jail parameter, so procfs can be mounted when a prison's root is in its fstab.

Re: svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread James Gritton
On 2015-02-06 19:23, Garrett Cooper wrote: On Feb 6, 2015, at 9:54, Jamie Gritton ja...@freebsd.org wrote: Author: jamie Date: Fri Feb 6 17:54:53 2015 New Revision: 278323 URL: https://svnweb.freebsd.org/changeset/base/278323 Log: Add mount.procfs jail parameter, so procfs can be mounted

svn commit: r278323 - in head: etc/rc.d usr.sbin/jail

2015-02-06 Thread Jamie Gritton
Author: jamie Date: Fri Feb 6 17:54:53 2015 New Revision: 278323 URL: https://svnweb.freebsd.org/changeset/base/278323 Log: Add mount.procfs jail parameter, so procfs can be mounted when a prison's root is in its fstab. Also fix a typo while I'm at it. PR: 197237 197066