Skip to content

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

Merged
arielswalker merged 1 commit into
nasa: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#2687
arielswalker merged 1 commit into
nasa:devfrom
arielswalker:part-cfs/workflows#177-update-existing-workflows-internal-logic

Conversation

@arielswalker

Copy link
Copy Markdown
Contributor

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

Master PR: nasa/cFS#961

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

@arielswalker
arielswalker changed the base branch from main to dev April 7, 2026 23:26
@arielswalker
arielswalker force-pushed the part-cfs/workflows#177-update-existing-workflows-internal-logic branch from df5d19a to 800f997 Compare April 8, 2026 15:01
@arielswalker
arielswalker requested a review from jphickey April 13, 2026 16:25
@arielswalker
arielswalker merged commit 256bc4d into nasa:dev Apr 13, 2026
5 of 8 checks passed
@arielswalker
arielswalker deleted the part-cfs/workflows#177-update-existing-workflows-internal-logic branch April 13, 2026 18:49
@chillfig chillfig added this to the v7.0.1 milestone May 13, 2026
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