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.

Sign up for GitHub

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

Jump to bottom

Add RTEMS build and test to CI #289

Closed
3 tasks done
nmullane opened this issue Jul 1, 2021 · 0 comments · Fixed by #290
Closed
3 tasks done

Add RTEMS build and test to CI #289

nmullane opened this issue Jul 1, 2021 · 0 comments · Fixed by #290

Comments

@nmullane
Copy link
Contributor

nmullane commented Jul 1, 2021

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I reviewed the cFS README.md file to see if the feature is in the major future work.
  • I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

Is your feature request related to a problem? Please describe.
Currently the build, test, and run GitHub Actions workflow only tests building and running on native Linux. This won't catch any errors that only affect running in RTEMS or when cross-compiling.

Describe the solution you'd like
Add GitHub Actions workflow to cross-compile and run unit tests in both RTEMS 5 and RTEMS 4.11 using QEMU

Describe alternatives you've considered
None

Requester Info
Niall Mullane - GSFC 582 Intern
nmullane added a commit to nmullane/cFS that referenced this issue Jul 1, 2021
@nmullane
Fix nasa#289 Add RTEMS build and test workflows
045b28e 
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
@nmullane nmullane mentioned this issue Jul 1, 2021
Merged
2 tasks
nmullane added a commit to nmullane/cFS that referenced this issue Jul 12, 2021
@nmullane
Fix nasa#289 Add RTEMS build and test workflows
e253919 
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.
nmullane pushed a commit to nmullane/cFS that referenced this issue Jul 14, 2021
@astrogeco @nmullane
Fix nasa#289 Add RTEMS build and test workflows
59e2224 
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.
nmullane added a commit to nmullane/cFS that referenced this issue Jul 14, 2021
@nmullane
Fix nasa#289 Add RTEMS build and test worflows
441eba4 
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.
nmullane added a commit to nmullane/cFS that referenced this issue Jul 14, 2021
@nmullane
Fix nasa#289 Add RTEMS build and test worflows
16ed8f6 
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
nmullane added a commit to nmullane/cFS that referenced this issue Jul 21, 2021
@nmullane
Fix nasa#289, Add RTEMS build and test workflows
f604c7f 
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
nmullane added a commit to nmullane/cFS that referenced this issue Jul 26, 2021
@nmullane
Fix nasa#289, Add RTEMS build and test workflows
41f8974 
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
astrogeco added a commit that referenced this issue Jul 29, 2021
@astrogeco
Merge pull request #290 from nmullane/interns-8-add-rtems-testing
c0434cd 
Fix #289, Add RTEMS build and test workflows
Frank393 pushed a commit to Frank393/cFS that referenced this issue Aug 2, 2021
@nmullane @Frank393
Fix nasa#289, Add RTEMS build and test workflows
3cf670e 
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #289, Add RTEMS build and test workflows nmullane/cFS
1 participant
@nmullane