Skip to content

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

Open
arielswalker wants to merge 9 commits intomainfrom
part-cfs/workflows#177-update-workflows-internal-logic
Open

Part cFS/workflows#177, Update Workflows with Internal Logic#2
arielswalker wants to merge 9 commits intomainfrom
part-cfs/workflows#177-update-workflows-internal-logic

Conversation

@arielswalker
Copy link
Copy Markdown
Owner

@arielswalker arielswalker commented Apr 3, 2026

Checklist (Please check before submitting)

Describe the contribution
Branch from nasa#874. Tests updating already existing workflows to use internal logic. This does not delete or add new workflows that matches the internal repositories.

Testing performed
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

OSAL

CF

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 here.

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

@arielswalker arielswalker force-pushed the part-cfs/workflows#177-update-workflows-internal-logic branch 7 times, most recently from 6844341 to cdedbb9 Compare April 3, 2026 16:01
@arielswalker arielswalker force-pushed the part-cfs/workflows#177-update-workflows-internal-logic branch from cdedbb9 to 885c9f6 Compare April 6, 2026 16:33
@arielswalker arielswalker force-pushed the part-cfs/workflows#177-update-workflows-internal-logic branch 6 times, most recently from 7e90f32 to 4b83016 Compare April 8, 2026 20:11
@arielswalker arielswalker force-pushed the part-cfs/workflows#177-update-workflows-internal-logic branch from 4b83016 to 7f31ede Compare April 8, 2026 20:12
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