feat: Implement kitchen login for docker transport - #422
Conversation
Signed-off-by: Andrew Bobulsky <rulerof@gmail.com>
damacus
left a comment
There was a problem hiding this comment.
Happy to pull this in but where does it get used?
|
...wowza github notifications are the worst 😅 So if you want to converge a container without relying on having to install and configure openssh inside of the container for transport, you'd use the docker transport: transport:
name: dockerThis works great for transferring files and such, but without the patch, you lose a critical debugging feature, as If you'd like a ready-to-go example, you could try one of my cookbooks that uses my fork. To be candid, I think docker transport is both the crown jewel and missing piece of this driver. The built-in list of container mods that installs/configures openssh and such for various distro containers is really hacky and unreliable with improperly-detected distros or unsupported ones. I may be misdirecting some frustrations I've had with trying to use openssh in containers with systemd under platform emulation, but the requirement to keep SSH working in the container means that ssh-configuring chef code can break ssh-based transports.... you probably get the idea 🙃 |
Description
Implements
kitchen loginwhen using thedockertransport.Issues Resolved
Might not be any, although I briefly mentioned it in #387
Type of Change
_feat_Check List