Page MenuHomeFreeBSD
Differential D25414

bhyve: allow for automatic destruction on power-off
ClosedPublic
Actions

Authored by kaktus on Jun 23 2020, 5:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 11:22 PM
Unknown Object (File)
Tue, Apr 16, 12:35 PM
Unknown Object (File)
Wed, Mar 27, 10:57 PM
Unknown Object (File)
Dec 23 2023, 3:15 AM
Unknown Object (File)
Dec 22 2023, 2:47 PM
Unknown Object (File)
Sep 7 2023, 5:32 AM
Unknown Object (File)
Jul 3 2023, 4:51 PM
Unknown Object (File)
May 6 2023, 10:23 PM
View All 11 Files
Subscribers
0mp
bcran
grehan
imp
rgrimes

Details

Reviewers
swills
grehan
0mp
kib
Group Reviewers
bhyve
manpages
Commits
rS362612: bhyve: allow for automatic destruction on power-off
Summary

Introduce -D flag that allows for the VM to be destroyed on guest initiated power-off by the bhyve(8) process itself.
This is quality of life change that allows for simpler deployments without the need for bhyvectl --destroy.

Requested by: swills

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 31910

Event Timeline

kaktus created this revision.Jun 23 2020, 5:58 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptJun 23 2020, 5:58 PM
kaktus requested review of this revision.Jun 23 2020, 5:58 PM
Harbormaster completed remote builds in B31900: Diff 73531.Jun 23 2020, 5:58 PM
grehan accepted this revision.Jun 23 2020, 8:35 PM
grehan added a subscriber: grehan.

Nice change. Don't forget to update the man page.

This revision is now accepted and ready to land.Jun 23 2020, 8:35 PM
kaktus updated this revision to Diff 73551.Jun 23 2020, 10:04 PM
kaktus added a reviewer: manpages.

Update man page.

@grehan happy for a MFC in say 2 weeks after commit to head?

This revision now requires review to proceed.Jun 23 2020, 10:04 PM
Harbormaster completed remote builds in B31910: Diff 73551.Jun 23 2020, 10:04 PM
grehan added a comment.Jun 23 2020, 10:06 PM

happy for a MFC in say 2 weeks after commit to head?

Yep, sounds good.

0mp accepted this revision.Jun 23 2020, 10:12 PM
0mp added a subscriber: 0mp.

LGTM when it comes to the manual.

Also, maybe Relnotes: yes?

This revision is now accepted and ready to land.Jun 23 2020, 10:12 PM
swills added a comment.Jun 24 2020, 3:48 PM

This is a really great, thank you so much! Overall, this looks very good, only noticed one minor question.

usr.sbin/bhyve/bhyverun.c
807

Not sure 1 still makes sense in this case. It's not indicating a failure or issue of some kind, is it?

swills accepted this revision.Jun 24 2020, 4:59 PM
kaktus added a comment.Jun 24 2020, 7:35 PM

As mentioned on irc, I believe we're bound by the already defined exit codes, as describes in section EXIT STATUS of bhyve(8).

kib accepted this revision.Jun 25 2020, 11:44 AM
Closed by commit rS362612: bhyve: allow for automatic destruction on power-off (authored by kaktus). · Explain WhyJun 25 2020, 12:35 PM
This revision was automatically updated to reflect the committed changes.
kaktus added a commit: rS362612: bhyve: allow for automatic destruction on power-off.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
6 lines
13 lines

Diff 73551

View Options

usr.sbin/bhyve/bhyve.8

Loading...
View Options

usr.sbin/bhyve/bhyverun.c

Loading...