Feature/rebase to r2r vars

[emrgnt-cmplxty]

Important

Renames environment variables PORT to R2R_PORT and HOST to R2R_HOST across multiple files and fixes a test condition in runner_sdk.py.

• Environment Variables:
  • Rename PORT to R2R_PORT and HOST to R2R_HOST in docker_utils.py, compose.yaml, and app_entry.py.
  • Update environment variable references in compose.yaml for R2R_PORT and R2R_HOST.
• Tests:
  • Fix test condition in runner_sdk.py by changing delete_result["results"] != True to delete_result["results"] != None.

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

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
yc_demo	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 9, 2024 0:47am
yc-demo	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 9, 2024 0:47am

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
recommendation_platform	⬜️ Ignored (Inspect)			Oct 9, 2024 0:47am

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 4e78e03 in 20 seconds

More details

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

1. py/tests/integration/runner_sdk.py:1110

• Draft comment:
  The condition if delete_result["results"] != None: should be if delete_result["results"] != True: to correctly check for a successful deletion result.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The change from checking True to None seems intentional and aligns with the expected behavior of a deletion operation, which often returns None or an empty result to indicate success. The comment might be incorrect in suggesting to revert to checking True.
  I might be missing the specific behavior of the delete_collection method, which could indeed return True on success. However, typically, deletion operations return None or an empty result to indicate success.
  The change in the condition seems to align with common practices for deletion operations, and without specific documentation indicating otherwise, the comment's suggestion seems incorrect.
  The comment should be deleted as the change in the condition seems intentional and aligns with typical deletion operation results.

Workflow ID: wflow_1cYRxQa9zeXTwShG

---

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