|
|
DescriptionRevert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ )
Reason for revert:
crbug.com/559154 should be fixed
Original issue's description:
> Set the number of processes to use for telemetry_unittests on
> win platform to be 1 to temporarily fix the race condition on
> win bot.
> Cr-Commit-Position: refs/heads/master@{#361170}
TBR=maruel@chromium.org
NOTRY=true
BUG=559154
Committed: https://crrev.com/7a61e826d13ad9093ff81b6f566baeacd9494254
Cr-Commit-Position: refs/heads/master@{#363326}
Patch Set 1 #
Total comments: 2
Patch Set 2 : Add trailing comma #Messages
Total messages: 26 (14 generated)
Created Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1
Description was changed from ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > > BUG=559154 > NOTRY=true > > Committed: https://crrev.com/b3b8c6ebf55d4bb55d728456dfc675115da45dce > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=559154 ========== to ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org BUG=559154 ==========
The CQ bit was checked by nednguyen@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1498093002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1498093002/1
lgtm https://codereview.chromium.org/1498093002/diff/1/chrome/telemetry_unittests.... File chrome/telemetry_unittests.isolate (right): https://codereview.chromium.org/1498093002/diff/1/chrome/telemetry_unittests.... chrome/telemetry_unittests.isolate:23: '..' I'd prefer to keep the comma. :)
The CQ bit was unchecked by nednguyen@google.com
https://codereview.chromium.org/1498093002/diff/1/chrome/telemetry_unittests.... File chrome/telemetry_unittests.isolate (right): https://codereview.chromium.org/1498093002/diff/1/chrome/telemetry_unittests.... chrome/telemetry_unittests.isolate:23: '..' On 2015/12/04 17:41:35, M-A Ruel wrote: > I'd prefer to keep the comma. :) Done.
The CQ bit was checked by nednguyen@google.com
The patchset sent to the CQ was uploaded after l-g-t-m from maruel@chromium.org Link to the patchset: https://codereview.chromium.org/1498093002/#ps40001 (title: "Add trailing comma")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1498093002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1498093002/40001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: android_clang_dbg_recipe on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...) cast_shell_linux on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...) linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...) mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...) mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
The CQ bit was checked by nednguyen@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1498093002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1498093002/40001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
The CQ bit was checked by nednguyen@google.com
On 2015/12/05 00:00:21, commit-bot: I haz the power wrote: > Try jobs failed on following builders: > ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED, > http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...) Add NOTRY=true since failure in ios_dbg_simulator_ninja is unrelated
Description was changed from ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org BUG=559154 ========== to ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org NOTRY=true BUG=559154 ==========
The CQ bit was unchecked by nednguyen@google.com
The CQ bit was checked by nednguyen@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1498093002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1498093002/40001
Message was sent while issue was closed.
Description was changed from ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org NOTRY=true BUG=559154 ========== to ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org NOTRY=true BUG=559154 ==========
Message was sent while issue was closed.
Committed patchset #2 (id:40001)
Message was sent while issue was closed.
Description was changed from ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org NOTRY=true BUG=559154 ========== to ========== Revert of [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (patchset #3 id:40001 of https://codereview.chromium.org/1471673003/ ) Reason for revert: crbug.com/559154 should be fixed Original issue's description: > Set the number of processes to use for telemetry_unittests on > win platform to be 1 to temporarily fix the race condition on > win bot. > Cr-Commit-Position: refs/heads/master@{#361170} TBR=maruel@chromium.org NOTRY=true BUG=559154 Committed: https://crrev.com/7a61e826d13ad9093ff81b6f566baeacd9494254 Cr-Commit-Position: refs/heads/master@{#363326} ==========
Message was sent while issue was closed.
Patchset 2 (id:??) landed as https://crrev.com/7a61e826d13ad9093ff81b6f566baeacd9494254 Cr-Commit-Position: refs/heads/master@{#363326} |