Kind reminder.

Raw diff at https://marc.info/?l=openbsd-ports&m=178388909772054&q=mbox


On Sun, Jul 12, 2026 at 08:45:13PM +0000, Mikolaj Kucharski wrote:
> On Wed, Jul 01, 2026 at 02:04:14PM +0100, Stuart Henderson wrote:
> > On 2026/07/01 12:27, Mikolaj Kucharski wrote:
> > > Sometime ago, I think after this year Python upgrades rc.d(8) script
> > > stopped working with salt minion (I don't have master on -current
> > > OpenBSD). I see that on OpenBSD 7.9 proctitle works correctly, so it
> > > was after the 7.9 release.
> > 
> > I don't know about salt, but setproctitle does work properly on -current
> > 
> > $ python3
> > Python 3.14.6 (main, Jun 28 2026, 14:43:55) [Clang 22.1.6 ] on openbsd7
> > Type "help", "copyright", "credits" or "license" for more information.
> > >>> import setproctitle
> > >>> setproctitle.setproctitle('foo')
> > >>>
> > 
> > $ pgrep -lf python3
> > 52780 python3: foo
> > <snip others>
> > 
> 
> Here is a very minimal patch to fix the rc.d(8) proc title issue.
> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/sysutils/salt/Makefile,v
> diff -u -p -u -r1.201 Makefile
> --- Makefile	28 Feb 2026 07:01:30 -0000	1.201
> +++ Makefile	12 Jul 2026 20:38:52 -0000
> @@ -17,6 +17,7 @@ COMMENT =		remote execution and configur
>  
>  MODPY_DISTV =		3007.13
>  DISTNAME =		salt-${MODPY_DISTV}
> +REVISION =		0
>  
>  CATEGORIES =		sysutils net devel
>  
> Index: patches/patch-salt_scripts_py
> ===================================================================
> RCS file: patches/patch-salt_scripts_py
> diff -N patches/patch-salt_scripts_py
> --- /dev/null	1 Jan 1970 00:00:00 -0000
> +++ patches/patch-salt_scripts_py	12 Jul 2026 20:38:52 -0000
> @@ -0,0 +1,33 @@
> +Force the multiprocessing "fork" start method.  Python 3.14 changed the
> +POSIX default to "forkserver" (cpython gh-84559), which breaks Salt's
> +fork-based process model, setproctitle output and rc.d(8) pgrep matching.
> +
> +Index: salt/scripts.py
> +--- salt/scripts.py.orig
> ++++ salt/scripts.py
> +@@ -5,6 +5,7 @@ This module contains the function calls to execute com
> + import contextlib
> + import functools
> + import logging
> ++import multiprocessing
> + import os
> + import signal
> + import subprocess
> +@@ -16,6 +17,17 @@ from random import randint
> + 
> + import salt.defaults.exitcodes
> + from salt.exceptions import SaltClientError, SaltReqTimeoutError, SaltSystemExit
> ++
> ++if sys.platform != "win32":
> ++    # Salt's process model (ProcessManager, appendproctitle, signal
> ++    # escalation, daemonization) assumes fork semantics.  Python 3.14
> ++    # changed the default POSIX start method from "fork" to "forkserver"
> ++    # (https://github.com/python/cpython/issues/84559), which breaks
> ++    # process titles and process supervision.  Force "fork" explicitly.
> ++    try:
> ++        multiprocessing.set_start_method("fork", force=True)
> ++    except (RuntimeError, ValueError):
> ++        pass
> + 
> + log = logging.getLogger(__name__)
> + 
> 

-- 
Regards,
 Mikolaj