fix(env): docker compose variable interpolation issue for COMPOSE_PRO…

[hanktrizz]

Updated files

• generate-docker-compose
• docker-compose.yml

Summary

Added condition to ignore the environment variable COMPOSE_PROFILES. Otherwise, it will fail upon running docker compose up -d with the current interpolation setup.

Fixes #12081

Screenshots

Before	After

Checklist

• This change requires a documentation update, included: Dify Document
• I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
• I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
• I've updated the documentation accordingly.
• I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

[crazywoola]

dify/docker/docker-compose-template.yaml

Line 1 in e88ea71

x-shared-env: &shared-api-worker-env

You should modify this file instead of modify the .env.example directly.
The new .env.example is generated by this command
https://github.com/langgenius/dify/blob/1885d3df9968d778664bacdde4ea7a4d9448070f/docker/generate_docker_compose

[hanktrizz]

dify/docker/docker-compose-template.yaml

Line 1 in e88ea71

x-shared-env: &shared-api-worker-env

You should modify this file instead of modify the .env.example directly. The new .env.example is generated by this command https://github.com/langgenius/dify/blob/1885d3df9968d778664bacdde4ea7a4d9448070f/docker/generate_docker_compose

Correct me if I'm wrong @crazywoola but it seems the sequence of generation goes like this (as per main() inside generate-docker-compose file:

Reads .env.example -> creates the shared env variables block -> substitutes the anchor with the shared block -> generate the template.yaml -> re-generate the final docker-compose.yaml.

There is nowhere in the docker-compose-template.yaml that is dealing with the env variables COMPOSE_PROFILES or VECTOR_STORE directly.
As I am relatively new to this codebase, I am happy to stand corrected and keen to learn if you can help clarify this.

[laipz8200]

It seems that COMPOSE_PROFILES shouldn’t appear in the docker-compose.yaml file. You should modify the generate-docker-compose script to ignore this key.

[hanktrizz]

Okay. I'll try and ignore that specific key. Will commit soon-ish due to the fact it's Xmas season 😄

[hanktrizz]

@laipz8200 Hi, can you please have another look? I believe it's as straightforward as it gets in terms of ignoring that key as per your advice. I have regenerated the docker-compose.yml file and also checked it in. Verified that docker compose up -d still run smoothly after the changes as well.

Sorry for the delay due to holidays hhh.

[laipz8200]

Looks good to me. Now, the .env.example should be changed back.

[hanktrizz]

Looks good to me. Now, the .env.example should be changed back.

All done, mate. Please help to approve and merge at will 🙏

[laipz8200]

LGTM. Thank you for your contribution!