Libguestfs - Feb 2015 - [RFC] expand options of btrfs-set-seeding

Hi Rich,

	Currently we had btrfs-set-seeding for btrfstune.

	But btrfstune had more options:
	usage: btrfstune [options] device
	-S value	positive value will enable seeding, zero to disable, negative is not
allowed
	-r 		enable extended inode refs
	-x 		enable skinny metadata extent refs
	-f 		force to clear flags, make sure that you are aware of the dangers

	If I want to add more options for btrfstune:
	1) expand current btrfs-set-seeding command
	2) add new btrfs-tune-XXX which cover current btrfs-set-seeding
	3) add new btrfs-tune-XXX as supplyments for btrfs-set-seeding

	Which style do you prefer?

Regards,

- Chen

On Tue, Feb 17, 2015 at 07:09:37AM +0000, Chen, Hanxiao
wrote:
> Hi Rich,
> 
> 	Currently we had btrfs-set-seeding for btrfstune.
> 
> 	But btrfstune had more options:
> 	usage: btrfstune [options] device
> 	-S value	positive value will enable seeding, zero to disable, negative is
not allowed
> 	-r 		enable extended inode refs
> 	-x 		enable skinny metadata extent refs
> 	-f 		force to clear flags, make sure that you are aware of the dangers
> 
> 	If I want to add more options for btrfstune:
> 	1) expand current btrfs-set-seeding command
> 	2) add new btrfs-tune-XXX which cover current btrfs-set-seeding
> 	3) add new btrfs-tune-XXX as supplyments for btrfs-set-seeding
> 
> 	Which style do you prefer?
I think it will be less surprising for users if the libguestfs API
more closely follows the upstream btrfs commands (ie. options 2 and/or 3).

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-builder quickly builds VMs from scratch
http://libguestfs.org/virt-builder.1.html