Skip to content

ci: add fprime-stm32-baremetal-reference to the ext-ci manifest (replaces #6204) - #296

Closed
devin-ai-integration[bot] wants to merge 1 commit into
devin/1792479125-ext-ci-dispatchfrom
devin/1793759209-ext-ci-stm32
Closed

devin-ai-integration[bot] wants to merge 1 commit into
devin/1792479125-ext-ci-dispatchfrom
devin/1793759209-ext-ci-stm32

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Related Issue(s) nasa#6162 (replaces nasa#6204)
Has Unit Tests (y/n) n
Documentation Included (y/n) y
Generative AI was used in this contribution (y/n) AI

Change Description

Adds fprime-community/fprime-stm32-baremetal-reference as an ext-ci manifest entry (notify tier, on_pr: true, 45 min) and the STM32H753XI-EVAL2 row in supported-platforms.md, pointing at the ext repo's own ci.yml badge.

No new workflow, no self-hosted job, no secrets in nasa/fprime: the orchestrator dispatches the stm32 repo's ci.yml (hosted cross-compile + host UTs) for F´ PRs, and the stm32 repo owns the HIL stage on its own runner. Ext-repo side: fprime-stm32-baremetal-reference PR (link to follow; replaces fprime-community/fprime-stm32-baremetal-reference#32).

Rationale

nasa#6204 + stm32 #32 implemented the inverse of the ext-CI dispatch design: the ext repo dispatched into nasa/fprime with a cross-org actions: write token and nasa/fprime posted back with a second token. This PR is the design-conformant replacement.

Testing/Review Recommendations

  • owners is thomas-bc while on the fork (an orchestrator failure comment @-mentions owners, and the entry cannot dispatch until the stm32 ci.yml is on devel); switch to Iva-L when promoting upstream, per the _fork_testing note.
  • Deployment order: merge the stm32 ci.yml first, then this entry (GitHub only dispatches workflows present on the target default branch).

Future Work

  • v2: HIL for nasa/fprime PRs via a maintainer label → run_hil, behind a hil-hardware environment on the stm32 repo; orchestrator to treat waiting as pending.

Contributor Checklist

AI Usage (see policy)

Devin authored the manifest entry and docs row from the ext-CI plan and nasa#6204, under maintainer direction.

IAMAI

Link to Devin session: https://nasa-jpl-demo.devinenterprise.com/sessions/e3e60d4ee7c141f5bc10f5a3c267f511
Open in Devin Desktop: https://nasa-jpl-demo.devinenterprise.com/desktop/session/e3e60d4ee7c141f5bc10f5a3c267f511?variant=devin
Requested by: @thomas-bc-autobot


Devin Review

@devin-ai-integration

Copy link
Copy Markdown
Author

I'll fix CI failures and address comments from users with write access that start with 'Devin'.

  • Disable automatic comment, CI, and merge conflict monitoring

Prompt hidden (unlisted session)

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@Iva-L Iva-L left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Some suggestions to run the intended .yml file.

{
"id": "stm32-baremetal",
"repository": "fprime-community/fprime-stm32-baremetal-reference",
"workflow": "ci.yml",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The workflow file for the hil tests is hil.yml, ci.yml is only for format checks and cross-compilation.

| PolarFire SoC | VxWorks 7 | RISC-V | TBD | N/A |
| PyCubed | Zephyr | RISC-V | [`fprime-zephyr-reference`](https://github.com/fprime-community/fprime-zephyr-reference) | N/A |
| Raspberry Pi | Linux | ARMv8 | [`fprime-workshop-led-blinker`](https://github.com/fprime-community/fprime-workshop-led-blinker) | [![External Repo: RPI LedBlinker](https://github.com/nasa/fprime/actions/workflows/ext-raspberry-led-blinker.yml/badge.svg)](https://github.com/nasa/fprime/actions/workflows/ext-raspberry-led-blinker.yml) |
| STM32H753XI-EVAL2 | Baremetal | ARMv7-M | [`fprime-stm32-baremetal-reference`](https://github.com/fprime-community/fprime-stm32-baremetal-reference) | [![CI](https://github.com/fprime-community/fprime-stm32-baremetal-reference/actions/workflows/ci.yml/badge.svg)](https://github.com/fprime-community/fprime-stm32-baremetal-reference/actions/workflows/ci.yml) |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same thing here, for hil tests is hil.yml and for cross-compilation is ci.yml

@devin-ai-integration

Copy link
Copy Markdown
Author

Superseded by nasa#6328 (ext-ci landed upstream) and nasa#6378 (stm32 manifest entry).

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