Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cFS Integration candidate: 2021-07-27 #313

Merged
merged 8 commits into from
Aug 3, 2021
Merged

Conversation

astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Jul 21, 2021

Context

Combines

nasa/cFE#1701, v6.8.0-rc1+dev789
nasa/osal#1116, v5.1.0-rc1+dev578
nasa/cFS-GroundSystem#192, v2.2.0-rc1+dev58

Tests

Authors

@skliper
@jphickey
@zanzaben
@pepepr08
@nmullane
@PavLL
@ArielSAdams

Description

cFS PR #304

Fix #264, CodeQL Dependent on Successful Build

Submodule PRs

cFE

nasa/cFE#1699, correct return code check

nasa/cFE#1700, documentation for FS APIs that return OSAL codes

nasa/cFE#1695, Adding coverage tests for cfe_es_apps.c

nasa/cFE#1704, update docs for CFE_FS_BackgroundFileDumpRequest

nasa/cFE#1706, correct return code mismatches

nasa/cFE#1720, Requirements update for Caelum

nasa/cFE#1721, Add null check to CFE_ES_TaskID_ToIndex.

nasa/cFE#1719, scrub command documentation

nasa/cFE#1715, Add time arithmetic functional tests

nasa/cFE#1673, Add File Utility Functional Tests.

nasa/cFE#1711, Add custom epoch support to TIME UT

nasa/cFE#1739, Add cast to MIR prints

OSAL

nasa/osal#1114, Add unit test branch coverage

cFS Ground System

nasa/cFS-GroundSystem#185, Update tlm for ES Blockstats/memstats and TBL HK

nasa/cFS-GroundSystem#188, * might be referenced before assignment

arielswalker and others added 2 commits July 27, 2021 14:08
Add two workflows for building and testing cFS in both RTEMS 4.11 and 5
Add scripts folder into .github for running all unit tests inside QEMU
Use dockerhub to store docker image containing all QEMU and RTEMS
toolchain and dependencies

Address PR comments by jphickey:
  Remove monitoring of stdout by correctly using batch-mode option
  Correctly use make SIMULATION variable instead of manually modifying
  targets.cmake during the actions workflow.

Address PR comments by zanzaben:
  Remove forgotten targets.cmake target system

Use makefile to run qemu tests and create qemu disk images instead of
shell scripts

Use wildcard matching to find and run all unit tests. Skip network tests
that fail with the SKIP_NET_TESTS flag.

Clean .mk scripts, ignore failing network tests

Run network tests to see github failure behavior

Remove  from output failed tests step

Update Workflow README
@astrogeco astrogeco changed the title cFS Integration candidate: 2021-07-21 cFS Integration candidate: 2021-07-27 Jul 29, 2021
Combines:

nasa/cFE#1701, v6.8.0-rc1+dev789
nasa/osal#1116, v5.1.0-rc1+dev578
nasa/cFS-GroundSystem#192, v2.2.0-rc1+dev58

Includes:

**cFE**

nasa/cFE#1699, correct return code check
nasa/cFE#1700, documentation for FS APIs that return OSAL codes
nasa/cFE#1695, Adding coverage tests for cfe_es_apps.c
nasa/cFE#1673, Add File Utility Functional Tests.
nasa/cFE#1711, Add custom epoch support to TIME UT
nasa/cFE#1720, Requirements update for Caelum
nasa/cFE#1721, Add null check to CFE_ES_TaskID_ToIndex.
nasa/cFE#1719, scrub command documentation
nasa/cFE#1715, Add time arithmetic functional tests
nasa/cFE#1704, update docs for CFE_FS_BackgroundFileDumpRequest
nasa/cFE#1706, correct return code mismatches

**osal**

nasa/osal#1114, Add unit test branch coverage

**cFS-GroundSystem**

nasa/cFS-GroundSystem#185, Update tlm for ES Blockstats/memstats and TBL HK
nasa/cFS-GroundSystem#188, * might be referenced before assignment

Co-authored-by: Jacob Hageman <skliper@users.noreply.github.com>
Co-authored-by: Joseph Hickey <jphickey@users.noreply.github.com>
Co-authored-by: Alex Campbell <zanzaben@users.noreply.github.com>
Co-authored-by: Jose F Martinez Pedraza <pepepr08@users.noreply.github.com>
Co-authored-by: Niall Mullane <nmullane@users.noreply.github.com>
Co-authored-by: Paul <pavll@users.noreply.github.com>
@astrogeco astrogeco added the community Includes community contributions label Aug 3, 2021
@astrogeco astrogeco marked this pull request as ready for review August 3, 2021 17:08
@astrogeco astrogeco merged commit fe54f66 into main Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Includes community contributions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make CodeQL workflow dependent on a succesful build
3 participants