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

Removing manual installation build tools 34.0.0 #1740

Merged
merged 1 commit into from
Jan 2, 2025

Conversation

cdsap
Copy link
Member

@cdsap cdsap commented Jan 2, 2025

Recent jobs on CI builds from AGP versions were failing with:

java.io.IOException: Unable to create debug keystore in /home/runner/.config/.android because it is not writable.

There is an initial workaround to fix this problem in PR #1720. The proposed solution is valid, but while trying to understand the reason for the failure, we observed that it is caused by an attempt to write to the Android folder during the build. This occurs in a folder created by a different user (root). The original folder is created when manually installing the SDK build tool version 34.0.0(during the gha setup):

      - name: List Android Packages
        run: sudo ${ANDROID_SDK_ROOT}/cmdline-tools/latest/bin/sdkmanager --list | sed -n '/Available Packages/q;p'
      - name: Accept license 34.0.0
        run: echo "y" | sudo ${ANDROID_SDK_ROOT}/cmdline-tools/latest/bin/sdkmanager "build-tools;34.0.0"

Given this situation, we need to understand what caused this change in the GHA infrastructure: The ubuntu-latest workflows have started rolling out the ubuntu-24.04 image since December 5th (scheduled to be completed by January 17th). One of the changes in the new version is the use of SDK build tools version 34.0.0.

This PR removes the manual installation of the SDK tools (performed as root), resolving the folder permission issue caused by the user mismatch.

@cdsap cdsap marked this pull request as ready for review January 2, 2025 17:56
@cdsap cdsap requested a review from a team January 2, 2025 17:57
@cdsap cdsap merged commit 5e720e6 into main Jan 2, 2025
21 checks passed
@cdsap cdsap deleted the removing_installation_build_tool_34 branch January 2, 2025 18:21
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.

2 participants