Bug 232186 – Bring up queues for Monterey

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Ryan Haddad]

Bring up queues for Monterey

[Radar WebKit Bug Importer]

<rdar://problem/84568508>

[Ryan Haddad]

Created attachment 442225 [details]
Patch

[Ryan Haddad]

Monterey won't be released until Monday, but I figured I may as well get the patch up.

[Alexey Proskuryakov]

Comment on attachment 442225 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=442225&action=review

> Tools/CISupport/build-webkit-org/config.json:174
> +                    { "name": "Apple-Monterey-Release-WK2-WebGL-GPUProcess-Tests", "factory": "TestAllButJSCFactory", "builddir": "monterey-release-tests-wk2-webgl-gpuprocess",

Looks like we are moving this bot from Big Sur, not adding one for Monterey. Is this the right thing to do?

> Tools/CISupport/build-webkit-org/wkbuild.py:81
>          ("LayoutTests/platform/mac-bigsur", ["mac-bigsur"]),
> +        ("LayoutTests/platform/mac-monterey", ["mac-monterey"]),

This looks wrong, changes to Monterey expectations should trigger tests on all older OSes too. Same for Big Sur.

[Ryan Haddad]

(In reply to Alexey Proskuryakov from comment #4)
> Comment on attachment 442225 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=442225&action=review
> 
> > Tools/CISupport/build-webkit-org/config.json:174
> > +                    { "name": "Apple-Monterey-Release-WK2-WebGL-GPUProcess-Tests", "factory": "TestAllButJSCFactory", "builddir": "monterey-release-tests-wk2-webgl-gpuprocess",
> 
> Looks like we are moving this bot from Big Sur, not adding one for Monterey.
> Is this the right thing to do?

No, it was not. I have updated the patch to leave it in place.

> 
> > Tools/CISupport/build-webkit-org/wkbuild.py:81
> >          ("LayoutTests/platform/mac-bigsur", ["mac-bigsur"]),
> > +        ("LayoutTests/platform/mac-monterey", ["mac-monterey"]),
> 
> This looks wrong, changes to Monterey expectations should trigger tests on
> all older OSes too. Same for Big Sur.

Good catch. Fixed for both Big Sur and Monterey.

[Ryan Haddad]

Created attachment 442388 [details]
[fast-cq] Patch for landing

[EWS]

Committed r284799 (243506@main): <https://commits.webkit.org/243506@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 442388 [details].

[Ryan Haddad]

Reopening for a follow-up fix.

[Ryan Haddad]

Created attachment 443430 [details]
[fast-cq] Patch for landing

[EWS]

Committed r285353 (243912@main): <https://commits.webkit.org/243912@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 443430 [details].