Conversation
❌ Deploy Preview for easypanel-templates failed.
|
|
Thanks for the GoClaw template contribution — it’s a strong start and the overall structure is good. Required changes before merge Fix schema consistency for providerApiKey Pin container images to immutable versions Correct metadata link labeling Recommended improvements (non-blocking but strongly suggested) Add X-Forwarded-Proto and X-Forwarded-Host headers in nginx proxy config. Once the required items are updated, I can re-review quickly for production readiness. |
|
For serviceName and other related serviceNames, we prefer serviceName: input.appServiceName and input.appServiceName-db and so on for any other service. |
|
I made the suggested adjustments. |
Ahson-Shaikh
left a comment
There was a problem hiding this comment.
Have you also tested the PR on Easypanel?
| serviceName: `${input.appServiceName}-db`, | ||
| password: databasePassword, | ||
| image: "pgvector/pgvector:pg18", | ||
| env: "PGDATA=/var/lib/postgresql/data", |
There was a problem hiding this comment.
Why did you need to specify this? The managed postgres instance would have its mounts already defined
There was a problem hiding this comment.
It was necessary because PGVector 18 changed the file folder, and Easypanel didn't from version 16 to 18, and without that, it gave an error about the data loading folder.
Multi-agent AI gateway with teams, delegation & orchestration. Single Go binary, 11+ LLM providers, 5 channels.