Skip to content

ci(integration): set the server's local-dev stand-in switch under both env names - #593

Open
gjedlicska wants to merge 1 commit into
mainfrom
gergo/job-system-local-dev-env
Open

gjedlicska wants to merge 1 commit into
mainfrom
gergo/job-system-local-dev-env

Conversation

@gjedlicska

Copy link
Copy Markdown
Member

Why

docker-compose-internal.yml runs speckle-server:latest with the off-cluster Docker stand-in for viewer-.dat generation, switched on by CONVERSION_RUNNER_LOCAL_DEV. The server's job-system boundary refactor (ENG-10149, on feat/job-system) renames that switch to JOB_SYSTEM_LOCAL_DEV. Once that branch reaches main, latest reads the new name and .dat generation in this suite would throw JobSystemNotInClusterError.

What

Set the switch under both names. Today's latest reads the old one, the post-rename image reads the new one; either way the stand-in is on. No merge ordering needed between the two repos.

Follow-up

Drop CONVERSION_RUNNER_LOCAL_DEV once the rename is on a published latest.

Verification

DOCKER_GID=999 docker compose -f docker-compose-internal.yml config resolves with both keys on the speckle-server service.

…h env names

The internal compose runs speckle-server:latest with the off-cluster Docker
stand-in for viewer-.dat generation. The server's job-system boundary refactor
(ENG-10149, on feat/job-system) renames the switch from
CONVERSION_RUNNER_LOCAL_DEV to JOB_SYSTEM_LOCAL_DEV; `latest` will start
reading the new name whenever that branch reaches main. Setting both keeps the
suite green across the transition with no merge ordering between the repos.
Drop CONVERSION_RUNNER_LOCAL_DEV once the rename is on a published `latest`.
@linear-code

linear-code Bot commented Sep 30, 2026

Copy link
Copy Markdown

ENG-10266

@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.71%. Comparing base (9908ab9) to head (e76fcde).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #593      +/-   ##
==========================================
- Coverage   75.71%   75.71%   -0.01%     
==========================================
  Files         483      483              
  Lines       27028    27028              
  Branches     2724     2724              
==========================================
- Hits        20464    20463       -1     
  Misses       5586     5586              
- Partials      978      979       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This branch has not been deployed

No deployments
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