Kim Moir [:kmoir] ET Assignee
	Description • 8 years ago

I'll attach a patch

	Kim Moir [:kmoir] ET Assignee
	Comment 2 • 8 years ago

Comment on attachment 8770261 [details] [diff] [review]
bug1286347.patch

and merged to production

	Kim Moir [:kmoir] ET Assignee
	Comment 3 • 8 years ago

just looked at a test master and it reconfig'ed fine with this change

we can expect higher load than normal on fx-team and m-i as a result

jgriffin has credentials to look at the seta server, he is looking to see if apache is running on it

	Kim Moir [:kmoir] ET Assignee
	Comment 4 • 8 years ago

4:35 PM <jgriffin> kmoir: so do we need to add 'windows7-32-vm' here? https://github.com/mozilla/ouija/blob/master/src/server.py#L460
4:36 PM <kmoir> yes
4:36 PM <kmoir> I think so
4:36 PM <kmoir> I don't know what the deployment process is
4:36 PM <kmoir> if you could do that
4:36 PM <kmoir> then restart it or whatver
4:37 PM <kmoir> if it works I could test it from my dev master with seta enabled
4:37 PM <kmoir> I have to go pick up my son at day camp
4:37 PM <kmoir> will be back in an hour 
4:38 PM kmoir → kmoir-afk
4:38 PM <kmoir-afk> does that sound okay?
4:38 PM <jgriffin> kmoir-afk: sure, I think it's fixed now though
4:38 PM <jgriffin> http://alertmanager.allizom.org/data/setadetails/?date=2016-07-12&buildbot=1&branch=fx-team&inactive=1 works
4:38 PM <kmoir-afk> oh okay
4:38 PM <kmoir-afk> yay
4:38 PM <kmoir-afk> okay I will just go check on my master again
4:40 PM nthomas|away → •nthomas
4:41 PM <kmoir-afk> I need to land a fix to skip that platform from seta
4:42 PM <kmoir-afk> but it looks good jgriffin, thanks
4:42 PM <kmoir-afk> my patch is in buildboy
4:42 PM <kmoir-afk> buildbot
4:42 PM <kmoir-afk> hah
4:42 PM <jgriffin> ok great

	Kim Moir [:kmoir] ET Assignee
	Comment 5 • 8 years ago

previous patch can be reverted too with second patch to add platform

	Rail Aliiev [:rail]
	Comment 6 • 8 years ago

Comment on attachment 8770286 [details] [diff] [review]
bug1286347-2.patch

http://alertmanager.allizom.org/data/setadetails/ tells me that it should be "windows7-32-vm". r+ with that fix.

	Kim Moir [:kmoir] ET Assignee
	Comment 7 • 8 years ago

Comment on attachment 8770286 [details] [diff] [review]
bug1286347-2.patch

The script doesn't look at 

http://alertmanager.allizom.org/data/setadetails/ 

it looks at this for example

http://alertmanager.allizom.org/data/setadetails/?date=2016-07-12&buildbot=1&branch=mozilla-inbound&inactive=1

and the key there is 
"Windows 7 32-bit VM"

	Rail Aliiev [:rail]
	Comment 8 • 8 years ago

Comment on attachment 8770286 [details] [diff] [review]
bug1286347-2.patch

Ah, I see. I was too lazy to add those params to the script to get to the proper endpoint :)

	Kim Moir [:kmoir] ET Assignee
	Comment 9 • 8 years ago

patch I landed to re-eanble seta and ignore the new windows platform until we have more data to coalesce tests

	Kim Moir [:kmoir] ET Assignee
	Comment 10 • 8 years ago

I checked a test, build and the scheduling masters and it looks like the reconfig went fine with this change.

	Kim Moir [:kmoir] ET Assignee
	Comment 11 • 8 years ago

verified on test scheduling master that tests are being skipped

opened bug 1286550  to address consuming windows 7 vm data in bbconfigs