Skip to content

feat(configuration): activate workflow with build and flash#28

Open
charlesmadjeri wants to merge 1 commit into
devfrom
cma/feat-build-flash-ui-workflow
Open

feat(configuration): activate workflow with build and flash#28
charlesmadjeri wants to merge 1 commit into
devfrom
cma/feat-build-flash-ui-workflow

Conversation

@charlesmadjeri
Copy link
Copy Markdown
Contributor

@charlesmadjeri charlesmadjeri commented May 10, 2026

Issue: Sentience-Robotics/lucy_ros2#105

PR summary

Activate workflow & configure-pipeline

  • Dry run then activate: The modal runs ConfigurePipeline with dry_run: true against the TARGET preset (mapping_file = target name) for validation/generate feedback only.
  • Wet run on active: After /config/activate promotes the target to active, the workflow starts ConfigurePipeline again with mapping_file: '' (active YAML on disk), dry_run: false, boards_to_flash from the modal, and build_only from BUILD ONLY.
  • ACTIVATE ONLY: Skips build/flash entirely after /config/activate and refreshes config/active state.
  • Board selection: Empty selection no longer maps to “flash all boards”; it forces ACTIVATE ONLY and guards runActivateWorkflow so build/flash never runs without an explicit board selection when boards exist.
  • Mutual exclusion: BUILD ONLY and ACTIVATE ONLY clear each other when one is turned on; switches stay enabled unless the workflow is running (no cross-disabling).

Pipeline / robot metadata (related)

  • After a successful flash (non-empty flashed boards), active_meta.yaml records flashed_name / flashed_at; GetConfig returns flashed_config_name / flashed_at for the UI FLASHED tag.

@charlesmadjeri charlesmadjeri self-assigned this May 10, 2026
Base automatically changed from cma/hardware-config-ui to dev May 10, 2026 22:32
Signed-off-by: Charles Madjeri <80175305+charlesmadjeri@users.noreply.github.com>
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