Commit 047d2cb
committed
fix: Correct frontend ports and prevent internet pulls in offline mode
This commit fixes critical deployment issues across production and offline configurations:
## Production (docker-compose.prod.yml)
1. **Frontend Port Fix**:
- Change port mapping from 5173:80 to 5173:8080
- Update healthcheck to use port 8080 instead of 80
- Matches actual nginx configuration listening on port 8080
- **Fixes**: Frontend unhealthy, app not loading in browser
2. **Environment Variables**:
- Add MODELS_DIRECTORY to backend and celery-worker
- Ensures consistency with main docker-compose.yml
## Offline (docker-compose.offline.yml)
1. **Prevent Internet Access**:
- Add `pull_policy: never` to ALL services (7 services total)
- postgres, minio, redis, opensearch, backend, celery-worker, frontend, flower
- **Critical**: Ensures no internet pulls in air-gapped environments
- Images must be pre-loaded with `docker load`
2. **Frontend Port Fix**:
- Change port mapping from 80:80 to 80:8080
- Add healthcheck with correct port 8080
- Matches nginx configuration
3. **Model Cache**:
- Add model cache volume mounts to flower service
- Ensures flower has access to pre-downloaded models
## Root Cause
The frontend container's nginx listens on port 8080 (per nginx.conf), but both
prod and offline compose files were mapping to port 80, causing healthchecks
to fail and making the app unreachable.
## Testing
- Production: Verified with one-line installer
- Offline: Ensures no docker pull attempts in air-gapped systems
- Both: Frontend now accessible and healthy
This resolves the issue where setup completed but the app didn't load.1 parent 8d0c3e4 commit 047d2cb
2 files changed
Lines changed: 21 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
10 | 11 | | |
11 | 12 | | |
12 | 13 | | |
| |||
25 | 26 | | |
26 | 27 | | |
27 | 28 | | |
| 29 | + | |
28 | 30 | | |
29 | 31 | | |
30 | 32 | | |
| |||
43 | 45 | | |
44 | 46 | | |
45 | 47 | | |
| 48 | + | |
46 | 49 | | |
47 | 50 | | |
48 | 51 | | |
| |||
56 | 59 | | |
57 | 60 | | |
58 | 61 | | |
| 62 | + | |
59 | 63 | | |
60 | 64 | | |
61 | 65 | | |
| |||
79 | 83 | | |
80 | 84 | | |
81 | 85 | | |
| 86 | + | |
82 | 87 | | |
83 | 88 | | |
84 | 89 | | |
| |||
150 | 155 | | |
151 | 156 | | |
152 | 157 | | |
| 158 | + | |
153 | 159 | | |
154 | 160 | | |
155 | 161 | | |
| |||
215 | 221 | | |
216 | 222 | | |
217 | 223 | | |
| 224 | + | |
218 | 225 | | |
219 | 226 | | |
220 | | - | |
| 227 | + | |
221 | 228 | | |
222 | 229 | | |
223 | 230 | | |
224 | 231 | | |
225 | 232 | | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
226 | 239 | | |
227 | 240 | | |
228 | 241 | | |
| 242 | + | |
229 | 243 | | |
230 | 244 | | |
231 | 245 | | |
| |||
243 | 257 | | |
244 | 258 | | |
245 | 259 | | |
| 260 | + | |
| 261 | + | |
246 | 262 | | |
247 | 263 | | |
248 | 264 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
110 | 110 | | |
111 | 111 | | |
112 | 112 | | |
| 113 | + | |
113 | 114 | | |
114 | 115 | | |
115 | 116 | | |
| |||
181 | 182 | | |
182 | 183 | | |
183 | 184 | | |
| 185 | + | |
184 | 186 | | |
185 | 187 | | |
186 | 188 | | |
| |||
220 | 222 | | |
221 | 223 | | |
222 | 224 | | |
223 | | - | |
| 225 | + | |
224 | 226 | | |
225 | 227 | | |
226 | 228 | | |
| |||
231 | 233 | | |
232 | 234 | | |
233 | 235 | | |
234 | | - | |
| 236 | + | |
235 | 237 | | |
236 | 238 | | |
237 | 239 | | |
| |||
0 commit comments