Skip to content

Conversation

Copy link

Copilot AI commented Nov 5, 2025

The install.sh script was updated to use uv, systemd, gunicorn, and bd_settings.cfg, but script_for_github_actions.sh still used the old pip/virtualenv, supervisor, uwsgi, and split config files.

Changes

Package management

  • Replaced virtualenv setup with uv sync
  • Changed to snap-based installation for node and astral-uv

Configuration

  • Unified config writes to /srv/BuildingDepot/configs/bd_settings.cfg (was cs_config/ds_config)
  • Updated directory from /srv/buildingdepot to /srv/BuildingDepot

Service management

  • Replaced supervisor with systemd service files (bd-central, bd-data, bd-replica)
  • Removed uwsgi configuration (gunicorn now handled by systemd ExecStart)
  • Changed service control from supervisorctl to systemctl

Deployment functions

  • Consolidated deploy_centralservice() and deploy_dataservice() into single deploy_services()
  • Removed setup_venv() and deploy_config() functions

Test compatibility

  • Kept setup_bd.py "test" argument (vs "install") to generate test credentials for CI

Net change: -61 lines (20% reduction)

Original prompt

This section details on the original issue you should resolve

<issue_title>Fix script_for_github_actions.sh script</issue_title>
<issue_description>The install.sh script was recently updated to change a few parts of the deployment

  • config file location (now using bd_settings.cfg)
  • systemd not supervisorctl
  • uv and not pip
  • gunicorn and not uwsgi

The script_for_github_actions.sh script is somewhat similar, but with more hardcoded variables for the github actions deployment. This script should be fixed to use the new installation method.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits November 5, 2025 18:42
Co-authored-by: weinshel <6042920+weinshel@users.noreply.github.com>
…sary mkdir

Co-authored-by: weinshel <6042920+weinshel@users.noreply.github.com>
Co-authored-by: weinshel <6042920+weinshel@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix script for github actions deployment Migrate script_for_github_actions.sh to uv/systemd deployment Nov 5, 2025
Copilot AI requested a review from weinshel November 5, 2025 18:52
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.

2 participants