enable firstboot configuration for arm images
Needs ReviewPublic
Actions

Authored by jmg on Oct 8 2020, 12:06 AM.

Details

Reviewers

imp
manu
• ian

Summary

This revision has three changes.

libexec/configinit -- this is an import of cperciva's ec2-scripts configinit w/ a minor change to use find -s -exec instead of | sort.

libexec/rc/rc.d/fs_configinit -- firstboot script that will process a given file through (1). The default has the file at /boot/msdos/config.init, allowing users to drop a file there, which can do things like preload rc.conf for static IPs, install packages on first boot and more.

The advantage of this is it allows people with Windows or MacOSX or Linux to more easily configure a FreeBSD for first boot. Allowing things like auto install of X or other setups via a single file drop.

release/arm64/PINE64-LTS.conf -- a sample of how to enable (2) on an image.

Test Plan

I was going to attempt to do a complete release build, but that is just
not doable easily.

So I have made sure that make install or distribute works in a VM, and
verified that when the proper rc.conf variable is set, that the actions
are taken.

I have also manually installed these files into a proper PINE64-LTS image
and verified that it works.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 34051
Build 31234: arc lint + arc unit

Event Timeline

jmg created this revision.Oct 8 2020, 12:06 AM

Herald added a subscriber: andrew. · View Herald TranscriptOct 8 2020, 12:06 AM

jmg requested review of this revision.Oct 8 2020, 12:06 AM

Harbormaster completed remote builds in B34051: Diff 78001.Oct 8 2020, 12:06 AM

imp added inline comments.Oct 8 2020, 12:16 AM

release/release.sh
78	shouldn't these be := to assign defaults?

I am planning on committing 1 and 2 individually.

the parts of release.sh can be ignored. That was some work to make things testable, but I've abandoned that work for now, since it's too cumbersome.

If I knew arc better (and the docs for arc were better), I would have excluded that change.

I think enabling fs_configinit for all arm images is safe, as if the file does not exist, nothing bad happens.

jmg added inline comments.Oct 8 2020, 12:20 AM

release/release.sh
78	You can ignore these. I don't plan on committing them. As for the = vs := I just copied what worked from rc.d scripts. I don't even know how the bare = works, as it's not documented in sh(1), but it does.