Action Failed

[AceCoderLaura]

I added this action to my workflow but it seems to have failed. Here's the log from the Actions run:

Run android-actions/setup-android@v2
  env:
    BUILD_CONFIGURATION: Release
    SOLUTION_FILE_PATH: src\Adapt.Presentation.XamarinForms.sln
C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -NoLogo -Sta -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command "$ErrorActionPreference = 'Stop' ; try { Add-Type -AssemblyName System.IO.Compression.FileSystem } catch { } ; [System.IO.Compression.ZipFile]::ExtractToDirectory('C:\actions-runner\_work\_temp\1a1339f9-26c7-4f5f-b738-0c7d74ff2dc8', 'C:\actions-runner\_work\_temp\903f36fd-72ff-4212-8fd9-291663a9a96e')"
(node:7968) UnhandledPromiseRejectionWarning: Error: Unable to locate executable file: C:\actions-runner\_work\_temp\903f36fd-72ff-4212-8fd9-291663a9a96e\tools\bin\sdkmanager.bat. Please verify either the file path exists or the file can be found within a directory specified by the PATH environment variable. Also verify the file has a valid extension for an executable file.
    at Object.<anonymous> (C:\actions-runner\_work\_actions\android-actions\setup-android\v2\dist\index.js:1:2585)
    at Generator.next (<anonymous>)
    at fulfilled (C:\actions-runner\_work\_actions\android-actions\setup-android\v2\dist\index.js:1:601)
(node:7968) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:7968) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

[ViliusSutkus89]

@daveol , I'm 80% sure this error wouldn't happen with updated version. Any chance to tag a release?

@AceCoderLaura , in the meantime, you can use my fork or fork and tag it yourself. No promises to keep my fork up forever though.

- name: Setup Android SDK
  uses: ViliusSutkus89/setup-android@v2.0.3-rc1

[ViliusSutkus89]

@AceCoderLaura , You should be able to use android-actions/setup-android@v2 now, re-run the failing workflow to see if it works now.

@daveol , Welcome to the project! 😄

[AceCoderLaura]

@ViliusSutkus89 Cheers, I'll test it now and let you know how it goes.

[AceCoderLaura]

It seems to run correctly now. Although the ANDROID_SDK_ROOT environment variable doesn't appear to be set. That's probably a separate issue though.

[AceCoderLaura]

Oh, it might be due to this new error that must've started appearing after the first successful run.

Run android-actions/setup-android@v2
C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -NoLogo -Sta -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command "$ErrorActionPreference = 'Stop' ; try { Add-Type -AssemblyName System.IO.Compression.FileSystem } catch { } ; [System.IO.Compression.ZipFile]::ExtractToDirectory('C:\actions-runner\_work\_temp\16ff75f4-df58-4d90-9c41-4480f60c45c7', 'C:\actions-runner\_work\_temp\351f9f94-7883-436f-a3b7-7f5cbe75c384')"
(node:7028) UnhandledPromiseRejectionWarning: Error: dest already exists.
    at doRename (C:\actions-runner\_work\_actions\android-actions\setup-android\v2\dist\index.js:1:17873)
    at Object.moveSync (C:\actions-runner\_work\_actions\android-actions\setup-android\v2\dist\index.js:1:17775)
    at C:\actions-runner\_work\_actions\android-actions\setup-android\v2\dist\index.js:1:27200
    at Generator.next (<anonymous>)
    at fulfilled (C:\actions-runner\_work\_actions\android-actions\setup-android\v2\dist\index.js:1:25035)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(node:7028) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:7028) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I'm using self-hosted runners so they're not new machine images each time. That's probably why the destination already exists.

[ViliusSutkus89]

I'm using self-hosted runners so they're not new machine images each time. That's probably why the destination already exists.

This is one of the reasons. I'm more concerned about the realization that the path C:\actions-runner\_work\_temp\351f9f94-7883-436f-a3b7-7f5cbe75c384 is the same on each run. We get it from tool cache's extractZip, which internally does path.join(_getTempDirectory(), uuidV4()). I'm having a hard time believing that uuidV4() gives the same value on each run.

ANDROID_SDK_ROOT is not being set because we errored out 😞

Of course, this could be fixed by properly cleaning up after the extraction, which I haven't done yet 😅 . Could be a one liner, let me see what's possible.

[ViliusSutkus89]

Try manually removing C:\actions-runner\_work\_temp\351f9f94-7883-436f-a3b7-7f5cbe75c384 and running the workflow again. Did a release which should remove the folder after extraction.

[AceCoderLaura]

Alrighty, I'll give it a try.

[AceCoderLaura]

The C:\actions-runner\_work\_temp\ folder was empty. The same error I posted two hours ago occurs when I run the workflow.

[ViliusSutkus89]

No idea, to be honest. I've just noticed that tool-cache creates dest folder before extracting, so whatever else folder would I give - it would execute the same code path and throw the same exception.

  dest = await _createExtractFolder(dest)

  if (IS_WINDOWS) {
    await extractZipWin(file, dest)
  } ...

Any chance your runner is borked somehow?

[AceCoderLaura]

Hmm, I could try reinstalling the runner. I'll do that and report back.

[AceCoderLaura]

I reinstalled the runner but the error persists.