Explitly unset SSH_AUTH_SOCK; it is preferred to the -i flag
authorAlex Vandiver <alexmv@dropbox.com>
Wed, 22 Jul 2015 18:51:37 +0000 (11:51 -0700)
committerAlex Vandiver <alexmv@dropbox.com>
Wed, 22 Jul 2015 18:51:40 +0000 (11:51 -0700)
commit2a498ae29cd2d4eeac52fa123bb97e5e2d5b1c90
tree932d58e2ff3d5b4984ff92eb14dcbbdf78d829c2
parent231f711e3f0ccb8601bea55f0e6d562b8674488e
Explitly unset SSH_AUTH_SOCK; it is preferred to the -i flag

Even if the -i flag is explicitly passed via the command line, openssh
ignores the identity file if an socket exists to an SSH authentication
agent.  In cases where puppet has been configured to use an explicit
identity file, altering behavior based on the calling environment of
puppet violates the principle of least surprise, and can lead to
inconsistent deployments.

Work around this odd corner case of ssh by explicitly unsetting
SSH_AUTH_SOCK inside the ssh wrapper.
lib/puppet/provider/vcsrepo/git.rb