Closed
Bug 1218784
Opened 10 years ago
Closed 10 years ago
Reduce scopes assigned to worker types
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1220738
People
(Reporter: pmoore, Assigned: dustin)
References
Details
For good measure, we should probably also audit the scopes assigned to the various worker types. Of course they are already capped at the scopes that the client possesses, but all the same would be nice to keep them at a minimum.
Comment 1•10 years ago
|
||
Also, part of the discussion is the work that Jonas might do related to assigning temporary credentials when claiming a task so that the worker would only need a very limited set of scopes for polling the queue task URLs and a couple of other small things.
| Assignee | ||
Comment 2•10 years ago
|
||
Bug 1217088 and the lead-up partially addresses this -- worker types are defined by roles, specifically
https://tools.taskcluster.net/auth/roles/#worker-type:aws-provisioner-v1%252f*
Then bug 1134342 addressed the part where the task scopes must be satisfied by the workerType's scopes.
So, the `worker-type:aws-provisioner-v1/*` role can now be dialed back to include only the scopes required by the worker itself. I think that is just queue operations (polling, claiming, reclaiming), and that everything else including putting artifacts and extending task-graphs is now handled with the temp creds for the task.
Greg, does this match your expectation?
Flags: needinfo?(garndt)
| Assignee | ||
Updated•10 years ago
|
Assignee: nobody → dustin
Comment 3•10 years ago
|
||
That's my impression as well. We should be able to do that once 1220738 is complete. Also, our existing tasks need to be audited to make sure they have the appropriate scopes for creating/retreiving artifacts. I think most tasks neglect to have those scopes and rely on the bug that the worker uses it's own creds that has scopes.
Flags: needinfo?(garndt)
| Assignee | ||
Comment 4•10 years ago
|
||
Oh, I forgot about that bug. I'll just dupe this one over
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Updated•7 years ago
|
Component: Authentication → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•