Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(192)

Issue 6331007: Use different proxy ports, and log port conflicts. Hopefully works around (Closed)

Created:
9 years, 11 months ago by dgarrett
Modified:
9 years, 7 months ago
Reviewers:
thieule, sosa
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Use different proxy ports, and log port conflicts. Hopefully works around build problems, and will give better diagnostics if not. Change-Id: I2f7a5d9b2e0c63a8189ad212125c7a5c0178c3c4 BUG=10567 TEST= Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=e70ccb9

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -9 lines) Patch
M bin/cros_au_test_harness.py View 3 chunks +5 lines, -6 lines 0 comments Download
M lib/cros_test_proxy.py View 1 chunk +8 lines, -3 lines 2 comments Download

Messages

Total messages: 2 (0 generated)
dgarrett
9 years, 11 months ago (2011-01-19 19:50:23 UTC) #1
sosa
9 years, 11 months ago (2011-01-19 20:07:54 UTC) #2
Thanks!

LGTM w/ cpl nits

http://codereview.chromium.org/6331007/diff/1/lib/cros_test_proxy.py
File lib/cros_test_proxy.py (right):

http://codereview.chromium.org/6331007/diff/1/lib/cros_test_proxy.py#newcode108
lib/cros_test_proxy.py:108: import os
please import os at the top

http://codereview.chromium.org/6331007/diff/1/lib/cros_test_proxy.py#newcode109
lib/cros_test_proxy.py:109: os.system("sudo netstat -l --tcp -n -p")
use '' rather than "" to be consistent.

Powered by Google App Engine
This is Rietveld 408576698