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

Adds initial support for building Hedgehog RPI images #294

Closed
wants to merge 2 commits into from

Conversation

aut0exec
Copy link
Contributor

🗣 Description

Initial support for creating an RPI Hedgehog image.

💭 Motivation and context

There was a request for this functionality and an open issue tracked on the INL Malcolm repo.

Raspberry Pi's are cheap and readily available these days and resources on the RPI 4 8GB are likely sufficient to support running a Hedgehog sensor. The Debian process used to build base Debian RPI images was used and then the necessary additions to build Debian into Hedgehog were added/installed via a chroot.

Raspberry Pi, Hedghehog ARM sensor

🧪 Testing

Full builds were done and tested on a RPI 4 8GB model. An actual RPI 4 sensor was setup and connected to a Malcolm instance via the guide here. No further testing beyond the E2E guide has been performed thus far. More testing recommended to ensure that the RPI image contains all the necessary configurations to function properly in a production setup.

✅ Pre-approval checklist

  • [✅ ] This PR has an informative and human-readable title.
  • [ ✅ ] All future TODOs are captured in issues, which are referenced
    in code comments.
  • [ ✅] All relevant type-of-change labels have been added.
  • [✅ ] I have read the CONTRIBUTING document.
  • [ ✅] These code changes follow cisagov code standards.
  • [ ✅] All relevant repo and/or project documentation has been updated
    to reflect the changes in this PR.

✅ Pre-merge checklist

  • Revert dependencies to default branches.
  • Finalize version.

✅ Post-merge checklist

@aut0exec aut0exec added the enhancement New feature or request label Jan 13, 2024
@aut0exec aut0exec requested a review from mmguero January 13, 2024 00:54
@aut0exec aut0exec self-assigned this Jan 13, 2024
)
values = [x.strip() for x in values]

print

Check notice

Code scanning / CodeQL

Statement has no effect Note

This statement has no effect.
Comment on lines +122 to +125
backports_stanza = """
%s
deb http://deb.debian.org/debian/ %s main %s
""" % (backports_enable, backports_suite, firmware_component)

Check warning

Code scanning / CodeQL

Unreachable code Warning

This statement is unreachable.
@mmguero
Copy link
Collaborator

mmguero commented Jan 18, 2024

I've pulled the commit(s) from this branch into my development fork, when it's included in the next Malcolm release I'll come back here and close this PR.

@aut0exec
Copy link
Contributor Author

Added a fix for the console not displaying UTF-8 properly and ensure that the sd-card is re-partitioned on first boot of the pi.

@mmguero
Copy link
Collaborator

mmguero commented Jan 26, 2024

Thanks I've pulled your update into my release testing branch and will test it out first thing Monday.

@mmguero
Copy link
Collaborator

mmguero commented Feb 14, 2024

commits included in and superceded by #297

@mmguero mmguero closed this Feb 14, 2024
@mmguero mmguero deleted the Raspberry_Pi_Sensor_Builds branch February 15, 2024 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants