Skip to content

Add runner's architecture in bazel cache name #1285

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

Merged
merged 1 commit into from
Jun 18, 2025

Conversation

k311093
Copy link
Collaborator

@k311093 k311093 commented Jun 18, 2025

No description provided.

@k311093 k311093 marked this pull request as ready for review June 18, 2025 07:13
@k311093 k311093 requested a review from ikicha June 18, 2025 07:14
@k311093 k311093 enabled auto-merge June 18, 2025 07:28
@k311093 k311093 disabled auto-merge June 18, 2025 09:21
@k311093 k311093 force-pushed the add-architecture branch from 56294d2 to 5332c22 Compare June 18, 2025 09:35
@k311093 k311093 changed the title Add runner's architecture in bazel cache name Add runner's architecture, task name in bazel cache name Jun 18, 2025
@k311093
Copy link
Collaborator Author

k311093 commented Jun 18, 2025

Checked caching at #1289, it shows that cache for debian build does not speeds up cvd unit test, update cache for each task.

It might also need to find a way to docker-image-check task which runs bazel while building docker image.

@k311093 k311093 force-pushed the add-architecture branch from 5332c22 to 9861d51 Compare June 18, 2025 09:54
@k311093 k311093 changed the title Add runner's architecture, task name in bazel cache name Add runner's architecture in bazel cache name Jun 18, 2025
@k311093 k311093 force-pushed the add-architecture branch from 9861d51 to 375f461 Compare June 18, 2025 09:56
@k311093
Copy link
Collaborator Author

k311093 commented Jun 18, 2025

Looks like it's due to different directory name, just changed directory name

@k311093 k311093 added the kokoro:run Run e2e tests. label Jun 18, 2025
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label Jun 18, 2025
@k311093 k311093 force-pushed the add-architecture branch from 375f461 to 0fc3de9 Compare June 18, 2025 10:02
@k311093 k311093 force-pushed the add-architecture branch from 0fc3de9 to 32e75a6 Compare June 18, 2025 10:03
@k311093 k311093 added kokoro:run Run e2e tests. labels Jun 18, 2025
@k311093 k311093 enabled auto-merge June 18, 2025 10:28
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label Jun 18, 2025
@k311093 k311093 added this pull request to the merge queue Jun 18, 2025
Merged via the queue into google:main with commit 082e6c1 Jun 18, 2025
17 checks passed
@k311093 k311093 deleted the add-architecture branch June 19, 2025 06:03
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