WIP: A workaround to wait for client RPC server start-up [chromiumos/third_party/autotest : master]

7 views
Skip to first unread message

Tom Wai-Hong Tam (Code Review)

unread,
Nov 6, 2012, 3:14:48 AM11/6/12
to
Tom Wai-Hong Tam has uploaded a new change for review.

Change subject: WIP: A workaround to wait for client RPC server start-up
......................................................................

WIP: A workaround to wait for client RPC server start-up

Change-Id: If1df1f697dc336f2ace63d6d5e5a5393acc32916
---
M server/cros/servo_test.py
1 file changed, 3 insertions(+), 0 deletions(-)


git pull ssh://gerrit.chromium.org:29418/chromiumos/third_party/autotest refs/changes/31/37431/1
--
To view, visit https://gerrit.chromium.org/gerrit/37431
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If1df1f697dc336f2ace63d6d5e5a5393acc32916
Gerrit-PatchSet: 1
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Tom Wai-Hong Tam <wai...@chromium.org>

Tom Wai-Hong Tam (Code Review)

unread,
Nov 6, 2012, 9:27:49 PM11/6/12
to Gerrit
Tom Wai-Hong Tam has posted comments on this change.

Change subject: faft: A workaround to wait for client RPC server start-up
......................................................................


Patch Set 2:

Seems that we do need this work-around. I notice that this issue happens most of the time. It is a temporary fix.

--
To view, visit https://gerrit.chromium.org/gerrit/37431
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If1df1f697dc336f2ace63d6d5e5a5393acc32916
Gerrit-PatchSet: 2
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Tom Wai-Hong Tam <wai...@chromium.org>
Gerrit-Reviewer: Gerrit <chrom...@google.com>
Gerrit-Reviewer: Tom Wai-Hong Tam <wai...@chromium.org>

Vic Yang (Code Review)

unread,
Nov 6, 2012, 9:29:58 PM11/6/12
to Tom Wai-Hong Tam, Gerrit, Mike Truty, Vadim Bendebury, Yusuf Mohsinally
Vic Yang has posted comments on this change.

Change subject: faft: A workaround to wait for client RPC server start-up
......................................................................


Patch Set 2: Looks good to me, approved

Is there a bug to track this?

--
To view, visit https://gerrit.chromium.org/gerrit/37431
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If1df1f697dc336f2ace63d6d5e5a5393acc32916
Gerrit-PatchSet: 2
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Tom Wai-Hong Tam <wai...@chromium.org>
Gerrit-Reviewer: Gerrit <chrom...@google.com>
Gerrit-Reviewer: Mike Truty <tr...@chromium.org>
Gerrit-Reviewer: Tom Wai-Hong Tam <wai...@chromium.org>
Gerrit-Reviewer: Vadim Bendebury <vbe...@chromium.org>
Gerrit-Reviewer: Vic Yang <victo...@chromium.org>
Gerrit-Reviewer: Yusuf Mohsinally <mohsi...@chromium.org>

Yusuf Mohsinally (Code Review)

unread,
Nov 6, 2012, 9:31:34 PM11/6/12
to Tom Wai-Hong Tam, Gerrit, Mike Truty, Vadim Bendebury, Vic Yang, Yusuf Mohsinally
Yusuf Mohsinally has posted comments on this change.

Change subject: faft: A workaround to wait for client RPC server start-up
......................................................................


Patch Set 2: Looks good to me, but someone else must approve

--
To view, visit https://gerrit.chromium.org/gerrit/37431
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If1df1f697dc336f2ace63d6d5e5a5393acc32916
Gerrit-PatchSet: 2
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Tom Wai-Hong Tam <wai...@chromium.org>
Gerrit-Reviewer: Gerrit <chrom...@google.com>
Gerrit-Reviewer: Mike Truty <tr...@chromium.org>
Gerrit-Reviewer: Tom Wai-Hong Tam <wai...@chromium.org>
Gerrit-Reviewer: Vadim Bendebury <vbe...@chromium.org>
Gerrit-Reviewer: Vic Yang <victo...@chromium.org>
Gerrit-Reviewer: Yusuf Mohsinally <mohsi...@chromium.org>
Gerrit-Reviewer: Yusuf Mohsinally <mohsi...@google.com>

Tom Wai-Hong Tam (Code Review)

unread,
Nov 6, 2012, 9:42:10 PM11/6/12
to Gerrit, Mike Truty, Vadim Bendebury, Vic Yang, Yusuf Mohsinally, Yusuf Mohsinally
Tom Wai-Hong Tam has posted comments on this change.

Change subject: faft: A workaround to wait for client RPC server start-up
......................................................................


Patch Set 3: Verified; Looks good to me, approved; Ready

The bug is created in http://crosbug.com/p/16037

--
To view, visit https://gerrit.chromium.org/gerrit/37431
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If1df1f697dc336f2ace63d6d5e5a5393acc32916
Gerrit-PatchSet: 3
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Tom Wai-Hong Tam <wai...@chromium.org>
Reply all
Reply to author
Forward
0 new messages