|
|
Created:
6 years, 4 months ago by a.kotkavuo Modified:
6 years, 2 months ago CC:
chromium-reviews Base URL:
https://chromium.googlesource.com/chromium/src.git@master Project:
chromium Visibility:
Public. |
DescriptionRemove references to psutil.error in Android Buildbot scripts
The error module is not a part of psutil's API and it has been moved
to _error in recent releases. The exceptions in psutil are supposed to
be accessed directly from the top level module.
BUG=406396
R=frankf@chromium.org
Committed: https://crrev.com/1a1f95be69c2586798f3872f0c1f6aaea2565609
Cr-Commit-Position: refs/heads/master@{#300724}
Patch Set 1 #Messages
Total messages: 35 (16 generated)
a.kotkavuo@partner.samsung.com changed reviewers: + tonyg@chromium.org
The CQ bit was checked by tonyg@chromium.org
lgtm
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/497763002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
a.kotkavuo@partner.samsung.com changed reviewers: + navabi@chromium.org
The CQ bit was checked by tonyg@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/497763002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
a.kotkavuo@partner.samsung.com changed reviewers: + cmp@chromium.org, craigdh@chromium.org
a.kotkavuo@partner.samsung.com changed reviewers: + ilevy@chromium.org
Pinging owners.
a.kotkavuo@partner.samsung.com changed required reviewers: + cmp@chromium.org
This should be a pretty easy issue to go through and review. It's been almost two months. Is there something I messed up in the process?
jbudorick@chromium.org changed reviewers: - craigdh@chromium.org, frankf@chromium.org, ilevy@chromium.org
jbudorick@chromium.org changed required reviewers: + navabi@chromium.org - cmp@chromium.org
jbudorick@chromium.org changed reviewers: + jbudorick@chromium.org
lgtm + revised reviewers. Armand, can you give this an OWNERS review?
jbudorick@chromium.org changed required reviewers: - navabi@chromium.org
Submitting after OWNERS changes in b/a/buildbot. Thanks for the patch!
The CQ bit was checked by jbudorick@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/497763002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...) linux_chromium_rel on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
On 2014/10/22 17:30:27, I haz the power (commit-bot) wrote: > Try jobs failed on following builders: > chromium_presubmit on tryserver.chromium.linux > (http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...) Apparently, some of the chromium_presubmit bots don't have psutil installed. Infra ticket filed: https://code.google.com/p/chromium/issues/detail?id=426106 > linux_chromium_rel on tryserver.chromium.linux > (http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by jbudorick@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/497763002/1
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/1a1f95be69c2586798f3872f0c1f6aaea2565609 Cr-Commit-Position: refs/heads/master@{#300724} |