Skip to content
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

Merged
merged 3 commits into from
Jul 18, 2024

Conversation

La0
Copy link
Contributor

@La0 La0 commented Jul 17, 2024

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.

@La0 La0 requested a review from a team as a code owner July 17, 2024 15:47
@bhearsum
Copy link
Contributor

I'm not sure if I need to declare that scope in grants.yml for the hook or if it's assigned automatically.

As far as I know, this will not happen automatically, and the role certainly doesn't have it already.

Copy link
Contributor

@bhearsum bhearsum left a 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.

@La0
Copy link
Contributor Author

La0 commented Jul 18, 2024

I added the scope request in hooks.yml relevant part

Copy link
Contributor

@bhearsum bhearsum left a 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).

@La0
Copy link
Contributor Author

La0 commented Jul 18, 2024

Thank you, I added it there too

@bhearsum bhearsum enabled auto-merge (squash) July 18, 2024 14:00
@bhearsum bhearsum merged commit 519ee59 into mozilla-releng:main Jul 18, 2024
10 checks passed
@La0
Copy link
Contributor Author

La0 commented Jul 19, 2024

Thanks for the merge & apply @bhearsum

Unfortunately, Taskcluster still complains about missing scopes on that task

[taskcluster:error] Docker configuration could not be created.  This may indicate an authentication error when validating scopes necessary for running the task. 
 Error: Insufficient scopes to attach cache volumes.
The task is missing the following scopes:

[object Object]

This requested devices requires the task scopes to satisfy the following scope expression:
[object Object]

@jcristau
Copy link
Contributor

Seems you're using generic-worker scopes in a docker-worker task, that won't work.

@La0
Copy link
Contributor Author

La0 commented Jul 19, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants