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

Checkout fails on windows-latest when using @master #84

Closed
arlyon opened this issue Dec 3, 2019 · 8 comments · Fixed by #86
Closed

Checkout fails on windows-latest when using @master #84

arlyon opened this issue Dec 3, 2019 · 8 comments · Fixed by #86

Comments

@arlyon
Copy link

arlyon commented Dec 3, 2019

Builds are failing on windows. Logs attached.

2019-12-03T16:15:23.3178351Z ##[section]Starting: Request a runner to run this job
2019-12-03T16:15:24.2697462Z Requesting a hosted runner in current repository's account/organization with labels: 'windows-latest', require runner match: True
2019-12-03T16:15:24.3347571Z Labels matched hosted runners has been found, waiting for one of them get assigned for this job.
2019-12-03T16:15:24.3834888Z ##[section]Finishing: Request a runner to run this job
2019-12-03T16:15:31.7862022Z Current runner version: '2.161.0'
2019-12-03T16:15:31.7863222Z Prepare workflow directory
2019-12-03T16:15:31.8356961Z Prepare all required actions
2019-12-03T16:15:31.8391209Z Download action repository 'actions/checkout@master'
2019-12-03T16:15:32.7930286Z Download action repository 'actions/setup-python@master'
2019-12-03T16:15:33.7783103Z Download action repository 'actions/upload-artifact@master'
2019-12-03T16:15:34.1439691Z ##[group]Run actions/checkout@master
2019-12-03T16:15:34.1440081Z with:
2019-12-03T16:15:34.1440360Z   repository: arlyon/graphics
2019-12-03T16:15:34.1440572Z   token: ***
2019-12-03T16:15:34.1440704Z   clean: true
2019-12-03T16:15:34.1440818Z   fetch-depth: 1
2019-12-03T16:15:34.1440948Z   lfs: false
2019-12-03T16:15:34.1441076Z ##[endgroup]
2019-12-03T16:15:35.1466185Z Added matchers: 'checkout-git'. Problem matchers scan action output for known warning or error strings and report these inline.
2019-12-03T16:15:35.1467998Z Syncing repository: arlyon/graphics
2019-12-03T16:15:35.1468618Z Working directory is 'd:\a\graphics\graphics'
2019-12-03T16:15:35.1568360Z [command]"C:\Program Files\Git\bin\git.exe" version
2019-12-03T16:15:35.1768829Z git version 2.23.0.windows.1
2019-12-03T16:15:35.2022792Z Removed matchers: 'checkout-git'
2019-12-03T16:15:35.2024723Z ##[error]Command failed: rd /s /q "d:\a\graphics\graphics"
The process cannot access the file because it is being used by another process.

2019-12-03T16:15:35.2106752Z ##[error]Node run failed with exit code 1
2019-12-03T16:15:35.2147889Z Cleaning up orphan processes

Changing the version to @v1 seems to resolve the problem.

@gh-davidc
Copy link

Same thing here, also I've seen that even your Github Actions failed in your last commit v2-beta...master

I'm downgrading to v1, thanks @arlyon !

@runspired
Copy link

runspired commented Dec 3, 2019

Seems to be broken in all OS's as a result of #70

vlsi added a commit to apache/calcite that referenced this issue Dec 3, 2019
vlsi added a commit to apache/calcite-avatica that referenced this issue Dec 3, 2019
vlsi added a commit to apache/jmeter that referenced this issue Dec 3, 2019
@ericsciple
Copy link
Contributor

@arlyon thanks! fixed now. i'll push the tag forward

@arlyon
Copy link
Author

arlyon commented Dec 3, 2019

Thanks for the quick fix :)

@dlemstra
Copy link

dlemstra commented Dec 5, 2019

We are still experiencing this issue and it looks like it is related to the fact that we specify ref: refs/heads/master in our build (https://github.com/ImageMagick/ImageMagick/blob/53ba89cc416c2f1255197f85d1a17c85a0c06760/.github/workflows/daily.yml#L79). Should I open a new issue for this or is it related to this one? The build log can be found here: https://github.com/ImageMagick/ImageMagick/runs/334366719

@ericsciple
Copy link
Contributor

@dlemstra it looks like you specify remotes/origin/master as the ref. refs/heads/master or master should work.

It looks like a different bug than the one this issue was tracking. I think it should be a new issue.

@ericsciple
Copy link
Contributor

@dlemstra note, for a quick mitigation you can also pin back to actions/checkout@v1. I'm still interested in this bug though, it's a ref format i hadnt considered (v2 contains more narrow refspec during fetch in order to achieve perf improvements).

@ericsciple
Copy link
Contributor

@dlemstra fyi I opened #98 to track the issue you described

wangxlong pushed a commit to wangxlong/calcite that referenced this issue Feb 13, 2020
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 a pull request may close this issue.

5 participants