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

Enable win8 tests for Cedar

RESOLVED FIXED

Status

Release Engineering
Platform Support
P1
normal
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: armenzg, Assigned: armenzg)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(6 attachments, 2 obsolete attachments)

(Assignee)

Description

5 years ago
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.
(Assignee)

Updated

5 years ago
Depends on: 844126
(Assignee)

Updated

5 years ago
Blocks: 780050
(Assignee)

Updated

5 years ago
Depends on: 844982
(Assignee)

Comment 1

5 years ago
Created attachment 718020 [details] [diff] [review]
win8 buildbotcustom
Attachment #718020 - Flags: review?(coop)
(Assignee)

Comment 2

5 years ago
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.
(Assignee)

Comment 3

5 years ago
Created attachment 718554 [details] [diff] [review]
win8 configs
Attachment #718554 - Flags: review?(coop)
(Assignee)

Comment 4

5 years ago
Created attachment 718555 [details] [diff] [review]
win8 buildbotcustom

I attached the wrong patch. It had clutter from another one.
Attachment #718020 - Attachment is obsolete: true
Attachment #718020 - Flags: review?(coop)
Attachment #718555 - Flags: review?(coop)

Updated

5 years ago
Attachment #718554 - Flags: review?(coop) → review+

Updated

5 years ago
Attachment #718555 - Flags: review?(coop) → review+

Comment 5

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

Comment 6

5 years ago
(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).
(Assignee)

Comment 7

5 years ago
Created attachment 719003 [details] [diff] [review]
win8 puppet
Attachment #719003 - Flags: review?(coop)
(Assignee)

Comment 8

5 years ago
Created attachment 719004 [details] [diff] [review]
win8 graphs
Attachment #719004 - Flags: review?(coop)
(Assignee)

Comment 9

5 years ago
Created attachment 719005 [details] [diff] [review]
win8 slavealloc (ftr)

I already deploy this one.
(Assignee)

Comment 10

5 years ago
Created attachment 719697 [details] [diff] [review]
win8 buildbotcustom

All patches are ready.
I would like to land this and reconfig in the morning.
Attachment #719697 - Flags: review?(coop)
(Assignee)

Updated

5 years ago
OS: Mac OS X → Windows 8
Priority: -- → P1
Hardware: x86 → x86_64
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?
Attachment #719003 - Flags: review?(coop) → review+

Updated

5 years ago
Attachment #719004 - Flags: review?(coop) → review+
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?
Attachment #719697 - Flags: review?(coop) → review+
(Assignee)

Updated

5 years ago
Attachment #718554 - Flags: checked-in+
(Assignee)

Updated

5 years ago
Attachment #718555 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Attachment #719003 - Flags: checked-in+
(Assignee)

Updated

5 years ago
Attachment #719004 - Flags: checked-in+
(Assignee)

Comment 13

5 years ago
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.
(Assignee)

Updated

5 years ago
Depends on: 846823
(Assignee)

Comment 14

5 years ago
Created attachment 720087 [details] [diff] [review]
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.
Attachment #720087 - Flags: checked-in+
(Assignee)

Comment 15

5 years ago
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.
Summary: Enable win8 tests for Cedar and Ash → Enable win8 tests for Cedar
(Assignee)

Updated

5 years ago
Attachment #719697 - Flags: checked-in+
(Assignee)

Comment 16

5 years ago
(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
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Updated

5 years ago
No longer depends on: 846823
(Assignee)

Updated

5 years ago
No longer depends on: 844982
(Assignee)

Updated

5 years ago
Blocks: 826050
No longer blocks: 731280
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.