Skip to content

Commit 720b2a6

Browse files
NotYuShengclaude
andcommitted
fix: Add missing NetworkPolicy egress rules for pdf-extraction-service
- Add MinIO:9000 access for storing extracted images, JSON, and job results - Add Redis:6379 access for document file list management - Remove incorrect comment about "no internal service dependencies" - Update README service communication matrix to include pdf-extraction-service - Fix NetworkPolicy that would have blocked essential MinIO and Redis access - Based on source code analysis: service uses S3Utils and RedisDocumentFileList 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 1a988e8 commit 720b2a6

2 files changed

Lines changed: 17 additions & 4 deletions

File tree

README.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,11 +140,12 @@ 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+
| **pdf-extraction-service** | • pdf-processor-service | • minio:9000<br>• redis:6379<br>• DNS resolution<br>• HTTPS (docling model downloads) |
143144
| **metadata-service** | • pdf-processor-service | • chromadb:8000<br>• minio:9000<br>• redis:6379<br>• DNS resolution<br>• HTTPS (external vLLM) |
144145
| **cleaner** | *No ingress (background service)* | • minio:9000<br>• chromadb:8000<br>• redis:6379<br>• DNS resolution |
145146
| **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* |
147+
| **redis** | • pdf-processor-service<br>• pdf-extraction-service<br>• metadata-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
148+
| **minio** | • pdf-processor-service<br>• pdf-extraction-service<br>• metadata-service<br>• cleaner | • DNS resolution<br>*No outbound calls* |
148149

149150
#### Network Policy Configuration
150151

helm/pdf-extraction-service/values.yaml

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,20 @@ networkPolicy:
165165
allowHTTPS: true
166166
# Allow HTTP for internal services
167167
allowHTTP: false
168-
# ZERO-TRUST: pdf-extraction-service only needs external model access
169-
allowedTargets: [] # No internal service dependencies
168+
# ZERO-TRUST: pdf-extraction-service can only call specific services
169+
allowedTargets:
170+
# MinIO for storing extracted images, JSON, and job results
171+
- podSelector:
172+
matchLabels:
173+
app.kubernetes.io/name: minio
174+
ports:
175+
- 9000
176+
# Redis for document file list management
177+
- podSelector:
178+
matchLabels:
179+
app.kubernetes.io/name: redis
180+
ports:
181+
- 6379
170182
# Custom egress rules (empty by default)
171183
customRules: []
172184

0 commit comments

Comments
 (0)