Re: [PATCH] Provide completion candidates for remote containers over a TRAMP connection

[Gene Goykhman]

Hi Michael,

Michael Albinus <michael.albinus@gmx.de> wrote:

> For the futere I could imagine more different values. t and nil are fine
> ATM, but if there are more methods which support this feature, it could
> be a list. '("docker" "podman" "kubernetes") would enable this for the
> respective methods, and disable it for all other methods in multi-hop
> completions. WDYT?

I think this is a great idea. What about calling the variable 
`tramp-completion-multi-hop-methods', with a default of '("docker" "podman")? I 
am not confident including "kubernetes" here as I have not tested it but could 
do so if you are comfortable with that.

> Furthermore, the result of tramp-container--completion-function is
> cached now. I've obeserved that the function is called 5 times in a row;
> we don't need so many shells to be applied on remote.

Ah nice. I hadn't noticed this.

> What you could perhaps contribute is documentation of
> tramp-completion-remote-containers (with its changed name) in
> tramp.texi. Maybe in node "Ad-hoc multi-hops"?

I'll get started on this and an addition to etc/NEWS about the new user option.