Use xo_open_container instead of xo_open_instance
Increase the PROCSTAT_XO_VERSION
Use threadid as key

In D54990#1257180, @lwhsu wrote:

WWW: information?

In D54924#1256188, @js wrote:

Can you share before and after output?

Question: since this is a breaking change, should I increment the PROCSTAT_XO_VERSION integer?

That could do it! Thank you for all the help!

• sysutils/watchcat: use full git hash

• sysutils/watchcat: delete voc-* deps and use GH_TUPLE
• sysutils/watchcat: better pkg-descr
• add watchcat to sysutils/Makefile
• sysutils/watchcat: improve environment

Noted! Will diff again with the requested changes. Thank you!!!

Rebasing the files from 2 years ago might be too much to do, so I am abandoning this revision, and I shall do a new diff now that service.sh has much more nicer features which will indeed help -s as well.

• add watchcat to sysutils/Makefile

• sysutils/watchcat: delete voc-* deps and use GH_TUPLE
• sysutils/watchcat: better pkg-descr

Well I learned a lot doing this port. Submitting the new changes now.

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

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

In D32130#1139636, @oaccounts_chingarande.com wrote:

I am taking over this issue and will be responsible for this. let me know how I can assign my self this ticket

• remove ping6.8.gz from ObsoleteFiles.inc

In D42203#966915, @lwhsu wrote:

We also need to remove OLD_FILES+=usr/share/man/man8/ping6.8.gz from ObsoleteFiles.inc.

In D42203#966915, @lwhsu wrote:

We also need to remove OLD_FILES+=usr/share/man/man8/ping6.8.gz from ObsoleteFiles.inc.

Oh and we also have to close it, because it did land! ugh Phabricator is very Project-Management-y :)

In D40188#920799, @jamie wrote:

Commited in eb5bfdd06565. I forgot to add the review to the commit message :-/

In D38826#920142, @ihor_antonovs.family wrote:

IMHO this is superseded by https://reviews.freebsd.org/D40188

In D42203#962979, @bcr wrote:

Why do you need a docs review for this?

This has landed in 5ac2a874d070, closing it.

• Remove the memo and add jail_conf_dir

make sure that zfskeys, ip6addrctl, jail and ipfw_netflow return non-zero when not running

In D37383#849570, @lwhsu wrote:

Don't forget to modify usr.sbin/service/service.8 :)

In D32130#729639, @fernape wrote:

In D32130#729265, @antranigv_freebsd.am wrote:

Hey @0mp , what I did not understand is that if you already changed the code in the Diff or should I resend the patch with your changes using arc? Sorry, first time seeing this :) Let me know!

arc patch D32130 shows that it is not changed ;-)
You need to resend it.

Hey @0mp , what I did not understand is that if you already changed the code in the Diff or should I resend the patch with your changes using arc? Sorry, first time seeing this :) Let me know!

Update the date

• Update date

• Add help menu

• Remove the whitespace

• update manpage for gstat.8

In D24570#721083, @dvl wrote:

Is this at the stage where I could start using it on a test system to provide feedback?

Update the man page, also fix the paths

Change from jail.d to jail.conf.d

Update rc.conf.5

@kp should we merge this or it's stale? :) Just reminding.

Hey @jamie thanks for the feedback.

While I understand @pi 's point, I think jail.conf.d is better. I'm also planning to submit my base-based written in shell jail orchestration tool, using jail.d would be better for integrations (e.g. hooks, scripts, etc).

Fixed styling issue at line 3848

Update the man page according to textproc/igor

Update the date in the man page.

Fixed an unintended slash in rc script

Hmm, I think it looks better this way?