Skip to content

Commit 60e465a

Browse files
ndyakovndyakov
committed
wip
1 parent bdb19f5 commit 60e465a

15 files changed

+115
-93
lines changed

‎.github/actions/run-tests/action.yml‎

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,7 @@ runs:
2121
CLIENT_LIBS_TEST_IMAGE: "redislabs/client-libs-test:${{ inputs.redis-version }}"
2222
run: |
2323
set -e
24-
redis_major_version=$(echo "$REDIS_VERSION" | grep -oP '^\d+')
25-
if (( redis_major_version < 8 )); then
26-
echo "Using redis-stack for module tests"
27-
else
28-
echo "Using redis CE for module tests"
29-
fi
24+
redis_version_np=$(echo "$REDIS_VERSION" | grep -oP '^\d+.\d+')
3025
3126
# Mapping of redis version to redis testing containers
3227
declare -A redis_version_mapping=(
@@ -36,14 +31,13 @@ runs:
3631
)
3732
3833
if [[ -v redis_version_mapping[$REDIS_VERSION] ]]; then
39-
echo "REDIS_MAJOR_VERSION=${redis_major_version}" >> $GITHUB_ENV
34+
echo "REDIS_VERSION=${redis_version_np}" >> $GITHUB_ENV
4035
echo "REDIS_IMAGE=redis:${{ inputs.redis-version }}" >> $GITHUB_ENV
4136
echo "CLIENT_LIBS_TEST_IMAGE=redislabs/client-libs-test:${redis_version_mapping[$REDIS_VERSION]}" >> $GITHUB_ENV
4237
else
4338
echo "Version not found in the mapping."
4439
exit 1
4540
fi
46-
sleep 10 # time to settle
4741
shell: bash
4842
- name: Set up Docker Compose environment with redis ${{ inputs.redis-version }}
4943
run: docker compose --profile all up -d
@@ -58,5 +52,4 @@ runs:
5852
--ginkgo.skip-file="sentinel_test.go" \
5953
--ginkgo.skip-file="pubsub_test.go" \
6054
--ginkgo.skip-file="gears_commands_test.go" \
61-
--ginkgo.label-filter="!NonRedisEnterprise"
6255
shell: bash

‎.github/workflows/build.yml‎

Lines changed: 36 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ jobs:
1616
strategy:
1717
fail-fast: false
1818
matrix:
19-
go-version: [1.21.x, 1.22.x, 1.23.x]
19+
go-version: [1.22.x, 1.23.x]
2020

2121
steps:
2222
- name: Set up ${{ matrix.go-version }}
@@ -42,7 +42,12 @@ jobs:
4242
strategy:
4343
fail-fast: false
4444
matrix:
45-
go-version: [1.21.x, 1.23.x]
45+
redis-version:
46+
- "8.0-M03" # 8.0 milestone 4
47+
- "7.4.2" # should use redis stack 7.4
48+
go-version:
49+
- "1.22.x"
50+
- "1.23.x"
4651

4752
steps:
4853
- name: Set up ${{ matrix.go-version }}
@@ -53,10 +58,38 @@ jobs:
5358
- name: Checkout code
5459
uses: actions/checkout@v4
5560

61+
- name: Setup Test environment
62+
env:
63+
REDIS_VERSION: ${{ matrix.redis-version }}
64+
CLIENT_LIBS_TEST_IMAGE: "redislabs/client-libs-test:${{ matrix.redis-version }}"
65+
run: |
66+
set -e
67+
redis_version_np=$(echo "$REDIS_VERSION" | grep -oP '^\d+.\d+')
68+
69+
# Mapping of redis version to redis testing containers
70+
declare -A redis_version_mapping=(
71+
["8.0-M03"]="8.0-M04-pre"
72+
["7.4.2"]="rs-7.4.0-v2"
73+
)
74+
75+
if [[ -v redis_version_mapping[$REDIS_VERSION] ]]; then
76+
echo "REDIS_VERSION=${redis_version_np}" >> $GITHUB_ENV
77+
echo "REDIS_IMAGE=redis:${{ matrix.redis-version }}" >> $GITHUB_ENV
78+
echo "CLIENT_LIBS_TEST_IMAGE=redislabs/client-libs-test:${redis_version_mapping[$REDIS_VERSION]}" >> $GITHUB_ENV
79+
else
80+
echo "Version not found in the mapping."
81+
exit 1
82+
fi
83+
shell: bash
84+
- name: Set up Docker Compose environment with redis ${{ matrix.redis-version }}
85+
run: docker compose --profile all up -d
86+
shell: bash
5687
- name: Benchmark Tests
88+
env:
89+
RCE_DOCKER: "true"
90+
RE_CLUSTER: "false"
5791
run: make bench
5892

59-
6093
test-redis-ce:
6194
name: test-redis-ce
6295
runs-on: ubuntu-latest
@@ -72,7 +105,6 @@ jobs:
72105
- "1.23.x"
73106

74107
steps:
75-
76108
- name: Checkout code
77109
uses: actions/checkout@v4
78110

‎.github/workflows/doctests.yaml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
strategy:
2626
fail-fast: false
2727
matrix:
28-
go-version: [ "1.21", "1.22", "1.23" ]
28+
go-version: ["1.22", "1.23"]
2929

3030
steps:
3131
- name: Set up ${{ matrix.go-version }}
@@ -38,4 +38,4 @@ jobs:
3838

3939
- name: Test doc examples
4040
working-directory: ./doctests
41-
run: go test
41+
run: go test -v

‎.github/workflows/test-redis-enterprise.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ jobs:
4747
- name: Test
4848
env:
4949
RE_CLUSTER: true
50-
REDIS_MAJOR_VERSION: 7
50+
REDIS_VERSION: "7.4"
5151
run: |
5252
go test \
5353
--ginkgo.skip-file="ring_test.go" \

‎Makefile‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
GO_MOD_DIRS := $(shell find . -type f -name 'go.mod' -exec dirname {} \; | sort)
2-
export REDIS_MAJOR_VERSION := 7
2+
export REDIS_VERSION := "7.2"
33

44
redisstackdocker.start:
55
docker start go-redis-redis-stack || docker run -d --name go-redis-redis-stack -p 6379:6379 -e REDIS_ARGS="--enable-debug-command yes --enable-module-command yes" redis/redis-stack-server:latest

‎acl_commands_test.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -242,7 +242,7 @@ var _ = Describe("ACL permissions", Label("NonRedisEnterprise"), func() {
242242
})
243243

244244
It("set permissions for module commands", func() {
245-
SkipBeforeRedisMajor(8, "permissions for modules are supported for Redis Version >=8")
245+
SkipBeforeRedisVersion(8, "permissions for modules are supported for Redis Version >=8")
246246
Expect(client.FlushDB(ctx).Err()).NotTo(HaveOccurred())
247247
val, err := client.FTCreate(ctx, "txt", &redis.FTCreateOptions{}, &redis.FieldSchema{FieldName: "txt", FieldType: redis.SearchFieldTypeText}).Result()
248248
Expect(err).NotTo(HaveOccurred())
@@ -322,7 +322,7 @@ var _ = Describe("ACL permissions", Label("NonRedisEnterprise"), func() {
322322
})
323323

324324
It("set permissions for module categories", func() {
325-
SkipBeforeRedisMajor(8, "permissions for modules are supported for Redis Version >=8")
325+
SkipBeforeRedisVersion(8, "permissions for modules are supported for Redis Version >=8")
326326
Expect(client.FlushDB(ctx).Err()).NotTo(HaveOccurred())
327327
val, err := client.FTCreate(ctx, "txt", &redis.FTCreateOptions{}, &redis.FieldSchema{FieldName: "txt", FieldType: redis.SearchFieldTypeText}).Result()
328328
Expect(err).NotTo(HaveOccurred())
@@ -419,7 +419,7 @@ var _ = Describe("ACL Categories", func() {
419419
})
420420

421421
It("lists acl categories and subcategories with Modules", func() {
422-
SkipBeforeRedisMajor(8, "modules are included in acl for redis version >= 8")
422+
SkipBeforeRedisVersion(8, "modules are included in acl for redis version >= 8")
423423
aclTestCase := map[string]string{
424424
"search": "FT.CREATE",
425425
"bloom": "bf.add",

‎bench_test.go‎

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -291,11 +291,6 @@ func BenchmarkClusterPing(b *testing.B) {
291291

292292
ctx := context.Background()
293293
cluster := newClusterScenario()
294-
if err := startCluster(ctx, cluster); err != nil {
295-
b.Fatal(err)
296-
}
297-
defer cluster.Close()
298-
299294
client := cluster.newClusterClient(ctx, redisClusterOptions())
300295
defer client.Close()
301296

@@ -318,10 +313,6 @@ func BenchmarkClusterDoInt(b *testing.B) {
318313

319314
ctx := context.Background()
320315
cluster := newClusterScenario()
321-
if err := startCluster(ctx, cluster); err != nil {
322-
b.Fatal(err)
323-
}
324-
defer cluster.Close()
325316

326317
client := cluster.newClusterClient(ctx, redisClusterOptions())
327318
defer client.Close()
@@ -344,10 +335,6 @@ func BenchmarkClusterSetString(b *testing.B) {
344335

345336
ctx := context.Background()
346337
cluster := newClusterScenario()
347-
if err := startCluster(ctx, cluster); err != nil {
348-
b.Fatal(err)
349-
}
350-
defer cluster.Close()
351338

352339
client := cluster.newClusterClient(ctx, redisClusterOptions())
353340
defer client.Close()
@@ -372,21 +359,6 @@ func BenchmarkExecRingSetAddrsCmd(b *testing.B) {
372359
ringShard2Name = "ringShardTwo"
373360
)
374361

375-
for _, port := range []string{ringShard1Port, ringShard2Port} {
376-
if _, err := startRedis(port); err != nil {
377-
b.Fatal(err)
378-
}
379-
}
380-
381-
b.Cleanup(func() {
382-
for _, p := range processes {
383-
if err := p.Close(); err != nil {
384-
b.Errorf("Failed to stop redis process: %v", err)
385-
}
386-
}
387-
processes = nil
388-
})
389-
390362
ring := redis.NewRing(&redis.RingOptions{
391363
Addrs: map[string]string{
392364
"ringShardOne": ":" + ringShard1Port,

0 commit comments

Comments
 (0)