ci: Run Undefined runner #9453
ci: Run Undefined runner #9453
Conversation
|
CI Vulkan-ValidationLayers build queued with queue ID 373295. |
|
CI Vulkan-ValidationLayers build # 19080 running. |
|
CI Vulkan-ValidationLayers build # 19080 passed. |
| @@ -87,27 +87,34 @@ jobs: | |||
| env: | |||
| VK_KHRONOS_PROFILES_PROFILE_FILE: ${{ github.workspace }}/tests/device_profiles/min_core.json | |||
|
|
|||
| # Make sure a debug version builds, but no need to run tests on it (since we do with release already) | |||
| linux-debug: | |||
There was a problem hiding this comment.
Does this remove debug build?
There was a problem hiding this comment.
ya, we have debug builds tested inside internal CI, we just recently demoted the Debug run to a Debug build and honestly no sure what it offers (since it is fully tested internally)
So replacing those CI minutes with an Undefined Behavior runner seems better use of the credits
There was a problem hiding this comment.
I don't remember details about replacing debug build, but if was about timing, then trading timing for debug build validity checks (c-runtime, stl) is a bad move in my opinion! (less related to current PR).
There was a problem hiding this comment.
discussed, will keep the debug build
Attempt to add a runner for Undefined Behavior for #9384