Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(479)

Issue 1841083002: Change Windows 10 from SP0 to the build number. (Closed)

Created:
4 years, 8 months ago by M-A Ruel
Modified:
4 years, 8 months ago
Reviewers:
tandrii(chromium)
CC:
chromium-reviews, infra-reviews+luci-py_chromium.org, brucedawson
Base URL:
git@github.com:luci/luci-py.git@master
Target Ref:
refs/heads/master
Project:
luci-py
Visibility:
Public.

Description

Change Windows 10 from SP0 to the build number. Previous: os=Windows-10-SP0 Now: os=Windows-10-10586 Call sites will need to be updated accordingly. Next will be to add 'preview' for preview releases. R=tandrii@chromium.org BUG=chromium:513354 Committed: https://github.com/luci/luci-py/commit/5b9c4a499d51778c4d8061097c1a27fead760f20

Patch Set 1 #

Total comments: 2

Patch Set 2 : shuffle lines around #

Unified diffs Side-by-side diffs Delta from patch set Stats (+35 lines, -16 lines) Patch
M appengine/swarming/swarming_bot/api/platforms/win.py View 1 2 chunks +35 lines, -16 lines 0 comments Download

Messages

Total messages: 11 (4 generated)
M-A Ruel
4 years, 8 months ago (2016-03-29 15:38:27 UTC) #1
tandrii(chromium)
LGTM % comment https://codereview.chromium.org/1841083002/diff/1/appengine/swarming/swarming_bot/api/platforms/win.py File appengine/swarming/swarming_bot/api/platforms/win.py (right): https://codereview.chromium.org/1841083002/diff/1/appengine/swarming/swarming_bot/api/platforms/win.py#newcode158 appengine/swarming/swarming_bot/api/platforms/win.py:158: # number. I think this comment ...
4 years, 8 months ago (2016-03-29 15:46:28 UTC) #2
M-A Ruel
https://codereview.chromium.org/1841083002/diff/1/appengine/swarming/swarming_bot/api/platforms/win.py File appengine/swarming/swarming_bot/api/platforms/win.py (right): https://codereview.chromium.org/1841083002/diff/1/appengine/swarming/swarming_bot/api/platforms/win.py#newcode158 appengine/swarming/swarming_bot/api/platforms/win.py:158: # number. On 2016/03/29 15:46:27, tandrii(chromium)-OOO wrote: > I ...
4 years, 8 months ago (2016-03-29 15:51:53 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1841083002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1841083002/20001
4 years, 8 months ago (2016-03-29 15:52:51 UTC) #7
commit-bot: I haz the power
Committed patchset #2 (id:20001) as https://github.com/luci/luci-py/commit/5b9c4a499d51778c4d8061097c1a27fead760f20
4 years, 8 months ago (2016-03-29 15:55:24 UTC) #9
Will Harris
Hi I noticed the bots were still reporting Windows-10-SP0 e.g. https://chromium-swarm.appspot.com/restricted/bot/vm1030-m4 I am about to ...
4 years, 8 months ago (2016-03-29 23:02:24 UTC) #10
M-A Ruel
4 years, 8 months ago (2016-03-30 12:53:36 UTC) #11
Message was sent while issue was closed.
On 2016/03/29 23:02:24, Will Harris wrote:
> Hi I noticed the bots were still reporting Windows-10-SP0 e.g.
> 
> https://chromium-swarm.appspot.com/restricted/bot/vm1030-m4
> 
> I am about to commit some testing code for trybot swarming of some win10 tests
> and would like to know if I should specify dimension Windows-10-SP0 or
> Windows-10-10240 or Windows-10-10586?

I hadn't pushed to prod yet. I just did now.

Powered by Google App Engine
This is Rietveld 408576698