Commit b5f32138 authored by Daniel Verkamp's avatar Daniel Verkamp Committed by Jim Harris
Browse files

scripts/rpc.py: fix construct_aio_bdev block_size param 



Block size is supposed to be optional, but it was missing the nargs
setting that makes positional parameters actually optional.

Change-Id: I8324caa956d8eaecd3ce207c9296151512b6ebea
Signed-off-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-on: https://review.gerrithub.io/411942


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent 4a534344

scripts/rpc.py

+1 −1
Original line number Diff line number Diff line
@@ -135,7 +135,7 @@ if __name__ == "__main__": 
                              help='Add a bdev with aio backend')

    p.add_argument('filename', help='Path to device or file (ex: /dev/sda)')

    p.add_argument('name', help='Block device name')

    p.add_argument('block_size', help='Block size for this bdev', type=int, default=argparse.SUPPRESS)

    p.add_argument('block_size', help='Block size for this bdev', type=int, nargs='?', default=0)

    p.set_defaults(func=construct_aio_bdev)



    @call_cmd