Hey Michael, Jerome, and the rest of the devel list.
I’m not very vocal on this list, and it seems everyone seems to do their best chatting on the mailing list, so I’m bringing my talk here. This about my pull request: https://github.com/ansible/ansible/pull/5325
Jerome, thanks for your input. I’ll review and adjust fire as necessary.
Michael, yes, you’re right, and I probably should have thought about this – that adding new things should attempt to reformat as little existing code as possible. The existing codebase is complex and has a lot of stuff going on. Since I’m a total newb/rookie, my strategy was to trace the existing sudo logic flow and add in support for su in a similar fashion, since they’re accomplishing mostly the same goals. This worked pretty well on the existing ssh.py, but ssh_alt’s pipelining changed things, and while it’s not as pretty as it should be, it seems to be functional. This is the largest and most complex project I’ve contributed to, and I’m here to learn and contribute. Feedback is very welcome. Jerome brought up some good points in the PR that I hadn’t thought of, so I’ll probably drop some fixes in and try to be finished up by Friday this week.
Based on your comments, we’re going to beta this functionality as it’s implemented (a refactor). Jerome suggested breaking the functionality out into some, though I should admit I don’t have a vision for how this could or even should be accomplished.
So, my PR does include ssh and ssh_alt. I excluded hostvars and the other conn plugins. What did I miss, if anything? What’s the path forward – what else can I do to close the loop and get things rolling?