Btrfs devel - Feb 2013 - btrfs doesn't report proper error when removing subvolume

Hi.

petrb@bots-bnr1:/mnt$ btrfs subvolume delete ext2_saved/
Delete subvolume ''/mnt/ext2_saved''
ERROR: cannot delete ''/mnt/ext2_saved''

this is not really very descriptive. It would be really cool if it
told me why it can''t delete it.

For example:


Permission denied. Cannot delete ...
Disk write error. Cannot delete

Descriptive error messages won''t hurt ;) or is it so big deal to find
out the reason?
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

> petrb@bots-bnr1:/mnt$ btrfs subvolume delete ext2_saved/
> Delete subvolume ''/mnt/ext2_saved''
> ERROR: cannot delete ''/mnt/ext2_saved''
> 
> this is not really very descriptive. It would be really cool if it
> told me why it can''t delete it.
Hmm.  I think this has been fixed for a long time.

  [zab@f18 ~]$ btrfs sub delete /mnt/btrfs/blah/
  Delete subvolume ''/mnt/btrfs/blah''
  ERROR: cannot delete ''/mnt/btrfs/blah'' - Operation not
permitted

http://git.kernel.org/?p=linux/kernel/git/mason/btrfs-progs.git;a=commit;h=17cf679fb3622a9e3452777d07fcf3a346266ff1

Where''d you find that btrfs-progs version? :)

- z
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

hm, it''s latest ubuntu version (ubuntu 12 LTS) kernel is some 3.2.x I
think

On Wed, Feb 20, 2013 at 8:57 PM, Petr Bena <benapetr@gmail.com>
wrote:
> hm, it''s latest ubuntu version (ubuntu 12 LTS) kernel is some
3.2.x I think
>
>
> On Wed, Feb 20, 2013 at 7:13 PM, Zach Brown <zab@zabbo.net> wrote:
>>
>> > petrb@bots-bnr1:/mnt$ btrfs subvolume delete ext2_saved/
>> > Delete subvolume ''/mnt/ext2_saved''
>> > ERROR: cannot delete ''/mnt/ext2_saved''
>> >
>> > this is not really very descriptive. It would be really cool if it
>> > told me why it can''t delete it.
>>
>> Hmm.  I think this has been fixed for a long time.
>>
>>   [zab@f18 ~]$ btrfs sub delete /mnt/btrfs/blah/
>>   Delete subvolume ''/mnt/btrfs/blah''
>>   ERROR: cannot delete ''/mnt/btrfs/blah'' - Operation
not permitted
>>
>>
>>
http://git.kernel.org/?p=linux/kernel/git/mason/btrfs-progs.git;a=commit;h=17cf679fb3622a9e3452777d07fcf3a346266ff1
>>
>> Where''d you find that btrfs-progs version? :)
>>
>> - z
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html