Bug 140627 - Add iOS EWS to build.webkit.org/dashboard
Summary: Add iOS EWS to build.webkit.org/dashboard
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-19 10:50 PST by Alexey Proskuryakov
Modified: 2022-02-28 04:12 PST (History)
7 users (show)

See Also:


Attachments
Patch v1 (119.45 KB, patch)
2015-01-24 08:17 PST, David Kilzer (:ddkilzer)
ap: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2015-01-19 10:50:39 PST
This is necessary to easily keep track of iOS EWS health.

The code is in Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard
Comment 1 Alexey Proskuryakov 2015-01-21 10:26:25 PST
It also needs to be added to dashboard metrics page.
Comment 2 David Kilzer (:ddkilzer) 2015-01-24 07:45:42 PST
(In reply to comment #0)
> This is necessary to easily keep track of iOS EWS health.
> 
> The code is in
> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard

https://build.webkit.org/dashboard/

(In reply to comment #1)
> It also needs to be added to dashboard metrics page.

https://build.webkit.org/dashboard/metrics.html
Comment 3 David Kilzer (:ddkilzer) 2015-01-24 08:17:17 PST
Created attachment 245283 [details]
Patch v1
Comment 4 David Kilzer (:ddkilzer) 2015-01-24 08:18:18 PST
(In reply to comment #3)
> Created attachment 245283 [details]
> Patch v1

I have no clue if any of this is correct; it's the first time I've modified this code, and I'm not sure if there is a way to test this locally.
Comment 5 Alexey Proskuryakov 2015-01-24 08:34:10 PST
> I'm not sure if there is a way to test this locally.

Yes, just open the main HTML files in Safari.
Comment 6 Alexey Proskuryakov 2015-01-24 08:50:38 PST
Comment on attachment 245283 [details]
Patch v1

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

What I see in the patch all looks right. Please test locally to make sure that all the needed pieces are in place.

> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/Dashboard.js:28
> +        iOS8Device: { name: "ios-8", readableName: "iOS 8", order: 5 },

So you want iOS to be the first line on the dashboard? I concur, however right now, the line is almost empty, and thus not so useful. Perhaps the right thing to do is to put it under Mac for now, and to move up once it's more complete?
Comment 7 David Kilzer (:ddkilzer) 2015-01-24 18:19:33 PST
(In reply to comment #6)
> Comment on attachment 245283 [details]
> Patch v1
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=245283&action=review
> 
> What I see in the patch all looks right. Please test locally to make sure
> that all the needed pieces are in place.

How do I do that?  I mentioned above I don't know how to test this locally.

> > Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/Dashboard.js:28
> > +        iOS8Device: { name: "ios-8", readableName: "iOS 8", order: 5 },
> 
> So you want iOS to be the first line on the dashboard? I concur, however
> right now, the line is almost empty, and thus not so useful. Perhaps the
> right thing to do is to put it under Mac for now, and to move up once it's
> more complete?

That's fine.
Comment 8 Tim Horton 2015-01-24 18:24:37 PST
(In reply to comment #7)
> (In reply to comment #6)
> > Comment on attachment 245283 [details]
> > Patch v1
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=245283&action=review
> > 
> > What I see in the patch all looks right. Please test locally to make sure
> > that all the needed pieces are in place.
> 
> How do I do that?  I mentioned above I don't know how to test this locally.

Comment #5 explains.
Comment 9 Timothy Hatcher 2015-01-24 19:05:50 PST
You might need to disable local file restrictions in the develop menu for it to work.
Comment 10 David Kilzer (:ddkilzer) 2015-01-26 14:42:15 PST
Committed r179144: <http://trac.webkit.org/changeset/179144>