How do I enable _use_sudo_login from the cli?

[TyIsI]

As per title.

Unless I've missed something, _use_sudo_login can't be enabled from the command-line, which matters if I want to check a path dependent fact.

Alternatively, would it be better to add the -l flag to the default shell-executable command? This would seem like an easier solution to fix issues like #1374.

EDIT1: Or have a shell-options argument?
EDIT2: The reason I'm asking is because a fact I'm working on depends on correct PATH resolution within the environment.

Happy to submit a PR!

Thanks!

[TyIsI]

Example:

tyisi@Akira:~/workspace/fafo/pyinfra(0)$ uv run pyinfra -y --sudo beast fact nix.Version
--> Loading config...
--> Loading inventory...
--> Connecting to hosts...
    [beast] Connected

--> Gathering facts...
[beast] sudo password:
    [beast] Loaded fact nix.Version

--> Fact data for: nix.Version
{
    "beast": ""
}

--> Disconnecting from hosts...
tyisi@Akira:~/workspace/fafo/pyinfra(0)$ uv run pyinfra -y --sudo --shell-executable "sh -l" beast fact nix.Version
--> Loading config...
--> Loading inventory...
--> Connecting to hosts...
    [beast] Connected

--> Gathering facts...
[beast] sudo password:
    [beast] Loaded fact nix.Version

--> Fact data for: nix.Version
{
    "beast": "2.34.1"
}

--> Disconnecting from hosts...
tyisi@Akira:~/workspace/fafo/pyinfra(0)$

P.S. Yes, I'm working on nix facts and operations providers.

[TyIsI]

I guess a work-around is having a group_data/all.py file with:

_sudo = True
_use_sudo_login = True

[TyIsI]

Oh! There actually is support for a config.py!

And that actually works too! 🤦

[TyIsI]

I'll create an issue for use_sudo_login as a missing CLI arg.

[TyIsI]

Fixed by @wucm667 through #1728