update "eden daemon" to also pass through KRB5CCNAME

simpkins · facebook-github-bot · commit 645b6fca0a18 · 2017-02-22T18:37:42.000-08:00
Summary:
When the "eden daemon" CLI command has to run edenfs via sudo, also explicitly
pass through the KRB5CCNAME environment variable.  This will allow edenfs and
it's hg import helper to find the correct Kerberos credentials if it needs to
connect to a remote server.

Reviewed By: wez, bolinfest

Differential Revision: D4584132

fbshipit-source-id: 6c72e4eca76d81b3d4b53a9e72a0e8388da30a7d
diff --git a/eden/fs/cli/config.py b/eden/fs/cli/config.py
@@ -435,6 +435,7 @@ def _build_eden_environment(self):
             # contents.  Preserve SSH environment variables needed to do this.
             'SSH_AUTH_SOCK',
             'SSH_AGENT_PID',
+            'KRB5CCNAME',
         ]
 
         for name, value in os.environ.items():

Original file line number	Diff line number	Diff line change
`@@ -435,6 +435,7 @@ def _build_eden_environment(self):`
`435`	`435`	`# contents. Preserve SSH environment variables needed to do this.`
`436`	`436`	`'SSH_AUTH_SOCK',`
`437`	`437`	`'SSH_AGENT_PID',`
	`438`	`+ 'KRB5CCNAME',`
`438`	`439`	`]`
`439`	`440`
`440`	`441`	`for name, value in os.environ.items():`