This is very good.

A few comments :
- The dependency on filesystem/local may not be sufficient if the
   VM's data are on a NFS filesytem for example. You might want to
   mimic what is done for Zones and add a dependency on
   milestone/multi-user-server

- You don't check for the validity of the vm/stop_method property.
   You could either do it within the method or use the Value Constraints
   from the new feature recently introduced : Template Extensions[1]

Regards,
-- Renaud

[1] http://opensolaris.org/os/project/vpanels/templates/

Alexandre Dumont wrote:
> Hi,
> 
> I've written a manifest and method to start Virtualbox headless VMs at 
> boot time. Both are explained in details at 
> http://adumont.serveblog.net/2009/07/21/virtualbox-smf/. Feel free to 
> comment me any further enhancement you may think of. Hope you find it 
> useful.
> 
> I'll be glad to have you publish it to "Converted services: manifests 
> and methods" (http://www.opensolaris.org/os/community/smf/manifests/), 
> should you think it deserves it of course.
> 
> Best regards,
> Alex
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> smf-discuss mailing list
> smf-discuss at opensolaris.org

Reply via email to