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

Liu Yuan namei.unix at gmail.com
Mon Jan 27 04:46:05 CET 2014


On Sun, Jan 26, 2014 at 04:21:16PM +0900, Hitoshi Mitake wrote:
> 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>
> 

Applied this patch, thanks

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

Okay, will do later.

Thanks
Yuan



More information about the sheepdog mailing list