On Thu, 2017-08-17 at 10:39 +0100, Radostin Stoyanov wrote:
> Resove an issue when only uid_map or only gid_map is specified.
typo: 'Resolve'
>
> We set the default values to None. However, `len(None)` will raise
> an exception. To avoid this we need to set the default values to
> [] (empty list).
> ---
> src/virtBootstrap/virt_bootstrap.py | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/src/virtBootstrap/virt_bootstrap.py
> b/src/virtBootstrap/virt_bootstrap.py
> index 0bc2e2b..f970838 100755
> --- a/src/virtBootstrap/virt_bootstrap.py
> +++ b/src/virtBootstrap/virt_bootstrap.py
> @@ -120,6 +120,12 @@ def bootstrap(uri, dest,
> logger.error("No write permissions on destination path '%s'", dest)
> sys.exit(1)
>
> + if uid_map is None:
> + uid_map = []
> +
> + if gid_map is None:
> + gid_map = []
> +
> source(uri=uri,
> fmt=fmt,
> username=username,
ACK with the typo fixed
--
Cedric
_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list
