New issue
Advanced search Search tips

Issue 864655 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 24
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 863236



Sign in to add a comment

Swarming: lower default priority to 200

Project Member Reported by mar...@chromium.org, Jul 17

Issue description

The default of 100 doesn't leave enough room for high priority task. Change the clients default to 200 to increase the usable dynamic range.
 
Blocking: 863236
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 23

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/luci/luci-go.git/+/11afeebb3cbc368c8cd2e00cd751f2de31853b66

commit 11afeebb3cbc368c8cd2e00cd751f2de31853b66
Author: Marc-Antoine Ruel <maruel@chromium.org>
Date: Mon Jul 23 18:44:27 2018

[swarming] lower default task priority to 200

If someone cares, they should use an higher priority.

Bug:  864655 
Change-Id: Ie03313af71c3e21f97ea58c7076319bb11777eac
Reviewed-on: https://chromium-review.googlesource.com/1140212
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>

[modify] https://crrev.com/11afeebb3cbc368c8cd2e00cd751f2de31853b66/client/cmd/swarming/trigger.go

Project Member

Comment 3 by bugdroid1@chromium.org, Jul 23

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/luci/luci-py.git/+/a1a5e671a05568d4b145e6c5614d9d932e92e2d3

commit a1a5e671a05568d4b145e6c5614d9d932e92e2d3
Author: Marc-Antoine Ruel <maruel@chromium.org>
Date: Mon Jul 23 19:30:47 2018

[client] lower default task priority to 200

If someone cares, they should use an higher priority.

Disable an unrelated pylint error due to a pylint upgrade.

Bug:  864655 
Change-Id: I985ec5c07729536357e5e247d9575e14ebe00cdf
Reviewed-on: https://chromium-review.googlesource.com/1140210
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/a1a5e671a05568d4b145e6c5614d9d932e92e2d3/client/swarming.py
[modify] https://crrev.com/a1a5e671a05568d4b145e6c5614d9d932e92e2d3/client/tests/swarming_test.py

Project Member

Comment 4 by bugdroid1@chromium.org, Jul 23

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ee2be2e09a5b8bdfc46dd09e1d51b942661db96d

commit ee2be2e09a5b8bdfc46dd09e1d51b942661db96d
Author: Marc-Antoine Ruel <maruel@chromium.org>
Date: Mon Jul 23 23:39:53 2018

Roll src/tools/swarming_client/ 9a518d097..486c9b53c (9 commits)

https://chromium.googlesource.com/infra/luci/client-py.git/+log/9a518d097dca..486c9b53c4d5

$ git log 9a518d097..486c9b53c --date=short --no-merges --format='%ad %ae %s'
2018-07-23 maruel [client] lower default task priority to 200
2018-07-23 maruel Roll infra_libs, gae_event_mon and gae_ts_mon.
2018-06-28 maruel client: fix exception on symlink
2018-06-26 martiniss Revert "Bump version of infra_libs to 1.4.0"
2018-06-25 martiniss Bump version of infra_libs to 1.4.0
2018-06-22 maruel [swarming] more task state documentation
2018-06-20 nodir Add Google RPC protobufs
2018-06-18 maruel [client] Fix regression in d335c4bf86
2018-06-18 maruel [client] Fix regression in a768d983f4

Created with:
  roll-dep src/tools/swarming_client

Bug:  855253 , 864655 
Change-Id: Ic98b9572df6ec364afa2595802abf266e3202a46
Reviewed-on: https://chromium-review.googlesource.com/1147154
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577327}
[modify] https://crrev.com/ee2be2e09a5b8bdfc46dd09e1d51b942661db96d/DEPS

Status: Fixed (was: Started)
Project Member

Comment 6 by bugdroid1@chromium.org, Jul 24

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/luci/luci-py.git/+/55e576612483aa3b24bc707efcd7698667bcfe64

commit 55e576612483aa3b24bc707efcd7698667bcfe64
Author: Marc-Antoine Ruel <maruel@chromium.org>
Date: Tue Jul 24 21:40:56 2018

[swarming] fix local_smoke_test

It regressed in a1a5e671a05568d4b145e6.

TBR=qyearsley@chromium.org
Bug:  864655 
Change-Id: Ibb6124fe7574667228e5657b25fd066e7a1330b8
Reviewed-on: https://chromium-review.googlesource.com/1148798
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>

[modify] https://crrev.com/55e576612483aa3b24bc707efcd7698667bcfe64/appengine/swarming/local_smoke_test.py

Sign in to add a comment