v4 cache attempt to write to non existent drive on windows about setup-go

[nixomose]

My quick workaround solution was to add a disk, mount drive d and make that directory that it wanted. :-)

When I get back to this in a day or two I will give that a try and let you know, thanks.

[dsame]

Hello @nixomose, i am going to close the issue because the problem does not exist anymore. Please feel free to reopen it or create a new one in case it appears again.

[nixomose]

in addition the readme.md says

The action will try to enable caching unless the cache input is explicitly set to false.

so I tried this:

      uses: actions/setup-go@v4
      with:
        go-version: ${{ env.GOVER }}
        cache: false      

and it still tries to write the cache to drive D:

[dusan-trickovic]

Hello, @nixomose! Thank you for reporting this, we will investigate it and see what can be done :)

[dsame]

Hello @nixomose, can you please try to set the environment variable AGENT_ISSELFHOSTED=1 as a quick workaround?

on:
...

env:
  AGENT_ISSELFHOSTED: 1

[dsame]

The problem expected to be solved with this PR merge https://github.com/actions/setup-go/pull/438/files#diff-39b2554fd18da165b59a6351b1aafff3714e2a80c1435f2de9706355b4d32351R6

[nixomose]

so I think that helped. but I've rebuilt the runners a different way, so likely your environment variable fixed it, but there's lots of other little changes I made that might have had some effect as well. either way, it's working.
thanks.

[Bykow]

Hey, I'm using actions/setup-go@v5 and I have the same issue.

I'm running a self hosted windows runner, and it has no drive D:
EPERM: operation not permitted, mkdir 'D:\actions-runner\_work\_tool\go\1.22.1\x64'

edit: Seems like the issue can't get reopened from a comment, how so ?

[nixomose]

I added this and it stopped using drive D:

env:
AGENT_ISSELFHOSTED: 1

[Bykow]

So this comment is not correct right ?

#432 (comment)

[nixomose]

If you're still having a problem, then I guess not.

[Bykow]

So adding the env variable did solve my issue. But the MR pointed seems to suggest that this enviromnent variable should not be necessary anymore.

[nixomose]

I do not know. But it works and that's what I needed.