Skip to content

Dev#65

Merged
haoruizhou merged 6 commits into
mainfrom
dev
Apr 28, 2026
Merged

Dev#65
haoruizhou merged 6 commits into
mainfrom
dev

Conversation

@haoruizhou

Copy link
Copy Markdown
Contributor

No description provided.

Refactor MacBook deployment config and Timescale init: update .env.macbook usage and variables (add TIMESCALE_TABLE and DBC_HOST_PATH, adjust compose invocation), change docker-compose.macbook-base.yml to use the new host DBC path default and remove the unused volumes section, and harden timescaledb_init_macbook.sql to only create indexes, enable compression, and add compression policies when the target table is a hypertable to avoid errors on non-hypertable tables.

Update SETUP_CARD.tex
- MACBOOK_DEPLOY.md: rewrite to reflect local TimescaleDB stack (macbook-base)
- run_ci_tests.sh: rename influxdb3 → timescaledb in container checks and log paths
- Remove all `profiles: [car/base]` and `--profile base` flags
- car-telemetry.service is the car deployment method (no Docker)
- macbook-base and rpi-base are standalone compose files (no profiles)
- Update SETUP_CARD.tex, MACBOOK_DEPLOY.md, WHICH_ONE.md accordingly
@haoruizhou haoruizhou merged commit 60888c6 into main Apr 28, 2026
14 of 15 checks passed
@haoruizhou haoruizhou deleted the dev branch April 28, 2026 01:57
@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review

- ${DBC_HOST_PATH:-./example.dbc}:/app/active.dbc:ro

P1 Badge Fix default DBC bind path for MacBook stack

The telemetry volume defaults to ${DBC_HOST_PATH:-./example.dbc}, but this compose file lives in deploy/ while the checked-in DBC is at universal-telemetry-software/example.dbc. With the documented default .env.macbook, Docker resolves ./example.dbc to deploy/example.dbc (missing), so the telemetry container fails to start unless users manually override DBC_HOST_PATH.



P1 Badge Create the same Timescale table name the app writes

This initializer creates WFR26test, but the new MacBook compose config writes to TIMESCALE_TABLE=${TIMESCALE_TABLE:-WFR26test}_base (default WFR26test_base). TimescaleBridge issues ALTER TABLE/INSERT on the configured table and does not bootstrap missing base tables, so the default stack hits relation-not-found errors and never persists telemetry.


echo -e " 4. Start the stack: docker compose -f deploy/docker-compose.rpi-base.yml up -d"
echo -e " 5. Check role: docker compose -f deploy/docker-compose.rpi-base.yml logs telemetry | grep Role"

P2 Badge Keep car SocketCAN setup steps from launching base role

This script is for the car RPi, but the updated next-step command now starts docker-compose.rpi-base.yml, whose telemetry service is hardcoded to ROLE=base. Following these instructions on a car device starts the wrong role (and conflicts with the car systemd path), so expected car-side CAN transmit behavior does not come up.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

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