1636907 - Switch geckodriver downloads to use taskcluster index

Status: RESOLVED FIXED

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

Comment 1

[James Graham [:jgraham]]

Filed https://github.com/web-platform-tests/wpt/pull/23502 whimboo: can you review that one?

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

(In reply to James Graham [:jgraham] from comment #1)

Filed https://github.com/web-platform-tests/wpt/pull/23502 whimboo: can you review that one?

I would better ask Chris or someone else from the TC team to review that. I haven't worked with Taskcluster indexes, nor downloading the artifacts yet.

I wonder if it would make sense to use the https://pypi.org/project/taskcluster/ Python package here.

Comment 3

[Chris AtLee [:catlee]]

IMO, there's no benefit to using the taskcluster python package for simple things like this. It's a simple HTTP request to fetch the artifact.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

The patch has been merged upstream. What else needs to be done (maybe a downstream sync), or can we mark the bug as fixed now?

Comment 5

[James Graham [:jgraham]]

Let's call this one fixed; we don't use this code in m-c so there's no need to wait for a sync.