Skip to content

Commit fe29ec0

Browse files
committed
fix: add e2e test CI fixtures and scope ct install to target charts
Add missing CI files for the lago chart e2e test: - postgres.yml and redis.yml manifests for kind cluster infrastructure - e2e-values.yaml with required config for chart-testing install Scope ct install to specific charts in Taskfile, chart-testing-lago, and chart-testing-lago-mcp-server workflows to prevent sub-charts from being installed independently without their infrastructure. Expand chart-testing-lago.yml path triggers to include sub-chart dependencies (lago-config, lago-rails, lago-front, etc.) so the e2e test runs when any dependency changes. Fix redis key names in lago values.yaml (url -> uri) to match what lago-config templates actually reference.
1 parent 095cb34 commit fe29ec0

7 files changed

Lines changed: 102 additions & 6 deletions

File tree

.github/workflows/chart-testing-lago-mcp-server.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,4 +44,4 @@ jobs:
4444

4545
- name: Run chart-testing (install)
4646
if: steps.list-changed.outputs.changed == 'true'
47-
run: ct install --config ct.yaml --namespace default
47+
run: ct install --config ct.yaml --namespace default --charts charts/lago-mcp-server

.github/workflows/chart-testing-lago.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,11 @@ on:
22
pull_request:
33
paths:
44
- "charts/lago/**"
5+
- "charts/lago-config/**"
6+
- "charts/lago-rails/**"
7+
- "charts/lago-front/**"
8+
- "charts/lago-events-processor-worker/**"
9+
- "charts/lago-pdf/**"
510

611
name: Lint and Test Lago Chart
712

@@ -52,4 +57,4 @@ jobs:
5257

5358
- name: Run chart-testing (install)
5459
if: steps.list-changed.outputs.changed == 'true'
55-
run: ct install --config ct.yaml --namespace default
60+
run: ct install --config ct.yaml --namespace default --charts charts/lago

Taskfile.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ tasks:
149149
- task: kind:up
150150
- kubectl apply -f charts/lago/ci/postgres.yml
151151
- kubectl apply -f charts/lago/ci/redis.yml
152-
- ct install --config ct.yaml --namespace default
152+
- ct install --config ct.yaml --namespace default --charts charts/lago
153153

154154
render:
155155
cmd: >

charts/lago/ci/e2e-values.yaml

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
global:
2+
urls:
3+
api: "http://lago-api.default.svc.cluster.local:3000"
4+
front: "http://lago-front.default.svc.cluster.local:80"
5+
6+
database:
7+
uri: "postgres://lago:lago@lago-postgres.default.svc.cluster.local:5432/lago"
8+
9+
encryption:
10+
key: "911219513a368a9f5cb38b326e739a70"
11+
salt: "cabaabf712d9f7b2ce0ae1b12fc01eec"
12+
13+
signing:
14+
rsa: LS0tLS1CRUdJTiBQUklWQVRFIEtFWS0tLS0tCk1JSUJWZ0lCQURBTkJna3Foa2lHOXcwQkFRRUZBQVNDQVVBd2dnRThBZ0VBQWtFQXpmMnl4bUFFTlhUNXcxKy8KRDI0ckEzcFZTa01NSlExOWpvRmNHQ1pnOFMwQWF3OXcrb2twcEM5NVZBZ0huWnh3bmpSNUVTeUpLK2VMeWJJcwpEZ3JuTVFJREFRQUJBa0VBdFFaeEo3aVZ1a3pVZWdnWXBmelJQRHN6M1JGVENQVDBGRnBHanNxYmd4WE4zTmVHClE2Y2ttZTV3NWhlUGN4Y3Y5NXJ0d1BsL3VOTXdqYWZXUUIwbWdRSWhBUHdEOS8wK1RjUjhHWVN4V1d5cWNjMVgKY3VsNzVvcWNyTFlNNEdiVmJoblpBaUVBMFQ5eUhVa1Z1bFdDRWVGdTV5UStrR1pZdFZhc3Rvenc2enE1WUkyWAp5UmtDSVFEMTZiRjlMZVNZekRmaVlXUlBRdXFWWXJYQTI2UVVzZGxlTTZaczhlcmowUUlnUVB5VmNtYjZhbzZZCkFQbWt1Z0RWdFg4ejJjMzdnZzZ2M0Q2MnlEZnhoakVDSVFEd1o4ZXVxQ1VuN01Ia3Q2YVdpSStidWRFQ05INTgKanNYTXBKWEdEdVMvU3c9PQotLS0tLUVORCBQUklWQVRFIEtFWS0tLS0tCg==
15+
hmac: SOswQPXojXToX/l7R/FXUw==
16+
17+
redis:
18+
uri: "redis://lago-redis.default.svc.cluster.local:6379"
19+
20+
redisCache:
21+
uri: "redis://lago-redis.default.svc.cluster.local:6379"
22+
23+
redisStore:
24+
uri: "redis://lago-redis.default.svc.cluster.local:6379"

charts/lago/ci/postgres.yml

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
apiVersion: v1
2+
kind: Service
3+
metadata:
4+
name: lago-postgres
5+
spec:
6+
selector:
7+
app: lago-postgres
8+
ports:
9+
- port: 5432
10+
targetPort: 5432
11+
---
12+
apiVersion: apps/v1
13+
kind: Deployment
14+
metadata:
15+
name: lago-postgres
16+
spec:
17+
replicas: 1
18+
selector:
19+
matchLabels:
20+
app: lago-postgres
21+
template:
22+
metadata:
23+
labels:
24+
app: lago-postgres
25+
spec:
26+
containers:
27+
- name: postgres
28+
image: getlago/postgres-partman:15.0-alpine
29+
ports:
30+
- containerPort: 5432
31+
env:
32+
- name: POSTGRES_USER
33+
value: "lago"
34+
- name: POSTGRES_PASSWORD
35+
value: "lago"
36+
- name: POSTGRES_DB
37+
value: "lago"

charts/lago/ci/redis.yml

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
apiVersion: v1
2+
kind: Service
3+
metadata:
4+
name: lago-redis
5+
spec:
6+
selector:
7+
app: lago-redis
8+
ports:
9+
- port: 6379
10+
targetPort: 6379
11+
---
12+
apiVersion: apps/v1
13+
kind: Deployment
14+
metadata:
15+
name: lago-redis
16+
spec:
17+
replicas: 1
18+
selector:
19+
matchLabels:
20+
app: lago-redis
21+
template:
22+
metadata:
23+
labels:
24+
app: lago-redis
25+
spec:
26+
containers:
27+
- name: redis
28+
image: redis:7-alpine
29+
ports:
30+
- containerPort: 6379

charts/lago/values.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,15 +39,15 @@ global:
3939
rsa: "" # LAGO_RSA_PRIVATE_KEY used by webhooks RS256
4040

4141
redis:
42-
url: ""
42+
uri: ""
4343
password: ""
4444

4545
redisCache:
46-
url: ""
46+
uri: ""
4747
password: ""
4848

4949
redisStore:
50-
url: ""
50+
uri: ""
5151
password: ""
5252

5353
sentryDsn: ""

0 commit comments

Comments
 (0)