vmrun: add -x flag to disable reboot loop

[rgrimes]

Summary

This adds a -x flag to allow the reboot loop of bhyve(8) to be managed by an external process rather than internally by vmrun.sh

Submitted By: Paul Vixie

[rgrimes]

I do not believe this is a desirable change, at least it does not do what it claims it does.

share/examples/bhyve/vmrun.sh

386

Man pages sayes this is reboot, not sure what the differences are yet. I believe it is actually reset, and you have to do the reboot externally by re running the bhyve(8) command.

389

There are now exit code of 4, "exited due to an error"

392

The only case that this new code would trigger on would be if the bhyve exit code is 0, and -x was specified. How does that add any exit information?

Furthermore that would break the reboot while [ 1 ] loop, is that what your trying to achive?

[rgrimes]

To add information, I have email from Paul Vixie and in fact he is wanting to break the reboot loop with -x so that it can be handled externally.

[araujo]

Personally I'm not a vmrun.sh user, but I have no objection related with this change and I don't see anything harmful.

[rgrimes]

Allan, would you update the diff please

share/examples/bhyve/vmrun.sh

98

Per emails with Paul Vixie we have come up with this:

-x Exit reboot loop even if bhyve exits with reboot code

Allan, can you update the diff please

128

We should probably call this exit_rebootloop, passthrough has a great deal of other context in bhyve.

[araujo]

Hi @allanjude is there anything pending to move this patch forward?

[allanjude]

I am back at home now, so I'll get this refreshed

[markj]

@allanjude ping

[rgrimes]

@allanjude I know your work load is huge as well, if you need me to push a new diff I can do that, but then you have to agree to commit it or I have to pester bde/phk for an approval.

share/examples/bhyve/vmrun.sh

128

please do make this exit_rebootloop, no doubts in my mind now as to what is going on here, and that this should not be called passthrough due to confusion factor.

386

Above statement is correct, we should probably fix up the man page to more clearly explain the exit code 0

392

Adding a note, to above, yes this is trying to break the reboot while [1] loop, ie you want to manage reboots in a loop outside bhyve(8).

[allanjude]

@rgrimes If you want to commandeer this review and put an updated diff, that would be great.

Note: this was originally submitted by Paul Vixie, it is not my work.

I am happy to commit it, although you are now release from mentorship if you want to commit it yourself, although remember to credit 'Paul Vixie (original version)'

[bcran]

I'm going through my list of reviews and noticed this one. Does it need more work, or can it be committed?

[jhb]

share/examples/bhyve/vmrun.sh

98

How about just "disable auto-reboot"

389

4 already falls into the "all other" comment

392

And, since the script uses 'exit $bhyve_exit' it does pass through a different exit value to the caller when -x is used (the caller would never see 0 otherwise).