[PATCH] conf, passt.1: Un-deprecate --host-lo-to-ns-lo

[PATCH] conf, passt.1: Un-deprecate --host-lo-to-ns-lo

[Stefano Brivio]

It was established behaviour, and it's now the third report about it:
users ask how to achieve the same functionality, and we don't have a
better answer yet.

The idea behind declaring it deprecated to start with, I guess, was
that we would eventually replace it by more flexible and generic
configuration options, which is still planned. But there's nothing
preventing us to alias this in the future to a particular
configuration.

So, stop scaring users off, and un-deprecate this.

Link: https://archives.passt.top/passt-dev/20240925102009.62b9a0ce@elisabeth/
Link: https://github.com/rootless-containers/rootlesskit/pull/482#issuecomment-2591855705
Link: https://github.com/moby/moby/issues/48838
Link: https://github.com/containers/podman/discussions/25243
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
---
 conf.c  | 3 +--
 passt.1 | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/conf.c b/conf.c
index dcfc1d6..db7950e 100644
--- a/conf.c
+++ b/conf.c
@@ -966,8 +966,7 @@ pasta_opts:
 		"  -U, --udp-ns SPEC	UDP port forwarding to init namespace\n"
 		"    SPEC is as described above\n"
 		"    default: auto\n"
-		"  --host-lo-to-ns-lo	DEPRECATED:\n"
-		"			Translate host-loopback forwards to\n"
+		"  --host-lo-to-ns-lo	Translate host-loopback forwards to\n"
 		"			namespace loopback\n"
 		"  --userns NSPATH 	Target user namespace to join\n"
 		"  --netns PATH|NAME	Target network namespace to join\n"
diff --git a/passt.1 b/passt.1
index 63a3a01..395c0c6 100644
--- a/passt.1
+++ b/passt.1
@@ -633,7 +633,7 @@ Configure UDP port forwarding from target namespace to init namespace.
 Default is \fBauto\fR.
 
 .TP
-.BR \-\-host-lo-to-ns-lo " " (DEPRECATED)
+.BR \-\-host-lo-to-ns-lo
 If specified, connections forwarded with \fB\-t\fR and \fB\-u\fR from
 the host's loopback address will appear on the loopback address in the
 guest as well.  Without this option such forwarded packets will appear
-- 
@@ -633,7 +633,7 @@ Configure UDP port forwarding from target namespace to init namespace.
 Default is \fBauto\fR.
 
 .TP
-.BR \-\-host-lo-to-ns-lo " " (DEPRECATED)
+.BR \-\-host-lo-to-ns-lo
 If specified, connections forwarded with \fB\-t\fR and \fB\-u\fR from
 the host's loopback address will appear on the loopback address in the
 guest as well.  Without this option such forwarded packets will appear
-- 
2.43.0

Re: [PATCH] conf, passt.1: Un-deprecate --host-lo-to-ns-lo

[David Gibson]

[-- Attachment #1: Type: text/plain, Size: 2420 bytes --]

On Thu, Feb 06, 2025 at 09:39:57AM +0100, Stefano Brivio wrote:
> It was established behaviour, and it's now the third report about it:
> users ask how to achieve the same functionality, and we don't have a
> better answer yet.
> 
> The idea behind declaring it deprecated to start with, I guess, was
> that we would eventually replace it by more flexible and generic
> configuration options, which is still planned. But there's nothing
> preventing us to alias this in the future to a particular
> configuration.
> 
> So, stop scaring users off, and un-deprecate this.

I concur.

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>

> 
> Link: https://archives.passt.top/passt-dev/20240925102009.62b9a0ce@elisabeth/
> Link: https://github.com/rootless-containers/rootlesskit/pull/482#issuecomment-2591855705
> Link: https://github.com/moby/moby/issues/48838
> Link: https://github.com/containers/podman/discussions/25243
> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
> ---
>  conf.c  | 3 +--
>  passt.1 | 2 +-
>  2 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/conf.c b/conf.c
> index dcfc1d6..db7950e 100644
> --- a/conf.c
> +++ b/conf.c
> @@ -966,8 +966,7 @@ pasta_opts:
>  		"  -U, --udp-ns SPEC	UDP port forwarding to init namespace\n"
>  		"    SPEC is as described above\n"
>  		"    default: auto\n"
> -		"  --host-lo-to-ns-lo	DEPRECATED:\n"
> -		"			Translate host-loopback forwards to\n"
> +		"  --host-lo-to-ns-lo	Translate host-loopback forwards to\n"
>  		"			namespace loopback\n"
>  		"  --userns NSPATH 	Target user namespace to join\n"
>  		"  --netns PATH|NAME	Target network namespace to join\n"
> diff --git a/passt.1 b/passt.1
> index 63a3a01..395c0c6 100644
> --- a/passt.1
> +++ b/passt.1
> @@ -633,7 +633,7 @@ Configure UDP port forwarding from target namespace to init namespace.
>  Default is \fBauto\fR.
>  
>  .TP
> -.BR \-\-host-lo-to-ns-lo " " (DEPRECATED)
> +.BR \-\-host-lo-to-ns-lo
>  If specified, connections forwarded with \fB\-t\fR and \fB\-u\fR from
>  the host's loopback address will appear on the loopback address in the
>  guest as well.  Without this option such forwarded packets will appear

-- 
David Gibson (he or they)	| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you, not the other way
				| around.
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]