Add Exclusion Flag for Postgres

[NolanTrem]

Important

Add --exclude-postgres flag to Docker setup to optionally skip Postgres container creation.

• Behavior:
  • Add --exclude-postgres flag to serve command in server.py to skip creating a Postgres container.
  • Modify run_docker_serve() in docker_utils.py to handle exclude_postgres flag.
  • Update check_set_docker_env_vars() in docker_utils.py to conditionally set Postgres environment variables.
• Docker Compose:
  • Add profiles: [postgres] to Postgres service in compose.full.yaml and compose.yaml.
  • Modify Docker commands in build_docker_command() in docker_utils.py to include/exclude Postgres profile based on flag.

^{This description was created by for f9d5f41. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to f9d5f41 in 55 seconds

More details

• Looked at 191 lines of code in 4 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. py/cli/utils/docker_utils.py:360

• Draft comment:
  The logic for pull_command and up_command is reversed. When exclude_postgres is true, the --profile postgres should not be included. Please swap the logic for pull_command and up_command when exclude_postgres is true.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The comment seems to be incorrect because the logic in the code aligns with the expected behavior: --profile postgres is included when exclude_postgres is false and excluded when true. The comment suggests swapping the logic, which would actually introduce a bug.
  I might be misunderstanding the intended behavior of the exclude_postgres flag. However, based on the code, the current implementation seems correct.
  The code clearly shows the intended behavior, and the comment's suggestion would lead to incorrect logic. The critique does not change the conclusion.
  The comment is incorrect because the current logic in the code is correct. The comment should be deleted.

2. py/cli/utils/docker_utils.py:357

• Draft comment:
  The logic for pull_command and up_command is reversed. When exclude_postgres is true, the --profile postgres should not be included. Please swap the logic for pull_command and up_command when exclude_postgres is true.
• Reason this comment was not posted:
  Marked as duplicate.

Workflow ID: wflow_r3krL0xUgpfWgIYt

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.