[sheepdog] [PATCH] dog: add 'vdi object map' command

Hitoshi Mitake mitake.hitoshi at gmail.com
Sun Jan 26 08:21:16 CET 2014


At Thu, 23 Jan 2014 16:43:02 +0800,
Liu Yuan wrote:
> 
> This command will show inode internal 'vdi' array states, is useful to check if
> cloned VDI's inode is sane or not.
> 
> usage:
> yliu at ubuntu-precise:~/sheepdog$ dog/dog vdi object map clone1
> Index       VID
> 00000000   c8f989
> 00000001   c8f989
> 00000002   c8f989
> 00000003   7c2b25
> 00000004   7c2b25
> 00000005   7c2b25
> ...
> 
> yliu at ubuntu-precise:~/sheepdog$ dog/dog vdi object map -i 4 clone1
> Index       VID
> 00000004   7c2b25
> 
> With this patch, original 'dog vdi object' becomes 'dog vdi object location'
> 
> Signed-off-by: Liu Yuan <namei.unix at gmail.com>
> ---
>  dog/vdi.c            | 62 ++++++++++++++++++++++++++++++++++++++++++++++------
>  tests/functional/028 |  4 ++--
>  tests/functional/029 |  2 +-
>  tests/functional/035 |  2 +-
>  tests/functional/058 |  2 +-
>  5 files changed, 60 insertions(+), 12 deletions(-)

Looks good to me.
Reviewed-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>

But please update CHANGELOG.md later. Because it breaks compatibility
of dog vdi object.

Thanks,
Hitoshi



More information about the sheepdog mailing list