-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Setup checkout cache for code review testing hook #32
Conversation
As far as I know, this will not happen automatically, and the role certainly doesn't have it already. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requesting changes for the grants.yml part.
I added the scope request in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'll need this in https://github.com/mozilla-releng/fxci-config/blob/main/grants.yml#L2583-L2588 as well (adding it to hooks.yml
doesn't change any of the actual grants - it just puts that scope into the hook task itself).
Thank you, I added it there too |
Thanks for the merge & apply @bhearsum Unfortunately, Taskcluster still complains about missing scopes on that task
|
Seems you're using generic-worker scopes in a docker-worker task, that won't work. |
Thanks for the quick reply ! I though that worker had moved towards generic-worker already. I'l make another PR to simply request docker-worker cache |
Refs mozilla/code-review#2193
Needed for mozilla/code-review#2235
The goal here is to use Taskcluster cache on the firefox-ci code-review testing hook so that mercurial checkouts can be shared amongst tasks.
I'm not sure if I need to declare that scope in
grants.yml
for the hook or if it's assigned automatically.