Skip to content

Add dev/prod environment support for cloud API to main - #507

Merged
ebrahimebrahim merged 2 commits into
mainfrom
cloud-dev-prod-env
Sep 15, 2026
Merged

ebrahimebrahim merged 2 commits into
mainfrom
cloud-dev-prod-env

Conversation

@stahir715

Copy link
Copy Markdown

Summary

Licensing layer

  • Apache-2.0 extension (3D Slicer, wellness/veterinary module, partner SDK, language binding)
  • AGPL-3.0 core (firmware, sensing/beamforming/reconstruction/signal-processing/calibration algorithms, hardware reference design)
  • Docs / data / tooling only
  • This PR moves a file between the AGPL core and the Apache layer — if checked, this is NOT an ordinary PR: it requires the license-manifest change process (and board approval for anything leaving the core). Link the approval.

Checklist

  • Commits are signed off (DCO): git commit -s
  • New files carry the correct SPDX-License-Identifier for their layer
  • Tests and docs updated
  • No incompatible third-party code introduced
  • CI is green and CODEOWNERS review requested

Notes for reviewers

Restores dev/prod environment selection for the cloud sync Cloud class. This was originally added on the add-cloud-dev-prod-environment-support branch (commit 04fd1f8) but never merged into main. That branch is now ~70 commits stale, so this PR cherry-picks that commit onto current main rather than merging the branch directly. Cloud.init gains an optional environment parameter (defaults to "prod", unchanged from current behavior) that selects between API_URL_PROD and the newly added API_URL_DEV.

@dblizinski
dblizinski enabled auto-merge (squash) September 15, 2026 14:40
posnova and others added 2 commits September 15, 2026 12:02
Signed-off-by: Seemal Tahir <stahir@openwater.cc>
Signed-off-by: Ebrahim Ebrahim <ebrahim.ebrahim@kitware.com>
@ebrahimebrahim

Copy link
Copy Markdown
Collaborator

I added validation so only "dev" and "prod" are accepted. Previously, a typo such as "deev" silently selected production, which could send development sync traffic to the production service.

@ebrahimebrahim
ebrahimebrahim enabled auto-merge (rebase) September 15, 2026 16:04
@ebrahimebrahim
ebrahimebrahim merged commit 286680b into main Sep 15, 2026
10 of 11 checks passed
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.

4 participants