Skip to content

Conversation

@norio-nomura
Copy link
Contributor

@norio-nomura norio-nomura commented Nov 2, 2025

Remove ControlMaster, ControlPath, and ControlPersist options, because Cygwin-based SSH clients do not support multiplexing when executing commands. References:
https://inbox.sourceware.org/cygwin/c98988a5-7e65-4282-b2a1-bb8e350d5fab@acm.org/T/
https://stackoverflow.com/questions/20959792/is-ssh-controlmaster-with-cygwin-on-windows-actually-possible
By removing these options:

  • Avoids execution failures when the control master is not yet available.
  • Prevents error messages such as:

    mux_client_request_session: read from master failed: Connection reset by peer
    ControlSocket ....sock already exists, disabling multiplexing
    mm_send_fd: sendmsg(2): Connection reset by peer\r\nmux_client_request_session: send fds failed\r\n

…rements on Windows

Remove ControlMaster, ControlPath, and ControlPersist options,
because Cygwin-based SSH clients do not support multiplexing when executing commands.
References:
  https://inbox.sourceware.org/cygwin/c98988a5-7e65-4282-b2a1-bb8e350d5fab@acm.org/T/
  https://stackoverflow.com/questions/20959792/is-ssh-controlmaster-with-cygwin-on-windows-actually-possible
By removing these options:
  - Avoids execution failures when the control master is not yet available.
  - Prevents error messages such as:
    > mux_client_request_session: read from master failed: Connection reset by peer
    > ControlSocket ....sock already exists, disabling multiplexing
    > mm_send_fd: sendmsg(2): Connection reset by peer\\r\\nmux_client_request_session: send fds failed\\r\\n
Signed-off-by: Norio Nomura <norio.nomura@gmail.com>

Signed-off-by: Norio Nomura <norio.nomura@gmail.com>
@norio-nomura norio-nomura force-pushed the remove-controlpath-in-requirements-on-windows branch from 81b55ec to 0d77569 Compare November 2, 2025 09:08
@norio-nomura
Copy link
Contributor Author

I'm expecting this change to fix the following error:

{"time":"2025-10-30T09:08:36.9735373Z","status":{"running":true,"degraded":true,"errors":["failed to satisfy the optional requirement 1 of 2 "systemd must be available": systemd is required to run containerd, but does not seem to be available.\nMake sure that you use an image that supports systemd. If you do not want to run\ncontainerd, please make sure that both 'container.system' and 'containerd.user'\nare set to 'false' in the config file.\n; skipping further checks: stdout="", stderr="mm_send_fd: sendmsg(2): Connection reset by peer\r\nmux_client_request_session: send fds failed\r\n": failed to execute script "systemd must be available": stdout="", stderr="mm_send_fd: sendmsg(2): Connection reset by peer\r\nmux_client_request_session: send fds failed\r\n": exit status 255"],"sshLocalPort":55172}}

@norio-nomura norio-nomura marked this pull request as ready for review November 2, 2025 14:09
@AkihiroSuda AkihiroSuda added this to the v2.0.0 milestone Nov 2, 2025
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@AkihiroSuda AkihiroSuda merged commit 77981cf into lima-vm:master Nov 2, 2025
63 of 64 checks passed
@norio-nomura norio-nomura deleted the remove-controlpath-in-requirements-on-windows branch November 3, 2025 00:32
@norio-nomura
Copy link
Contributor Author

Thanks! 🙏🏻

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.

2 participants