Skip to content

chore(infrastructure): update browser-worker uvicorn config (#4062) - #4064

Merged
mrveiss merged 2 commits into
Dev_new_guifrom
chore/4062-browser-worker-config
Apr 10, 2026
Merged

mrveiss merged 2 commits into
Dev_new_guifrom
chore/4062-browser-worker-config

Conversation

@mrveiss

@mrveiss mrveiss commented Apr 10, 2026

Copy link
Copy Markdown
Owner

Summary

Replace outdated uvicorn (Python) configuration in autobot-browser-worker.service template with current Node.js/Express implementation.

Changes Made

  • Updated ExecStart command from uvicorn to /usr/bin/node playwright-server.js
  • Changed EnvironmentFile location to /etc/autobot/autobot-browser-worker.env
  • Removed Python-specific environment variables (PYTHONUNBUFFERED, PYTHONDONTWRITEBYTECODE)
  • Added NODE_ENV=production environment variable
  • Updated restart policy from on-failure to always (matches actual behavior)
  • Changed RestartSec from 10 to 5s (faster recovery)
  • Updated timeout values (30s instead of 60s startup)
  • Added LimitNOFILE=65535 for file descriptor limits
  • Updated ReadWritePaths to match Node.js deployment paths

Validation

  • Systemd unit format verified as valid
  • Configuration aligns with Ansible playbook deployment at ansible/roles/browser/templates/playwright.service.j2
  • No dependencies on old Python/uvicorn infrastructure
  • Supports environment-based configuration via EnvironmentFile

Fixes #4062

🤖 Generated with Claude Code

mrveiss and others added 2 commits April 10, 2026 07:48
… to Node.js/Express (#4062)

Replace outdated uvicorn (Python) configuration with current Node.js/Express
implementation. Update environment variables, ExecStart command, restart policy,
and file paths to match actual deployment. Uses EnvironmentFile for
configuration instead of hardcoded values.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
…th ChromaDB (#4061)

Pin protobuf>=5.29.6,<8.0.0 and googleapis-common-protos>=1.57.0,<2.0.0 across
all three requirements files that include chromadb, eliminating the version conflict
where googleapis-common-protos 1.74.0 requires protobuf>=4.25.8. ChromaDB is bounded
to <2.0.0 to guard against breaking API changes; 1.5.5+ ships regenerated protos
compatible with protobuf 5.x so no 3.20.0 downgrade is needed.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

✅ SSOT Configuration Compliance: Passing

🎉 No hardcoded values detected that have SSOT config equivalents!

@mrveiss
mrveiss merged commit ce0276a into Dev_new_gui Apr 10, 2026
3 of 4 checks passed
@mrveiss
mrveiss deleted the chore/4062-browser-worker-config branch April 10, 2026 05:18
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.

1 participant