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

Fix #11, Add RTEMS 6 container for cFS test workflow #12

Merged
merged 2 commits into from May 24, 2022
Merged

Fix #11, Add RTEMS 6 container for cFS test workflow #12

merged 2 commits into from May 24, 2022

Conversation

ghost
Copy link

@ghost ghost commented May 17, 2022

This pull request adds a RTEMS 6 container and appropriate workflow to build it and add it to the container repository. The RTEMS 6 container will be used to support the RTEMS 6 cFS build and tests.

@ghost
Copy link
Author

ghost commented May 17, 2022

Note that in my forked repository I was able to get all 3 containers to build correctly, but I could not push the completed container.

@astrogeco astrogeco changed the title Fix 11 Add RTEMS 6 container for cFS test workflow Fix #11, Add RTEMS 6 container for cFS test workflow May 18, 2022
@astrogeco
Copy link
Contributor

astrogeco commented May 18, 2022

CCB:2022-05-18 APPROVED

  • @acudmore open issue in this repo to remove network stack fix when RTEMS does the fix on their end
  • choose a snapshot that works and open an issue to update it regularly
    • bonus points, automate with dependabot or a similar tool

@ghost
Copy link
Author

ghost commented May 18, 2022

I added a commit to use specific RTEMS repository commits, and documented the date for them in the Dockerfile.

@astrogeco astrogeco merged commit 5d55d23 into core-flight-system:main May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants