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

Nightly tests failing #855

Closed
stevedlawrence opened this issue Sep 22, 2023 · 8 comments · Fixed by #856, #894 or #940
Closed

Nightly tests failing #855

stevedlawrence opened this issue Sep 22, 2023 · 8 comments · Fixed by #856, #894 or #940
Assignees
Labels
bug Something isn't working build Issues related to building of the code CI Issues related to CI workflows testing Updates to test suite
Milestone

Comments

@stevedlawrence
Copy link
Member

It looks like the nightly CI tests have been failing for months:

https://github.com/apache/daffodil-vscode/actions/workflows/nightly.yml

I'm not sure what value the provide over PR and merge actions, but we should either get them working or remove if they don't provide value.

@scholarsmate scholarsmate linked a pull request Sep 22, 2023 that will close this issue
@scholarsmate
Copy link
Contributor

Thanks @stevedlawrence. The current CI tests against VS Code 1.67.2, the earliest version of VS Code supported for v1.4.0 of the extension. The nightly tests are designed to run against the latest stable and insider's builds of VS Code. We're going to want these to pass in general, especially for the latest stable release. VS Code 1.67.2 uses node 16 internally and the latest stable uses node 18 internally. I opened a PR to shift from testing under node 14 and 16, to 16 and 18. That may help fix some of the tests.

@scholarsmate scholarsmate added bug Something isn't working testing Updates to test suite build Issues related to building of the code CI Issues related to CI workflows labels Sep 22, 2023
@scholarsmate scholarsmate moved this to In Progress in daffodil-vscode v1.4.0 Sep 22, 2023
@scholarsmate scholarsmate added this to the 1.4.0 milestone Sep 22, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in daffodil-vscode v1.4.0 Sep 27, 2023
@arosien
Copy link
Contributor

arosien commented Sep 27, 2023

For the Windows failures, the message is

Error: EBUSY: resource busy or locked, rmdir

A likely cause (from SO) is

Another way you can trigger this error is if you are running an npm run [command] while cd'd into the directory that its altering. This happened to me where I closed every program and it didn't work, but realized I was inside the dist folder which needed to be removed and rebuilt. This triggered the Error: EBUSY: resource busy or locked, rmdir error in PowerShell and can be reproduced every time.

We are likely cleaning up some directory during the tests while that directory is "being used".

@arosien
Copy link
Contributor

arosien commented Sep 28, 2023

#856 didn't fix the nightly builds completely, the Windows builds are still failing due to the above. Should this issue be reopened, or a new one filed? @scholarsmate @stevedlawrence

@stevedlawrence
Copy link
Member Author

I'd vote for reopening this one if #856 didn't completely fix it. Then we have a full record of what PRs were needed to fix the issue.

@arosien
Copy link
Contributor

arosien commented Sep 28, 2023

#856 didn't fix the nightly builds completely, the Windows builds are still failing due to the above.

@arosien arosien reopened this Sep 28, 2023
@arosien arosien moved this from Done to Todo in daffodil-vscode v1.4.0 Sep 28, 2023
shanedell added a commit to shanedell/daffodil-vscode that referenced this issue Nov 29, 2023
- Add matrix to test different Java versions to main CI and nightly CI.
- Cleaned up the CI workflow file a bit.
  - The steps were too compacted, believe new layout makes it easier to read and follow.
- Add java_version to job names.
- Remove sbt compile from windows jobs, seemed to cause windows issues with jaxb.
- Believe the nightly CI maybe fixed with this update.
- Possibly fixes apache#855 but will wait to see if a nightly build passes before closing.

Closes apache#772
shanedell added a commit that referenced this issue Nov 29, 2023
- Add matrix to test different Java versions to main CI and nightly CI.
- Cleaned up the CI workflow file a bit.
  - The steps were too compacted, believe new layout makes it easier to read and follow.
- Add java_version to job names.
- Remove sbt compile from windows jobs, seemed to cause windows issues with jaxb.
- Believe the nightly CI maybe fixed with this update.
- Possibly fixes #855 but will wait to see if a nightly build passes before closing.

Closes #772
@github-project-automation github-project-automation bot moved this from Todo to Done in daffodil-vscode v1.4.0 Nov 29, 2023
@shanedell
Copy link
Contributor

shanedell commented Nov 29, 2023

Between #901 and #894 these nightly tests might have been fixed. Didn't mean for this ticket to get closed but going to wait till tomorrow to see if the nightly CI runs any better, if it doesn't I will reopen this issue

scholarsmate pushed a commit to ctc-oss/daffodil-vscode that referenced this issue Dec 6, 2023
- Add matrix to test different Java versions to main CI and nightly CI.
- Cleaned up the CI workflow file a bit.
  - The steps were too compacted, believe new layout makes it easier to read and follow.
- Add java_version to job names.
- Remove sbt compile from windows jobs, seemed to cause windows issues with jaxb.
- Believe the nightly CI maybe fixed with this update.
- Possibly fixes apache#855 but will wait to see if a nightly build passes before closing.

Closes apache#772
@arosien
Copy link
Contributor

arosien commented Jan 17, 2024

No change, nightly tests still failing. Seems to be only failing windows and mac when downloading VS Code for the integration tests.

@arosien arosien reopened this Jan 17, 2024
@arosien
Copy link
Contributor

arosien commented Jan 18, 2024

looks like the path inside the VSCode artifact changed to something the test code doesn't expect:

Downloaded VS Code into /Users/arosien/nteligen/daffodil-vscode/.vscode-test/vscode-darwin-1.67.2
Test error: Error: spawn /Users/arosien/nteligen/daffodil-vscode/.vscode-test/vscode-darwin-1.67.2/Visual Studio Code.app/Contents/MacOS/Electron ENOENT
Exit code:   -2
Failed to run tests: Failed
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
➜  daffodil-vscode git:(Shanedell/add-root-element-settings) ls /Users/arosien/nteligen/daffodil-vscode/.vscode-test/vscode-darwin-1.67.2
Contents
➜  daffodil-vscode git:(Shanedell/add-root-element-settings) ls /Users/arosien/nteligen/daffodil-vscode/.vscode-test/vscode-darwin-1.67.2/Contents/
CodeResources    Frameworks/      Info.plist       MacOS/           PkgInfo          Resources/       _CodeSignature/

arosien added a commit to arosien/daffodil-vscode that referenced this issue Jan 18, 2024
arosien added a commit that referenced this issue Jan 18, 2024
stricklandrbls pushed a commit to ctc-oss/daffodil-vscode that referenced this issue Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working build Issues related to building of the code CI Issues related to CI workflows testing Updates to test suite
Projects
No open projects
Status: Done
4 participants