valkey-io/valkey
Updated CI testing to improve catching tests early.
Chiusa
#625 aperta il 11 giu 2024
help wanted
Metriche repository
- Star
- (25.779 stelle)
- Metriche merge PR
- (Merge medio 17g 7h) (74 PR mergiate in 30 g)
Descrizione
We have had a recent string of issues with regressions being introduced into the daily rest run, that weren't being caught in the daily. After reviewing the daily failures, me and some other contributors came up with the following recommendation.
- Add in a second test suite that uses an allocator that is not jemalloc that has "no malloc size" enabled.
- Add a new CI target tests with TLS, but skip all of the "slow" tests. Included here is that we should add a slow tag to the payload fuzzer as they take a significant amount of time with TLS.
- Add a build target that actually builds the legacy unit test, to make sure they compile correctly.
We also discussed maybe adding in the sentinel tests as part of the build, however it wasn't clear if they regularly caught issues.