Bug 216889 – Notify igalia team about pre-existing test failures and build failures for JSC queues

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

[Aakash Jain]

Notify igalia team about pre-existing test failures and build failures for JSC queues.

[Aakash Jain]

Created attachment 409487 [details]
Patch

[Jonathan Bedard]

Comment on attachment 409487 [details]
Patch

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

> Tools/BuildSlaveSupport/ews-build/emails.json:10
> +      "[e-mail]"

Looks fine, but I would like someone from Igalia to verify they want this before we land it.

[Jonathan Bedard]

(In reply to Jonathan Bedard from comment #2)
> Comment on attachment 409487 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=409487&action=review
> 
> > Tools/BuildSlaveSupport/ews-build/emails.json:10
> > +      "[e-mail]"
> 
> Looks fine, but I would like someone from Igalia to verify they want this
> before we land it.

Looks like Igalia already verified in Slack that they want this.

[EWS]

Committed r267492: <https://trac.webkit.org/changeset/267492>

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

[Radar WebKit Bug Importer]

<rdar://problem/69450105>

[Guillaume Emont]

Comment on attachment 409487 [details]
Patch

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

>>> Tools/BuildSlaveSupport/ews-build/emails.json:10
>>> +      "[e-mail]"
>> 
>> Looks fine, but I would like someone from Igalia to verify they want this before we land it.
> 
> Looks like Igalia already verified in Slack that they want this.

Don't know if there's better naming to be had, but technically, that's not the whole jsc team, but only the ones working on embedded/32-bit things. We also have people doing feature work who are not included in that email. Maybe I'm being too pedantic though ;).

[Aakash Jain]

(In reply to Guillaume Emont from comment #6)
> Don't know if there's better naming to be had, but technically, that's not the whole jsc team, but only the ones working on embedded/32-bit things. We also have people doing feature work who are not included in that email. Maybe I'm being too pedantic though ;).
I guess specific name doesn't matter so much here. Feel free to change it to a better name though.

[Aakash Jain]

Restarted buildbot to pick up this change.