faft: Fix logging style [chromiumos/third_party/autotest : master]

17 views
Skip to first unread message

Vic Yang (Code Review)

unread,
Oct 30, 2012, 10:29:06 PM10/30/12
to
Vic Yang has uploaded a new change for review.

Change subject: faft: Fix logging style
......................................................................

faft: Fix logging style

Instead of python string formatting, this uses formatting support from
logging module.

BUG=chrome-os-partner:35902
TEST=Run FAFT suite and passed.

Change-Id: Icaac07f3d42e9fa7c7338e2c17094fa4e5236f72
Signed-off-by: Vic Yang <victo...@chromium.org>
---
M server/cros/faft_checkers.py
M server/cros/faftsequence.py
2 files changed, 33 insertions(+), 30 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icaac07f3d42e9fa7c7338e2c17094fa4e5236f72
Gerrit-PatchSet: 1
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Vic Yang <victo...@chromium.org>

Tom Wai-Hong Tam (Code Review)

unread,
Oct 31, 2012, 2:26:36 AM10/31/12
to Vic Yang, Gerrit, Mike Truty, Vadim Bendebury
Tom Wai-Hong Tam has posted comments on this change.

Change subject: faft: Fix logging style
......................................................................


Patch Set 3: Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icaac07f3d42e9fa7c7338e2c17094fa4e5236f72
Gerrit-PatchSet: 3
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Vic Yang <victo...@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>

Vic Yang (Code Review)

unread,
Oct 31, 2012, 6:43:19 AM10/31/12
to Gerrit, Mike Truty, Vadim Bendebury, Tom Wai-Hong Tam
Vic Yang has posted comments on this change.

Change subject: faft: Fix logging style
......................................................................


Patch Set 5: Verified; Looks good to me, approved

Rebase

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icaac07f3d42e9fa7c7338e2c17094fa4e5236f72
Gerrit-PatchSet: 5
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Vic Yang <victo...@chromium.org>

Vic Yang (Code Review)

unread,
Oct 31, 2012, 8:11:47 AM10/31/12
to Gerrit, Mike Truty, Vadim Bendebury, Tom Wai-Hong Tam
Vic Yang has posted comments on this change.

Change subject: faft: Fix logging style
......................................................................


Patch Set 5: Ready

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icaac07f3d42e9fa7c7338e2c17094fa4e5236f72
Gerrit-PatchSet: 5
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Vic Yang <victo...@chromium.org>

Vic Yang (Code Review)

unread,
Oct 31, 2012, 10:09:30 PM10/31/12
to Gerrit, Mike Truty, Vadim Bendebury, Tom Wai-Hong Tam
Vic Yang has posted comments on this change.

Change subject: faft: Fix logging style
......................................................................


Patch Set 5: Ready

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icaac07f3d42e9fa7c7338e2c17094fa4e5236f72
Gerrit-PatchSet: 5
Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: master
Gerrit-Owner: Vic Yang <victo...@chromium.org>
Reply all
Reply to author
Forward
0 new messages