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

Fix GMD tests #867

Merged
merged 22 commits into from
Jun 27, 2024
Merged

Fix GMD tests #867

merged 22 commits into from
Jun 27, 2024

Conversation

ArnyminerZ
Copy link
Member

Purpose

Research and maybe fix #866

Short description

Checklist

  • The PR has a proper title, description and label.
  • I have self-reviewed the PR.
  • I have added documentation to complex functions and functions that can be used by other modules.
  • I have added reasonable tests or consciously decided to not add tests.

Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
@ArnyminerZ ArnyminerZ changed the title Run CI Fix GMD tests Jun 26, 2024
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
@ArnyminerZ
Copy link
Member Author

ArnyminerZ commented Jun 26, 2024

I've tried all the options I can think of to see whether it is a cache issue since the task started failing randomly (https://github.com/bitfireAT/davx5-ose/actions/runs/9660381810) however nothing seems to work.

@ArnyminerZ
Copy link
Member Author

Does run locally without any issues

@ArnyminerZ
Copy link
Member Author

ArnyminerZ commented Jun 26, 2024

May be related to AGP since last merge to main-ose was in https://github.com/bitfireAT/davx5-ose/actions/runs/9537918257 before the first failing check

Edit: downgrading doesn't help

Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
Signed-off-by: Arnau Mora Gras <arnyminerz@proton.me>
@ArnyminerZ
Copy link
Member Author

Upgraded AGP again and restored to the original run command but cleared all caches from Github Actions

@rfc2822
Copy link
Member

rfc2822 commented Jun 27, 2024

Runs with large runner; maybe really a resource problem? We could try to reduce required resources by reducing heap sizes etc, or just use the large runner (which doesn't have free minutes).

@rfc2822 rfc2822 self-assigned this Jun 27, 2024
@rfc2822 rfc2822 added the bug Something isn't working label Jun 27, 2024
@rfc2822
Copy link
Member

rfc2822 commented Jun 27, 2024

Seems this was caused by too little free space on the runner. I think because the gradle user cache become too big, because cleaning up the old entries is not very reliable. I tried to improve caching with a separate task and by excluding transformed JARs (>1 GB).

@rfc2822 rfc2822 marked this pull request as ready for review June 27, 2024 16:12
@rfc2822 rfc2822 merged commit 18b1e52 into main-ose Jun 27, 2024
5 checks passed
@rfc2822 rfc2822 deleted the ci-tests branch June 27, 2024 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] Gradle managed device tests fail
2 participants