Skip to content

Part cFS/workflows#177, Update Existing Workflows with Internal Logic#961

Merged
arielswalker merged 1 commit intonasa:devfrom
arielswalker:part-cfs/workflows#177-update-existing-workflows-internal-logic
Apr 13, 2026
Merged

Part cFS/workflows#177, Update Existing Workflows with Internal Logic#961
arielswalker merged 1 commit intonasa:devfrom
arielswalker:part-cfs/workflows#177-update-existing-workflows-internal-logic

Conversation

@arielswalker
Copy link
Copy Markdown
Contributor

@arielswalker arielswalker commented Apr 7, 2026

Checklist (Please check before submitting)

Describe the contribution
Fixes internal cFS/workflows#177. Updates existing opensource workflows with the internal logic. Does not add new internal workflows.

Testing performed
Tested in this demo PR: arielswalker#2

build-doc-reusable.yml

build-run-app-reusable.yml

format-check.yml

unit-test-coverage-reusable.yml

actions/cppcheck

actions/setup-app

actions/start-cfs-container, actions/healthchecklogs, actions/stop-cfs-container

app-static-analysis-reusable.yml

cFE Demo

OSAL Demo

App/CF Demo

Expected behavior changes
Uses new containers as created here: https://github.com/arielswalker/containers/actions/runs/23948566629

Uses new scripts, actions, and internal logic.

System(s) tested on
GitHub Actions

Additional context
Any workflow that builds cFS using the old build commands will fail because of the techdev-multitarget-bundle changes. These workflows include:

Build and Test rtems 5 [OMIT_DEPRECATED=true]
Build, Test, and Run [OMIT_DEPRECATED = false]
Build, Test, and Run [OMIT_DEPRECATED = true]
cFS Documentation and Guides
CodeQL
These workflows were not updated because they do not exist internally. The next step is to replace/remove these workflows based on what is done internally.

Some workflows are failing due to findings with the code, rather than the workflow itself not using commands correctly.

.gitattributes was create because GitHub downloads the archive of the entire arielswalker/cFS repository to get to the actions/ folders, which includes things such as the symlink. When the runner downloads the archive and extracts it into a temporary staging folder, the extraction process tries to recreate every file and symlink. When it tries to create the simple_defs symlink, it sees that it points to a submodule directory (cfe/...) that wasn't included in the archive. The extraction process fails immediately, crashing the workflow before it even has a chance to look at your action. This error is seen here: https://github.com/arielswalker/cFE/actions/runs/23951653083/attempts/3

These changes will not work until containers are built and published first. PR to do so is found core-flight-system/containers#25.

It is assumed that these workflows will only be on dev, therefore all calls to the reusable workflows, checkout branches, and calls to scripts are using the dev branch.

Contributor Info - All information REQUIRED for consideration of pull request
Ariel Walker, MCSG TECH.

@arielswalker arielswalker force-pushed the part-cfs/workflows#177-update-existing-workflows-internal-logic branch from 2d21a4a to f5767bd Compare April 7, 2026 17:28
@arielswalker arielswalker force-pushed the part-cfs/workflows#177-update-existing-workflows-internal-logic branch 2 times, most recently from cafff14 to e8386b5 Compare April 8, 2026 20:06
@arielswalker arielswalker force-pushed the part-cfs/workflows#177-update-existing-workflows-internal-logic branch from e8386b5 to 5130b5a Compare April 8, 2026 20:15
@arielswalker arielswalker merged commit 938ce08 into nasa:dev Apr 13, 2026
17 of 28 checks passed
@arielswalker arielswalker deleted the part-cfs/workflows#177-update-existing-workflows-internal-logic branch April 13, 2026 16:24
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