Skip to content

Fix cFS/workflows#177#4

Open
arielswalker wants to merge 1 commit intomainfrom
fix-cfs-workflows-177
Open

Fix cFS/workflows#177#4
arielswalker wants to merge 1 commit intomainfrom
fix-cfs-workflows-177

Conversation

@arielswalker
Copy link
Copy Markdown
Owner

@arielswalker arielswalker commented Mar 25, 2026

Checklist (Please check before submitting)

Describe the contribution
Updates existing cFE open source workflows to match the internal ones.

Testing performed
Checks in this PR

Expected behavior changes
Format Check

cFS Documentation and Guides

  • uses setup-cfe hosted on cFE
  • uses new changes to build-deploy-doc.yml on cFS
  • removes checks for duplicate

Code Coverage Analysis

  • uses omit deprecate false, not true
  • removes checks for duplicate
  • uses setup-cfe hosted on cFE
  • using new cfsbuildenv-ubuntu22 container

Functional Test

  • not exact as internal workflow since the internal workflow includes actions to start, stop, and check health logs of a docker container that the internal self-hosted runner provides
  • uses enable unit test false, not true
  • removes checks for duplicate
  • uses setup-cfe hosted on cFE
  • uses start-cfs-container, stop-cfs-container, and healthcheck-logs hosted on cFS
  • using new cfsexec-ubuntu22 container

Format Check

  • uses new style, .clang-format

Static Analysis

  • uses setup-cfe hosted on cFE
  • uses cppcheck/action.yml hosted at nasa/cFS
  • using new cfsbuildenv-ubuntu22 container

System(s) tested on
GitHub Actions

Additional context
Updated setup-cfe/action.yml to be reusable for both internal and opensource workflows. This action has a new input called org that determines which organization to check out repositories. This allows only one copy of this file to be maintained.

These cFE workflow changes are dependent on new container hosted at core-flight-system/containers and new/updated files at nasa/cFS. This PR is just a demo that points to these updated files on my forked repositories.

Related PR for building and pushing the containers: arielswalker/containers#1

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

@arielswalker arielswalker force-pushed the fix-cfs-workflows-177 branch 8 times, most recently from f463957 to cdac4ef Compare March 31, 2026 17:58
@arielswalker arielswalker force-pushed the fix-cfs-workflows-177 branch 3 times, most recently from 910793e to 931e7e8 Compare April 3, 2026 16:02
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.

1 participant