Skip to content

Conversation

@machina-source
Copy link

Fixed the link to tracing

@dmitry-openai
Copy link
Collaborator

Thank you for the fix - we just made this update

DeanLuus22021994 pushed a commit to DeanLuus22021994/openai-agents-python that referenced this pull request Nov 8, 2025
File Structure Refactoring Complete:
-  All subdirectories created (statestore/, pubsub/, bindings/, secretstore/, configuration/)
-  All 7 YAML files moved and renamed
-  Documentation updated (README.md files)
-  TDD tests created (test_dapr_component_structure.py)
-  Verified all 6 components load successfully via Dapr metadata endpoint

Actual effort: 1.5 hours (estimated was 2-3 hours)

Verification:
- Dapr sidecar logs show all components loaded
- Metadata endpoint confirms all 6 components active
- No docker-compose.yml changes needed (Dapr scans subdirectories automatically)

Remaining: Update infrastructure CLI (moved to LOW PRIORITY openai#6)
DeanLuus22021994 pushed a commit to DeanLuus22021994/openai-agents-python that referenced this pull request Nov 8, 2025
…ocumentation

MEDIUM PRIORITY openai#6, openai#7, openai#8: COMPLETED

Volume Documentation:
- Added cleanup column to volume table
- Documented each volume's purpose and safety
- Added volume management commands (view sizes, clean caches)
-  Marked dangerous operations (database volumes)
- Provided docker-compose down -v cleanup instructions

Health Check Documentation:
- Added comprehensive health check table (interval, timeout, retries, start period)
- Documented health check commands (ps, inspect, logs)
- Provided tuning guidance for slower systems
- Explained why app-dev has no health check (uses sleep infinity)

GPU Support Documentation:
- Comprehensive requirements section (hardware, drivers, Docker Desktop)
- Configuration examples from docker-compose.yml
- Verification commands (nvidia-smi, torch.cuda)
- Troubleshooting guide:
  * WSL2 GPU support setup
  * Docker Desktop settings
  * Graceful CPU fallback
  * How to disable GPU reservation
- Usage notes (optional, graceful degradation, performance benefits)

All MEDIUM priority documentation items now complete!
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