Skip to content

Evan zhang/feature/deepiri gpu utils integration#170

Open
EZ6066 wants to merge 5 commits intodevfrom
evan_zhang/feature/deepiri-gpu-utils-integration
Open

Evan zhang/feature/deepiri gpu utils integration#170
EZ6066 wants to merge 5 commits intodevfrom
evan_zhang/feature/deepiri-gpu-utils-integration

Conversation

@EZ6066
Copy link
Copy Markdown
Member

@EZ6066 EZ6066 commented Apr 5, 2026

IMPORTANT:

  • PR must be opened from your personal branch → dev
  • You must tag @Team-Deepiri/support-team

Description

Updates submodule pointers for diri-cyrex and diri-helox and adjusts docker-compose.dev.yml so dev Cyrex builds align with deepiri-gpu-based base image selection and the integrated gpu-utils workflow.

  • Component, feature, or system affected: deepiri-platform — diri-cyrex, diri-helox submodules, docker-compose.dev.yml (Cyrex service)
  • Purpose of change: Integration / chore (record submodule SHAs + compose wiring for gpu-utils + Cyrex/Helox)

Changes

  • diri-cyrex — submodule bump to integration branch commit
  • diri-helox — submodule bump to integration branch commit
  • docker-compose.dev.yml — Cyrex build/env related updates for the integration

Related

  • Related PRs: deepiri-gpu-utils, diri-cyrex, diri-helox — branch evan_zhang/feature/deepiri-gpu-utils-integration (add PR links when open)

Testing

cd deepiri-platform
export CYREX_BASE_IMAGE=$(deepiri-gpu build-args --base-image-only)
export CYREX_DEVICE_TYPE=gpu
docker compose -f docker-compose.dev.yml build cyrex
docker compose -f docker-compose.dev.yml up -d cyrex
# After startup settles (~30–45s): curl http://127.0.0.1:8000/health

Additional testing details: Requires Docker, deepiri-gpu on PATH, and compose stack deps for Cyrex. Prefer 127.0.0.1 over localhost if you see connection resets during early startup.


Important Notes (Optional)

  • Merge order: gpu-utils → cyrex → helox → this PR is ideal; if submodules merge to dev first, rebase and refresh submodule SHAs to merged commits.
  • Do not commit docker-compose.dev.yml.bak or unrelated submodule bumps in this PR.

Workflow Checklist (Required)

  • Branch is up to date with dev
  • PR is from your branch → dev (no longer directly into main)
  • PR title follows convention (feat:, fix:, refactor:, etc.)
  • Tagged @Team-Deepiri/support-team

Review Requests

@Team-Deepiri/support-team

Hao291204 and others added 5 commits March 25, 2026 23:06
change docker/ollama moved to ops/docker/ollama
- Resolve docker-compose.dev.yml: persola DB comment (shared postgres)
- Omit accidental SSH key files from team_submodule_commands/backend-team/

Made-with: Cursor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants