-
Notifications
You must be signed in to change notification settings - Fork 187
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 GitHub Actions Build Fail on Mon, July 31st 2023 #4221
Comments
puzzling... |
Not a very useful error code either: https://learn.microsoft.com/en-us/visualstudio/msbuild/errors/msb8066?view=vs-2022 |
After managing to reproduce it on my machine I took a closer look into this. After running CTest, the generated Visual Studio project runs this command: setlocal
"C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe" -SC:/Users/teo/code/TileDB -BC:/Users/teo/code/TileDB/build/nightly/tiledb --check-stamp-file C:/Users/teo/code/TileDB/build/nightly/tiledb/test/CMakeFiles/generate.stamp
if %errorlevel% neq 0 goto :cmEnd
:cmEnd
endlocal & call :cmErrorLevel %errorlevel% & goto :cmDone
:cmErrorLevel
exit /b %1
:cmDone
if %errorlevel% neq 0 goto :VCEnd which seems to be the one that fails with an exit code of -1. I generated an MSBuild binary log of the failed test run. For anyone interested, it can be viewed with https://msbuildlog.com/. |
Maybe the way we run the tests with |
The last successful nightly build was on 366b15b and the first failing nightly build was on c2bbf59. Between them the biggest change was #4034, which still touched very little CMake stuff. |
I tried looking for a change as well in that time frame and couldn't find any smoking guns. I was wondering if there was a version change of something on the builders. Does that I wonder if we could add debug logging around that stamp file to see if/when it exists? Perhaps one of the filesystem tests does something weird when cleaning up data and accidentally deletes the CMake directory? |
I will try rerunning a nightly build on 366b15b.
Not on my machine. These MSBuild logs are from a local build. After many tries I could not make GHA upload them as artifacts.
The linked site has an online viewer. |
Huh! So it does. I must've just saw the macOS installation and clicked through without seeing the big green button for the online version. |
It suceeds. 😩 I will now try running a nightly build for each of the commits in the range. 🤞🏻 Something else I noticed is that around the time the builds started failing, GHA updated CMake from 3.26.4 to 3.27.0 but I don't think this is to blame, because locally I am still on 2.26.4. |
Found it. Nightlies started failing since #4087. Right before it it is #4034, whose nightlies succeed. |
I've confirmed on my fork that @teo-tsirpanis has seemed to have found that error in my SSL config PR. I ended up running a modified version with the ssl config tests disabled and that completes successfully. I'm running more tests to see why the tests appear to pass but apparently don't on the Windows nightly runners and not normal PR runners. One thing that I did realize is that the |
See run for more details:
https://github.com/TileDB-Inc/TileDB/actions/runs/5710056189
The text was updated successfully, but these errors were encountered: