diff mbox

[Branch,~linaro-validation/lava-dispatcher/trunk] Rev 442: Run bzr branch with BZR_HOME=/dev/null

Message ID 20121109154516.31643.91024.launchpad@ackee.canonical.com
State Accepted
Headers show

Commit Message

Andy Doan Nov. 9, 2012, 3:45 p.m. UTC 
Merge authors:
  Paul Sokolovsky (pfalcon)
Related merge proposals:
  https://code.launchpad.net/~pfalcon/lava-dispatcher/1076381-bzr_home/+merge/133636
  proposed by: Paul Sokolovsky (pfalcon)
  review: Approve - Andy Doan (doanac)
------------------------------------------------------------
revno: 442 [merge]
committer: Andy Doan <andy.doan@linaro.org>
branch nick: lava-dispatcher
timestamp: Fri 2012-11-09 09:43:53 -0600
message:
  Run bzr branch with BZR_HOME=/dev/null
modified:
  lava_dispatcher/actions/lava_test_shell.py


--
lp:lava-dispatcher
https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk

You are subscribed to branch lp:lava-dispatcher.
To unsubscribe from this branch go to https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk/+edit-subscription
diff mbox

Patch

=== modified file 'lava_dispatcher/actions/lava_test_shell.py'
--- lava_dispatcher/actions/lava_test_shell.py	2012-11-02 08:33:10 +0000
+++ lava_dispatcher/actions/lava_test_shell.py	2012-11-08 14:08:30 +0000
@@ -181,7 +181,11 @@ 
             os.chdir(testdir)
             for repo in testdef['install'].get('bzr-repos', []):
                 logging.info("bzr branch %s" % repo)
-                subprocess.check_call(['bzr', 'branch', repo])
+                # Pass non-existent BZR_HOME value, or otherwise bzr may
+                # have non-reproducible behavior because it may rely on
+                # bzr whoami value, presence of ssh keys, etc.
+                subprocess.check_call(['bzr', 'branch', repo],
+                    env={'BZR_HOME': '/dev/null', 'BZR_LOG': '/dev/null'})
                 name = repo.replace('lp:', '').split('/')[-1]
                 self._sw_sources.append(self._bzr_info(repo, name))
             for repo in testdef['install'].get('git-repos', []):