valkey-io/valkey

Updated CI testing to improve catching tests early.

已关闭

#625 创建于 2024年6月11日

 (0 条评论) (0 个反应) (0 位负责人)C (1,138 个派生)batch import
help wanted

仓库指标

星标
 (25,779 个星标)
PR 合并指标
 (平均合并 17天 7小时) (30 天内合并 74 个 PR)

描述

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.

  1. Add in a second test suite that uses an allocator that is not jemalloc that has "no malloc size" enabled.
  2. 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.
  3. 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.

贡献者指南