If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

add more g2.2xlarge machines to buildbot-configs and slavealloc

RESOLVED FIXED

Status

Release Engineering
Buildduty
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jlund, Assigned: aobreja)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

Comment hidden (empty)
(Reporter)

Comment 1

a year ago
currently we have:

300 use1 entries in slavealloc. only 100 of those are enabled

200 usw2 entries in slaveallow. *0* of those are enabled
See Also: → bug 1229791
(Reporter)

Comment 2

a year ago
(In reply to Jordan Lund (:jlund) from comment #1)
> currently we have:
> 
> 300 use1 entries in slavealloc. only 100 of those are enabled
> 
> 200 usw2 entries in slaveallow. *0* of those are enabled

sorry, to be more accurate. we have:

* 100 use1 "g-w732-spot-* g2.2xlarge" enabled
* 0 usw2   "g-w732-spot-* g2.2xlarge" enabled
* 200 use1 "t-w10-spot-* g2.2xlarge" disabled
* 100 usw2 "t-w10-spot-* g2.2xlarge" disabled

can the 't-w10-spot-*' be used for 'g-w732-spot-*' related jobs?

at any rate, we need usw2 spot instances enabled. Ideally every machine should be the same. e.g. "g-w732-spot-*"
(Reporter)

Updated

a year ago
Blocks: 1302530
(In reply to Jordan Lund (:jlund) from comment #2)
> * 100 use1 "g-w732-spot-* g2.2xlarge" enabled
> * 0 usw2   "g-w732-spot-* g2.2xlarge" enabled
> * 200 use1 "t-w10-spot-* g2.2xlarge" disabled
> * 100 usw2 "t-w10-spot-* g2.2xlarge" disabled
> 
> can the 't-w10-spot-*' be used for 'g-w732-spot-*' related jobs?

This is purely a nomenclature thing to make it clear when an instance is running tests for w7 vs w10. 

We should add 100 entries for g-w732-spot to usw2 in slavealloc. Make sure we update the buildbot-configs too.
Created attachment 8791261 [details]
bug1302549_slavealloc.csv

Adding the slaves to slavealloc.
Attachment #8791261 - Flags: review?(coop)
Created attachment 8791262 [details] [diff] [review]
bug1302549_bb.patch

Buildbot-config changes.
Attachment #8791262 - Flags: review?(coop)
Comment on attachment 8791261 [details]
bug1302549_slavealloc.csv

datacenter is listed as "us-weast-2" but should be "us-west-2"

r+ with that change
Attachment #8791261 - Flags: review?(coop) → review+
Comment on attachment 8791262 [details] [diff] [review]
bug1302549_bb.patch

Review of attachment 8791262 [details] [diff] [review]:
-----------------------------------------------------------------

::: mozilla-tests/production_config.py
@@ +26,4 @@
>  for i in range(1, 101) + range(102, 600):  # Omit 101 due to win7 golden issues // Bug 1223509
>      SLAVES['win7_vm']['t-w732-spot-%03i' % i] = {}
>  
> +for i in range(1, 201):

Your CSV is adding up to g-w732-spot-201, so it needs to be range(1, 202) because of the way range() works.

r+ with that change.
Attachment #8791262 - Flags: review?(coop) → review+
Created attachment 8791497 [details]
bug1302549_slavealloc.csv

New csv with changes west and last machine g-w732-spot-200.
Attachment #8791261 - Attachment is obsolete: true
Created attachment 8791498 [details]
bug1302549_slavealloc.csv
Attachment #8791497 - Attachment is obsolete: true
(Assignee)

Updated

a year ago
Attachment #8791262 - Flags: checked-in+
(Assignee)

Updated

a year ago
Attachment #8791498 - Flags: checked-in+
New 100 g-w732-spot machines were added having this configuration (usw2 and g2.2xlarge)

New machines are g-w732-spot-[101-200]
mysql> select count(*) from slaves where name like 'g-w732-spot%';
+----------+
| count(*) |
+----------+
|      200 |
+----------+
1 row in set (0.01 sec)

mysql> select slaveid, name from slaves where name like 'g-w732-spot%' order by cast(substring(name, -3) as unsigned) desc limit 5;
+---------+-----------------+
| slaveid | name            |
+---------+-----------------+
|   32210 | g-w732-spot-200 |
|   32208 | g-w732-spot-199 |
|   32206 | g-w732-spot-198 |
|   32204 | g-w732-spot-197 |
|   32202 | g-w732-spot-196 |
+---------+-----------------+
5 rows in set (0.00 sec)

Should we enable these new 100 machiens?
Flags: needinfo?(coop)
Assignee: nobody → aobreja
(In reply to Andrei Obreja [:aobreja][:buildduty] from comment #10)
> Should we enable these new 100 machines?

Yes, please.
Flags: needinfo?(coop)
All new g-w732-spot machines were enabled.
(Assignee)

Updated

a year ago
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.