Skip to content

Commit 1a988e8

Browse files
NotYuShengclaude
andcommitted
fix: Add missing NetworkPolicy egress rules for metadata-service
- Add ChromaDB:8000 access for retrieving document chunks - Add MinIO:9000 access for storing/loading job results - Add Redis:6379 access for document file list management - Update README service communication matrix to include metadata-service - Fix incomplete NetworkPolicy that would have blocked service communication - Resolves issue identified by code analysis of actual service dependencies 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 1b4c011 commit 1a988e8

2 files changed

Lines changed: 23 additions & 4 deletions

File tree

README.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -140,10 +140,11 @@ OmniPDF implements comprehensive zero-trust network policies with explicit servi
140140
| **pdf-processor-service** | • nginx<br>• frontend | • chat-service:8000<br>• pdf-extraction-service:8000<br>• embedder-service:8000<br>• docling-translation-service:8000<br>• pdf-renderer-service:8000<br>• image-captioner-service:8000<br>• metadata-service:8000<br>• minio:9000<br>• redis:6379<br>• DNS resolution |
141141
| **chat-service** | • pdf-processor-service | • chromadb:8000<br>• DNS resolution<br>• HTTPS (external vLLM) |
142142
| **embedder-service** | • pdf-processor-service | • chromadb:8000<br>• DNS resolution<br>• HTTPS (model downloads) |
143+
| **metadata-service** | • pdf-processor-service | • chromadb:8000<br>• minio:9000<br>• redis:6379<br>• DNS resolution<br>• HTTPS (external vLLM) |
143144
| **cleaner** | *No ingress (background service)* | • minio:9000<br>• chromadb:8000<br>• redis:6379<br>• DNS resolution |
144-
| **chromadb** | • chat-service<br>• embedder-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
145-
| **redis** | • pdf-processor-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
146-
| **minio** | • pdf-processor-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
145+
| **chromadb** | • chat-service<br>• embedder-service<br>• metadata-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
146+
| **redis** | • pdf-processor-service<br>• metadata-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
147+
| **minio** | • pdf-processor-service<br>• metadata-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
147148

148149
#### Network Policy Configuration
149150

helm/metadata-service/values.yaml

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,25 @@ networkPolicy:
206206
allowDNS: true
207207
allowHTTPS: true # For external vLLM in prestaging
208208
# ZERO-TRUST: metadata-service can only call specific services
209-
allowedTargets: [] # Configured via customRules for external vLLM
209+
allowedTargets:
210+
# ChromaDB for retrieving document chunks
211+
- podSelector:
212+
matchLabels:
213+
app.kubernetes.io/name: chromadb
214+
ports:
215+
- 8000
216+
# MinIO for storing/loading job results
217+
- podSelector:
218+
matchLabels:
219+
app.kubernetes.io/name: minio
220+
ports:
221+
- 9000
222+
# Redis for document file list management
223+
- podSelector:
224+
matchLabels:
225+
app.kubernetes.io/name: redis
226+
ports:
227+
- 6379
210228
# Custom egress rules for special cases (external vLLM, etc.)
211229
customRules: []
212230

0 commit comments

Comments
 (0)