Paths

Table of Contentst

Differential D52452

release: Prepare EC2 cloudware images to be buildable as non-root
ClosedPublic
Actions

Authored by markj on Tue, Sep 9, 2:54 PM.

Tags

None

Referenced Files

	F130294344: D52452.id161791.diff
	Sun, Sep 28, 3:08 AM

	F130257511: D52452.diff
	Sat, Sep 27, 8:04 PM

	Unknown Object (File)
	Wed, Sep 24, 11:04 AM

	Unknown Object (File)
	Tue, Sep 23, 5:38 PM

	Unknown Object (File)
	Wed, Sep 17, 11:31 AM

	Unknown Object (File)
	Wed, Sep 17, 11:05 AM

	Unknown Object (File)
	Wed, Sep 10, 7:47 AM

	Unknown Object (File)
	Tue, Sep 9, 9:10 PM

View All 11 Files

Subscribers

Details

Reviewers

emaste
cperciva

Group Reviewers

Commits

rG8027de008d9f: release: Prepare EC2 cloudware images to be buildable as non-root

Summary

Avoid chrooting to uninstall pkg, as it will not be installed in the first place.
Add metalog entries.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Tue, Sep 9, 2:54 PM

Herald added subscribers: jrtc27, imp. · View Herald TranscriptTue, Sep 9, 2:54 PM

markj requested review of this revision.Tue, Sep 9, 2:54 PM

Harbormaster completed remote builds in B66924: Diff 161791.Tue, Sep 9, 2:54 PM

markj added a parent revision: D52451: release: Make azure cloudware images buildable as non-root.Tue, Sep 9, 2:54 PM

markj added a child revision: D52453: release: Enable installing packages as a non-root user.

cperciva mentioned this in D52453: release: Enable installing packages as a non-root user.Tue, Sep 9, 3:33 PM

emaste mentioned this in D52460: boot.config.5: Clarify that this controls legacy boot blocks.Tue, Sep 9, 4:07 PM

emaste added inline comments.Tue, Sep 9, 4:18 PM

release/tools/ec2-builder.conf
59	Do we have cases where we'd add `./boot/loader.conf` twice? (E.g., here and in ec2.conf)

markj added inline comments.Tue, Sep 9, 7:23 PM

release/tools/ec2-builder.conf
59	Yes. It can happen even more than that, e.g., vmimage.subr will add loader.conf entries if the root file system is on ZFS. It's harmless to have duplicate entries. I suspect the right solution is to have each cloudware variant specify its modifications declaratively rather than having code which hacks up the image in various ad-hoc ways.

cperciva accepted this revision.Tue, Sep 9, 7:46 PM

This revision is now accepted and ready to land.Tue, Sep 9, 7:46 PM

Closed by commit rG8027de008d9f: release: Prepare EC2 cloudware images to be buildable as non-root (authored by markj). · Explain WhyTue, Sep 16, 8:51 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG8027de008d9f: release: Prepare EC2 cloudware images to be buildable as non-root.

Revision Contents
Changeset List

Path

Size

release/

tools/

1 line

ec2-builder.conf

5 lines

ec2-cloud-init.conf

1 line

1 line

31 lines

Diff 162202

release/tools/ec2-base.conf

Loading...

release/tools/ec2-builder.conf

Loading...

release/tools/ec2-cloud-init.conf

Loading...

release/tools/ec2-small.conf

Loading...

release/tools/ec2.conf

Loading...