Bug 222616 - Add Apple Silicon Devices to Big Sur builders and testers.
Summary: Add Apple Silicon Devices to Big Sur builders and testers.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Matt Lewis
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-03-02 13:50 PST by Matt Lewis
Modified: 2021-03-04 15:40 PST (History)
4 users (show)

See Also:


Attachments
Patch (7.48 KB, patch)
2021-03-02 14:01 PST, Matt Lewis
no flags Details | Formatted Diff | Diff
Patch (12.94 KB, patch)
2021-03-03 10:41 PST, Matt Lewis
no flags Details | Formatted Diff | Diff
Patch (12.76 KB, patch)
2021-03-03 13:26 PST, Matt Lewis
no flags Details | Formatted Diff | Diff
Patch (15.07 KB, patch)
2021-03-03 13:56 PST, Matt Lewis
no flags Details | Formatted Diff | Diff
Patch (15.06 KB, patch)
2021-03-03 14:01 PST, Matt Lewis
no flags Details | Formatted Diff | Diff
Patch (15.08 KB, patch)
2021-03-03 15:01 PST, Matt Lewis
no flags Details | Formatted Diff | Diff
Patch (15.21 KB, patch)
2021-03-04 10:32 PST, Matt Lewis
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Lewis 2021-03-02 13:50:10 PST
We are bringing Apple Silicon Devices online for Big Sur building and testing.
Comment 1 Radar WebKit Bug Importer 2021-03-02 13:50:20 PST
<rdar://problem/74946994>
Comment 2 Matt Lewis 2021-03-02 14:01:42 PST
Created attachment 421993 [details]
Patch
Comment 3 Jonathan Bedard 2021-03-02 14:23:14 PST
(In reply to Matt Lewis from comment #2)
> Created attachment 421993 [details]
> Patch

Looks like the patch is out of date.
Comment 4 Jonathan Bedard 2021-03-02 14:24:43 PST
Comment on attachment 421993 [details]
Patch

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

> Tools/CISupport/build-webkit-org/config.json:3
> +                    { "name": "bot105", "platform": "mac-bigsur" },

Why are we adding spaces after each platform, when they were not there before?
Comment 5 Matt Lewis 2021-03-03 09:43:36 PST
(In reply to Jonathan Bedard from comment #4)
> Comment on attachment 421993 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=421993&action=review
> 
> > Tools/CISupport/build-webkit-org/config.json:3
> > +                    { "name": "bot105", "platform": "mac-bigsur" },
> 
> Why are we adding spaces after each platform, when they were not there
> before?

Correct.(In reply to Jonathan Bedard from comment #3)
> (In reply to Matt Lewis from comment #2)
> > Created attachment 421993 [details]
> > Patch
> 
> Looks like the patch is out of date.
Yep going to need a rebase
Comment 6 Matt Lewis 2021-03-03 10:41:30 PST
Created attachment 422114 [details]
Patch
Comment 7 Jonathan Bedard 2021-03-03 11:06:23 PST
(In reply to Matt Lewis from comment #6)
> Created attachment 422114 [details]
> Patch

Failing services EWS
Comment 8 Matt Lewis 2021-03-03 13:26:38 PST
Created attachment 422143 [details]
Patch
Comment 9 Matt Lewis 2021-03-03 13:27:22 PST
(In reply to Jonathan Bedard from comment #7)
> (In reply to Matt Lewis from comment #6)
> > Created attachment 422114 [details]
> > Patch
> 
> Failing services EWS
I made a mistake in the triggers. Fixing
Comment 10 Matt Lewis 2021-03-03 13:56:12 PST
Created attachment 422147 [details]
Patch
Comment 11 Matt Lewis 2021-03-03 14:01:26 PST
Created attachment 422148 [details]
Patch
Comment 12 Ryan Haddad 2021-03-03 14:13:03 PST
Comment on attachment 422148 [details]
Patch

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

> Tools/CISupport/build-webkit-org/config.json:204
> +                      "platform": "mac-bigsur", "configuration": "debug", "architectures": ["arm64"],

Worth verifying, but I think only specifying arm64 will make these fail to find archives since we are only producing multi-arch ones.

> Tools/CISupport/build-webkit-org/config.json:208
> +                      "platform": "mac-bigsur", "configuration": "release", "architectures": ["arm64"],

Ditto.
Comment 13 Aakash Jain 2021-03-03 14:21:04 PST
(In reply to Ryan Haddad from comment #12)
> Worth verifying, but I think only specifying arm64 will make these fail to
> find archives since we are only producing multi-arch ones.
Would be good to test this patch on dev instance.
Comment 14 Matt Lewis 2021-03-03 15:01:59 PST
Created attachment 422152 [details]
Patch
Comment 15 Matt Lewis 2021-03-03 17:09:24 PST
(In reply to Aakash Jain from comment #13)
> (In reply to Ryan Haddad from comment #12)
> > Worth verifying, but I think only specifying arm64 will make these fail to
> > find archives since we are only producing multi-arch ones.
> Would be good to test this patch on dev instance.

Doing this now. Looks to be fine.
Comment 16 Matt Lewis 2021-03-04 08:40:06 PST
(In reply to Matt Lewis from comment #15)
> (In reply to Aakash Jain from comment #13)
> > (In reply to Ryan Haddad from comment #12)
> > > Worth verifying, but I think only specifying arm64 will make these fail to
> > > find archives since we are only producing multi-arch ones.
> > Would be good to test this patch on dev instance.
> 
> Doing this now. Looks to be fine.

the build and triggered testing for JSC

https://build.webkit-dev.org/#/builders/48/builds/5
https://build.webkit-dev.org/#/builders/81/builds/2
Comment 17 Matt Lewis 2021-03-04 10:32:32 PST
Created attachment 422254 [details]
Patch
Comment 18 Matt Lewis 2021-03-04 10:42:57 PST
Re running dev instance builds and test, will link them here when they complete.
Comment 20 Ryan Haddad 2021-03-04 14:47:10 PST
The layout test runs above didn't work to an unrelated change on the dev instance. It is working now: https://build.webkit-dev.org/#/builders/85/builds/2

This should be good for review.
Comment 21 Alexey Proskuryakov 2021-03-04 15:11:25 PST
Comment on attachment 422254 [details]
Patch

We are still specifying Buildbot.BuildArchitecture.SixtyFourBit on some of the queues. Would it make sense to differentiate x86_64 and Apple Silicon using Buildbot.BuildArchitecture? I don't remember what effect is has.
Comment 22 Matt Lewis 2021-03-04 15:40:44 PST
Comment on attachment 422254 [details]
Patch

Clearing flags on attachment: 422254

Committed r273926 (234880@main): <https://commits.webkit.org/234880@main>
Comment 23 Matt Lewis 2021-03-04 15:40:47 PST
All reviewed patches have been landed.  Closing bug.