Skip to content

Conversation

@hrfarmer
Copy link
Collaborator

@hrfarmer hrfarmer commented Oct 2, 2025

Add BootloaderTrigger Component

Description

The BootloaderTrigger component is meant to be used for (planned) automatic integration testing. The integration tester will use the Python fprime_gds library to trigger the TRIGGER_BOOTLOADER command, so that new firmware can be installed to a board without manual intervention. This should not be used (or even included) in a production configuration, as this would kill the satellite with no way to recover it.

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Z Tests
  • Manual testing (describe steps)
  • Start the GDS, run the ReferenceDeployment.bootloaderTrigger.TRIGGER_BOOTLOADER command, and verify that the board reboots and mounts the RP2350 volume.

Checklist

  • Written detailed sdd with requirements, channels, ports, commands, telemetry defined and correctly formatted and spelled
  • Have written relevant integration tests and have documented them in the sdd
  • Have done a code review with
  • Have tested this PR on every supported board with correct board definitions

Further Notes / Considerations

Probably don't need to worry about it immediately, but in the future we should think about how this component will be handled in a production configuration so there isn't risk of accidentally running it.

Copy link
Contributor

@Mikefly123 Mikefly123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the quick turn on this. Pretty cool functionality to get in, just gotta make sure we're careful about leaving it in for flight.

@hrfarmer hrfarmer merged commit d1b1502 into main Oct 2, 2025
2 checks passed
@hrfarmer hrfarmer deleted the bootloader-trigger branch October 2, 2025 12:30
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.

3 participants