-
Notifications
You must be signed in to change notification settings - Fork 21
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
Comments
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. |
For the Windows failures, the message is
A likely cause (from SO) is
We are likely cleaning up some directory during the tests while that directory is "being used". |
#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 |
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. |
#856 didn't fix the nightly builds completely, the Windows builds are still failing due to the above. |
- 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
- 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
- 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
No change, nightly tests still failing. Seems to be only failing windows and mac when downloading VS Code for the integration tests. |
looks like the path inside the VSCode artifact changed to something the test code doesn't expect:
|
Fixes apache#855... perhaps.
Fixes apache#855... perhaps.
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.
The text was updated successfully, but these errors were encountered: