Hi @rueian, CI is failing on the new IDMP/IDMPAUTO and XCFGSET tests.

It seems the Redis instances used in CI are running a version that doesn’t support these features yet (Redis 8.6+).

In go-redis, there is a test-skip helper called SkipBeforeRedisVersion (also used helper by the tests in redis/go-redis#3693)

Before I proceed, is there already a related func to skip tests based on Redis version (similar to go-redis)? If not, would you like me to add one?

Before I proceed, is there already a related func to skip tests based on Redis version (similar to go-redis)? If not, would you like me to add one?

No, we don't have such function. Could we just replace redis/redis-stack:7.4.0-v0 with redis 8.6.0?

Could we just replace redis/redis-stack:7.4.0-v0 with redis 8.6.0?

I tried switching the compat services in docker-compose.yml(in repo root) to redis:8.6.0 locally, but it broke several module-dependent tests (TFUNCTION*, TS., FT.), so a straight replacement doesn’t seem viable.

I also saw the redis-stack note that Redis 8 bundles Stack modules, but in our CI/test setup redis:8.6.0 still isn’t a drop-in replacement for the current redis-stack image (those commands are missing or not working as expected).

Given this, I’ve been thinking about a couple of approaches:

1. add a separate redis:8+ service only for the new IDMP/XCFGSET tests,
2. add a skip helper (like go-redis).

In my opinion, since more Redis 8.6+ features (IDMP/IDMPAUTO, XCFGSET, etc.) are being added over time, it might be a good time to update the CI setup to support 8.6+ once 😅

add a separate redis:8+ service only for the new IDMP/XCFGSET tests,

Thanks for testing. Let's do this.

OK — I’ll work on the related CI/test setup changes and open a separate PR.

Hi @jinbum-kim, would you like to continue on this?

Hi @rueian Sorry for the delay. I’ve been tied up with other work.

I just opened a PR for the CI/test setup changes needed for Redis 8.6+. Could you please take a look when you have a chance? After that’s merged, I’ll follow up with the remaining changes for this work right away.

Hi @rueian Sorry for the delay. I’ve been tied up with other work.

I just opened a PR for the CI/test setup changes needed for Redis 8.6+. Could you please take a look when you have a chance? After that’s merged, I’ll follow up with the remaining changes for this work right away.

Hi @jinbum-kim, the CI setup PR has been merged!

Hi @rueian, all test are passed. PTAL! thank you🙌

Thanks a lot @jinbum-kim!