Skip to content

Conversation

norio-nomura
Copy link
Contributor

HostAgent's sshConfig already has some ControlMaster related args in AdditionalArgs, so it is necessary to remove them to avoid overrides above Persist=false.

@norio-nomura
Copy link
Contributor Author

expecting to avoids error like:
https://github.com/lima-vm/lima/actions/runs/17967205396/job/51101966512?pr=4081#step:7:108

time="2025-09-24T05:38:26Z" level=info msg="[hostagent] ControlSocket /c/Users/runneradmin/.lima/default/ssh.sock already exists, disabling multiplexing\r"

@norio-nomura norio-nomura marked this pull request as draft September 25, 2025 00:44
@norio-nomura
Copy link
Contributor Author

In addition, I think I will make a PR to operate args related to ControlMaster, so I will reopen it after that.

@norio-nomura norio-nomura force-pushed the remove-control-master-args-from-reversesshfs-config-on-windows branch from 9ad821b to f536261 Compare September 30, 2025 06:50
@norio-nomura
Copy link
Contributor Author

norio-nomura commented Sep 30, 2025

Updated to rebased on #4093

@norio-nomura norio-nomura force-pushed the remove-control-master-args-from-reversesshfs-config-on-windows branch 4 times, most recently from 66ebdc0 to 0a507c7 Compare October 3, 2025 02:29
…config on Windows

HostAgent's `sshConfig` already has some ControlMaster related args in `AdditionalArgs`,
so it is necessary to remove them to avoid overrides above `Persist=false`.

Signed-off-by: Norio Nomura <norio.nomura@gmail.com>
@norio-nomura norio-nomura force-pushed the remove-control-master-args-from-reversesshfs-config-on-windows branch from 0a507c7 to 21ed649 Compare October 7, 2025 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant