Skip to content

Conversation

JasonAtClockwork
Copy link
Contributor

@JasonAtClockwork JasonAtClockwork commented Oct 7, 2025

Description of Changes

Removes the use of the Derived Data Cache during CI, will in

API and ABI breaking changes

None

Expected complexity level and risk

1 - Small change for CI

Testing

  • Re-ran tests on both Linux + Windows with the change

@JasonAtClockwork JasonAtClockwork marked this pull request as ready for review October 8, 2025 18:25
@bfops
Copy link
Collaborator

bfops commented Oct 14, 2025

Removes the use of the Derived Data Cache during CI, will in

?

@bfops
Copy link
Collaborator

bfops commented Oct 14, 2025

Can you add some explanation (maybe inline comments) about what impact these settings make?

@JasonAtClockwork
Copy link
Contributor Author

Can you add some explanation (maybe inline comments) about what impact these settings make?

Of course! I've added to the latest commit, let me know if the write up makes sense or if it needs more detail!

@bfops
Copy link
Collaborator

bfops commented Oct 14, 2025

It looks like we're still getting SIGSEGV sometimes, but if you feel confident that this will decrease the frequency, I'm all for it.

thread 'unreal_insert_vec' panicked at crates/testing/src/sdk.rs:142:9:
(running): Error running "\"/home/ue4/UnrealEngine/Engine/Binaries/Linux/UnrealEditor\" \"/__w/SpacetimeDB/SpacetimeDB/sdks/unreal/tests/TestClient/TestClient.uproject\" -NullRHI -Unattended -NoSound -nop4 -NoSplash -NoZen -ddc=InstalledNoZenLocalFallback -ExecCmds=\"Automation RunTests SpacetimeDB.TestClient.InsertVecTest; Quit\"": exited with non-zero exit status signal: 11 (SIGSEGV) (core dumped). Output:
setrlimit() failed with error 22 (Invalid argument)
- Max per-process value allowed is 0 (we wanted infinity).

(https://github.com/clockworklabs/SpacetimeDB/actions/runs/18387938924/job/52727623047)

@JasonAtClockwork
Copy link
Contributor Author

It looks like we're still getting SIGSEGV sometimes, but if you feel confident that this will decrease the frequency, I'm all for it.

thread 'unreal_insert_vec' panicked at crates/testing/src/sdk.rs:142:9:
(running): Error running "\"/home/ue4/UnrealEngine/Engine/Binaries/Linux/UnrealEditor\" \"/__w/SpacetimeDB/SpacetimeDB/sdks/unreal/tests/TestClient/TestClient.uproject\" -NullRHI -Unattended -NoSound -nop4 -NoSplash -NoZen -ddc=InstalledNoZenLocalFallback -ExecCmds=\"Automation RunTests SpacetimeDB.TestClient.InsertVecTest; Quit\"": exited with non-zero exit status signal: 11 (SIGSEGV) (core dumped). Output:
setrlimit() failed with error 22 (Invalid argument)
- Max per-process value allowed is 0 (we wanted infinity).

(https://github.com/clockworklabs/SpacetimeDB/actions/runs/18387938924/job/52727623047)

Oh that actually is a different error, setrlimit() is attempting to get some sort of resource increased but wasn't able. I'd guess memory would be a culprit here but I'm not 100% certain. I'll run some more tests on Linux and see if IO or Memory are higher than I expect.

@bfops
Copy link
Collaborator

bfops commented Oct 14, 2025

It looks like we're still getting SIGSEGV sometimes, but if you feel confident that this will decrease the frequency, I'm all for it.

thread 'unreal_insert_vec' panicked at crates/testing/src/sdk.rs:142:9:
(running): Error running "\"/home/ue4/UnrealEngine/Engine/Binaries/Linux/UnrealEditor\" \"/__w/SpacetimeDB/SpacetimeDB/sdks/unreal/tests/TestClient/TestClient.uproject\" -NullRHI -Unattended -NoSound -nop4 -NoSplash -NoZen -ddc=InstalledNoZenLocalFallback -ExecCmds=\"Automation RunTests SpacetimeDB.TestClient.InsertVecTest; Quit\"": exited with non-zero exit status signal: 11 (SIGSEGV) (core dumped). Output:
setrlimit() failed with error 22 (Invalid argument)
- Max per-process value allowed is 0 (we wanted infinity).

(https://github.com/clockworklabs/SpacetimeDB/actions/runs/18387938924/job/52727623047)

Oh that actually is a different error, setrlimit() is attempting to get some sort of resource increased but wasn't able. I'd guess memory would be a culprit here but I'm not 100% certain. I'll run some more tests on Linux and see if IO or Memory are higher than I expect.

Okay perfect, then it sounds like this PR is still at least an improvement. I'll approve as-is!

Copy link
Collaborator

@bfops bfops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It sounds like this addresses at least one source of errors, so let's merge it and keep iterating!

@JasonAtClockwork JasonAtClockwork added this pull request to the merge queue Oct 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 14, 2025
@bfops bfops added this pull request to the merge queue Oct 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 14, 2025
@bfops bfops added this pull request to the merge queue Oct 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 14, 2025
@bfops bfops added this pull request to the merge queue Oct 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 14, 2025
@bfops bfops added this pull request to the merge queue Oct 14, 2025
Merged via the queue into master with commit b7efd17 Oct 14, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants