844130 - Enable win8 tests for Cedar

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task, P1)

Description

[Armen [:armenzg]]

We want to run them continually and iron all remaining issues.

We can later enable it hidden on m-i to increase usage of the machines.

Comment 1

[Armen [:armenzg]]

Attachment: win8 buildbotcustom

Comment 2

[Armen [:armenzg]]

I have the following slaves taking jobs on staging:
t-w864-ix-002
t-w864-ix-007
t-w864-ix-008
t-w864-ix-009
t-w864-ix-010
t-w864-ix-042

Between today and tomorrow I will have them running on Cedar.
I'm working on the patches now and I want to see them run well on staging to tell IT that so far I have caught no issues.

Comment 3

[Armen [:armenzg]]

Attachment: win8 configs

Comment 4

[Armen [:armenzg]]

Attachment: win8 buildbotcustom

I attached the wrong patch. It had clutter from another one.

Comment 5

[Jim Mathies [:jimm]]

Cedar and Ash both look pretty busted. We left Elm in a good state if you would like to run tests on that.

Comment 6

[Armen [:armenzg]]

(In reply to Jim Mathies [:jimm] from comment #5)
> Cedar and Ash both look pretty busted. We left Elm in a good state if you
> would like to run tests on that.

Thanks for the offer but there are two reasons for those 2 branches.
Cedar is where we green tests out with the A-team and Ash is where we can test mozharness patches before landing them officially (to make sure we don't break anything).

Comment 7

[Armen [:armenzg]]

Attachment: win8 puppet

Comment 8

[Armen [:armenzg]]

Attachment: win8 graphs

Comment 9

[Armen [:armenzg]]

Attachment: win8 slavealloc (ftr)

I already deploy this one.

Comment 10

[Armen [:armenzg]]

Attachment: win8 buildbotcustom

All patches are ready.
I would like to land this and reconfig in the morning.

Comment 11

[Chris Cooper [:coop] (he/him)]

Comment on attachment 719003 [details] [diff] [review]
win8 puppet

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

::: secrets.pp.template
@@ +104,2 @@
>      $w764_tests_password = 'pass'
>      $vista_tests_password = 'pass'

Can you remove vista from this file too?

Comment 12

[Chris Cooper [:coop] (he/him)]

Comment on attachment 719697 [details] [diff] [review]
win8 buildbotcustom

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

::: process/factory.py
@@ +5767,5 @@
>                           max_depth=4,
>                           property_name="exepath",
>                           name="Find executable",
>                           ))
> +        elif self.OS in ('xp', 'vista', 'win7', 'w764', 'win8'):

Can we remove vista here too?

Comment 13

[Armen [:armenzg]]

These are running on Cedar:
https://tbpl.mozilla.org/?tree=Cedar&jobname=Rev3%20WINNT%206.2

Waiting for the jobs to be picked up and finished.

Comment 14

[Armen [:armenzg]]

Attachment: add win8 passwords on slavealloc (ftr)

I love how preprod does not need a password to be inserted.
This was causing the buildbot.tac to be returned with a password of None.

I already deployed this.

Comment 15

[Armen [:armenzg]]

I now see jobs being taken:
https://tbpl.mozilla.org/?tree=Cedar&jobname=Rev3%20WINNT%206.2

I will use bug 844982 to make those jobs turn green/orange rather than red or purple.
I'm very close to land it.

Comment 16

[Armen [:armenzg]]

(In reply to Armen Zambrano G. [:armenzg] from comment #15)
> I now see jobs being taken:
> https://tbpl.mozilla.org/?tree=Cedar&jobname=Rev3%20WINNT%206.2
> 
> I will use bug 844982 to make those jobs turn green/orange rather than red
> or purple.
> I'm very close to land it.

The jobs should be going green/orange in the next 30mins:
https://tbpl.mozilla.org/?tree=Cedar&jobname=WINNT