Paths

Table of Contentst

Differential D58982

nanobsd: Add an optional CIDATA partition
Needs ReviewPublic
Actions

Authored by senguptaangshuman17_gmail.com on Wed, Aug 19, 10:33 AM.

Tags

None

Referenced Files

	F167979917: D58982.id184533.diff
	Tue, Aug 25, 5:49 PM

	F167965665: D58982.id184549.diff
	Tue, Aug 25, 4:09 PM

	F167965047: D58982.id184576.diff
	Tue, Aug 25, 4:03 PM

	F167960483: D58982.id185014.diff
	Tue, Aug 25, 3:35 PM

	F167960473: D58982.id184949.diff
	Tue, Aug 25, 3:35 PM

	F167934884: D58982.id184606.diff
	Tue, Aug 25, 12:26 PM

	F167932133: D58982.id184349.diff
	Tue, Aug 25, 12:07 PM

	F167931587: D58982.id184482.diff
	Tue, Aug 25, 12:03 PM

View All 39 Files

Subscribers

Details

Reviewers

jlduran
jrm
ashish

Summary

To be used when provisioning via nuageinit.

Nuageinit still needs to understand read-only root filesystems and
persist the changes to /cfg.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 75862
Build 72745: arc lint + arc unit

Event Timeline

senguptaangshuman17_gmail.com created this revision.Wed, Aug 19, 10:33 AM

Herald added a subscriber: imp. · View Herald TranscriptWed, Aug 19, 10:33 AM

senguptaangshuman17_gmail.com requested review of this revision.Wed, Aug 19, 10:33 AM

Harbormaster completed remote builds in B75735: Diff 184349.Wed, Aug 19, 10:33 AM

senguptaangshuman17_gmail.com added a parent revision: D58981: nanobsd: Allow custom alignment boundaries.Wed, Aug 19, 10:34 AM

senguptaangshuman17_gmail.com updated this revision to Diff 184482.Thu, Aug 20, 10:13 AM

Harbormaster completed remote builds in B75816: Diff 184482.Thu, Aug 20, 10:13 AM

senguptaangshuman17_gmail.com updated this revision to Diff 184533.Thu, Aug 20, 12:25 PM

Harbormaster completed remote builds in B75862: Diff 184533.Thu, Aug 20, 12:25 PM

senguptaangshuman17_gmail.com updated this revision to Diff 184543.Thu, Aug 20, 2:12 PM

Harbormaster completed remote builds in B75869: Diff 184543.Thu, Aug 20, 2:12 PM

senguptaangshuman17_gmail.com edited parent revisions, added: D58980: nanobsd: Do not ship with a FreeBSD-local directory; removed: D58981: nanobsd: Allow custom alignment boundaries.Thu, Aug 20, 2:13 PM

senguptaangshuman17_gmail.com updated this revision to Diff 184549.Thu, Aug 20, 2:27 PM

Harbormaster completed remote builds in B75874: Diff 184549.Thu, Aug 20, 2:27 PM

senguptaangshuman17_gmail.com updated this revision to Diff 184576.Thu, Aug 20, 3:13 PM

Harbormaster completed remote builds in B75899: Diff 184576.Thu, Aug 20, 3:13 PM

Address review comments: clarify nanobsd.sh is_defined() comment

Herald added a reviewer: scheduler. · View Herald TranscriptThu, Aug 20, 7:04 PM

Harbormaster completed remote builds in B75920: Diff 184606.Thu, Aug 20, 7:04 PM

lwhsu removed a reviewer: scheduler.Thu, Aug 20, 8:17 PM

Clean up merge conflict markers in defaults.sh

Harbormaster completed remote builds in B75973: Diff 184755.Sat, Aug 22, 10:32 PM

Why FAT?

tools/tools/nanobsd/gpt.sh
78	Why FAT?

senguptaangshuman17_gmail.com edited parent revisions, added: D58978: nanobsd: Handle the data partition in fstab; removed: D58980: nanobsd: Do not ship with a FreeBSD-local directory.Sun, Aug 23, 8:22 PM

In D58982#1355684, @imp wrote:

Why FAT?

As per my knowledge, nuageinit(7) manpage accepts either msdosfs (FAT) or cd9660 (ISO9660).
For why FAT was chosen over ISO9660, I need @jlduran's views on this.

senguptaangshuman17_gmail.com updated this revision to Diff 184850.Sun, Aug 23, 11:04 PM

Harbormaster completed remote builds in B76036: Diff 184850.Sun, Aug 23, 11:04 PM

In D58982#1356214, @senguptaangshuman17_gmail.com wrote:

In D58982#1355684, @imp wrote:

Why FAT?

As per my knowledge, nuageinit(7) manpage accepts either msdosfs (FAT) or cd9660 (ISO9660).
For why FAT was chosen over ISO9660, I need @jlduran's views on this.

I have no preference of one over the other. iso9660 sounds like a nice fit since it is inherently read-only; however, msdos is easier to provision. The idea was to replicate what Raspberry Pi images do: the user inserts the SD card into their PC, and it gets mounted as a FAT device that they can modify and easily provision.

tools/tools/nanobsd/customizations.sh
142	This change seems unrelated.
tools/tools/nanobsd/defaults.sh
162	This change seems unrelated.
1562	This change seems unrelated.
tools/tools/nanobsd/gpt.sh
290–291	Why remove this comment?
363	This change seems unrelated.
366	This change seems unrelated.
379	This change seems unrelated.
387	This change seems unrelated. I think some of these changes should be squashed with D58964.
tools/tools/nanobsd/nanobsd.sh
38 ↗	(On Diff #184850)	This comment seems unrelated.

senguptaangshuman17_gmail.com edited parent revisions, added: D58974: nanobsd: Refactor cust_pkgng; removed: D58978: nanobsd: Handle the data partition in fstab.Tue, Aug 25, 6:07 AM

senguptaangshuman17_gmail.com updated this revision to Diff 184949.Tue, Aug 25, 6:09 AM

Harbormaster completed remote builds in B76072: Diff 184949.Tue, Aug 25, 6:09 AM

Rebase stack on top of D59043 update

Harbormaster completed remote builds in B76137: Diff 185014.Tue, Aug 25, 8:01 AM

Revision Contents
Changeset List

Path

Size

tools/

tools/

nanobsd/

customizations.sh

7 lines

5 lines

81 lines

Diff 184533

tools/tools/nanobsd/customizations.sh

Loading...

tools/tools/nanobsd/defaults.sh

Loading...

tools/tools/nanobsd/gpt.sh

Loading...