Closed Bug 1595763 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 20211 - taskcluster: ensure data.metadata.owner is a valid e-mail on push event (v2)

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox72 fixed)

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 20211 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/20211
Details from upstream follow.

Carlos Alberto Lopez Perez <clopez@igalia.com> wrote:

taskcluster: ensure data.metadata.owner is a valid e-mail on push event (v2)

taskcluster: ensure data.metadata.owner is a valid e-mail on push event (v2)

  • When the push event comes from a github action, event.pusher.email
    seems to be invalid, which causes a fatal error on taskcluster.

  • Previous attempt to fix this (0317a96431) has not worked because
    event.pusher.email is not defined for github actions and its unknown
    if event.pusher.name is.

  • Try to fix it this time by defaulting the mail in all cases to
    ${event.sender.login}@users.noreply.github.com which according to
    the GitHub API documentation examples should be always defined
    (hopefully that will be also true for github actions).

Related issue: #20106

PR 20211 applied with additional changes from upstream: 8aeb44900921dce5401a91a2ede05dbbbb2d56f2, 689b968cd1f8848dc2c62928ffd1b20598be3537
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7747d951c8e3
[wpt PR 20211] - taskcluster: ensure data.metadata.owner is a valid e-mail on push event (v2), a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.