747460 - consolidate windows peptest config files

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P5)

Description

[Aki Sasaki (not active)]

In peptest, we currently have prod_config.py (linux+mac), w764_config.py, win7_config.py, and xp_config.py.

I had to separate win7_config.py and xp_config.py in staging due to a differing virtualenv.  However, this difference didn't exist in production, so I ended up making the win7_config.py the same.  w764_config.py came about because of the windows logic in mozilla-tests/config.py was breaking on w764:

http://hg.mozilla.org/build/buildbot-configs/file/2bdb451619d9/mozilla-tests/config.py#l1228

Now we have 3 config files that are exactly the same, with logic on how to call them separately.

We should consolidate the config files to one windows_config.py and call that for all 3 platforms.

Comment 1

[Aki Sasaki (not active)]

Attachment: [mozharness] copy xp_config.py to windows_config.py

Rail: these 3 reviews aren't urgent (next week is fine), but they're pretty simple.

This patch copies xp_config.py to windows_config.py and can land at any point.

Comment 2

[Aki Sasaki (not active)]

Attachment: [configs] point to windows_config.py

This points windows peptest runs at windows_config.py.
This patch needs to land after attachment 617087 [details] [diff] [review], and will need a reconfig.

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 617087 [details] [diff] [review]
[mozharness] copy xp_config.py to windows_config.py

The easiest patch I've ever seen!

Comment 4

[Aki Sasaki (not active)]

Attachment: [mozharness] remove {win7,w764,xp}_config.py

This patch removes the extraneous xp,win7,w764 config files after buildbot is pointing at the new windows_config.py file.

This patch can land at any point after the reconfig for attachment 617089 [details] [diff] [review].

Comment 5

[Aki Sasaki (not active)]

Comment on attachment 617087 [details] [diff] [review]
[mozharness] copy xp_config.py to windows_config.py

http://hg.mozilla.org/build/mozharness/rev/69ef1c522d9f

Comment 6

[Aki Sasaki (not active)]

Comment on attachment 617089 [details] [diff] [review]
[configs] point to windows_config.py

http://hg.mozilla.org/build/buildbot-configs/rev/9b9d9927e501

Comment 7

[Aki Sasaki (not active)]

Comment on attachment 617090 [details] [diff] [review]
[mozharness] remove {win7,w764,xp}_config.py

http://hg.mozilla.org/build/mozharness/rev/6dc073486c2e

Comment 8

[Rail Aliiev [:rail]]

Landed to production yesterday.