Differential D59490

devel/libvirt: run virt-host-validate in rc script
ClosedPublic
Actions

Authored by novel on Mon, Sep 7, 4:48 PM.

Tags

None

Referenced Files

	F174595675: D59490.id186217.diff
	Sun, Oct 4, 1:11 PM

	Unknown Object (File)
	Tue, Sep 29, 3:41 PM

	Unknown Object (File)
	Fri, Sep 25, 11:33 PM

	Unknown Object (File)
	Thu, Sep 24, 6:09 AM

	Unknown Object (File)
	Thu, Sep 24, 5:53 AM

	Unknown Object (File)
	Thu, Sep 24, 4:44 AM

	Unknown Object (File)
	Thu, Sep 24, 12:48 AM

	Unknown Object (File)
	Mon, Sep 21, 11:07 PM

View All 44 Files

Subscribers

None

Details

Reviewers

Commits

R11:44dc35be3ca6: devel/libvirt: run virt-host-validate in rc script

Summary

Update the libvirtd rc script to run virt-host-validate(1)
before running libvirtd(8). This command provides information
about possibly missing kernel modules required for certain
features of libvirtd. Errors and warnings from this tool do not
prevent libvirt(8) from starting as most of the features
are optional.

Suggested by: arrowd
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository

R11 FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

novel requested review of this revision.Mon, Sep 7, 4:48 PM

novel created this revision.

Harbormaster completed remote builds in B76638: Diff 186143.Mon, Sep 7, 4:48 PM

novel added a reviewer: arrowd.Mon, Sep 7, 4:48 PM

arrowd added inline comments.Mon, Sep 7, 4:56 PM

devel/libvirt/files/libvirtd.in
52	Since we're ignoring the return code here, why make the whole check conditional?

novel added inline comments.Mon, Sep 7, 5:20 PM

devel/libvirt/files/libvirtd.in
52	For people who do not want extra noise on their console or logs.

LGTM, thanks.

This revision is now accepted and ready to land.Mon, Sep 7, 5:26 PM

Closed by commit R11:44dc35be3ca6: devel/libvirt: run virt-host-validate in rc script (authored by novel). · Explain WhyTue, Sep 8, 4:05 PM

This revision was automatically updated to reflect the committed changes.

novel added a commit: R11:44dc35be3ca6: devel/libvirt: run virt-host-validate in rc script.

Revision Contents
Changeset List

Path

Size

devel/

libvirt/

1 line

files/

19 lines

Diff 186217

devel/libvirt/Makefile

Loading...

devel/libvirt/files/libvirtd.in

Loading...