fix: Naming/defaults of new vfolder args (#137)

* `allow_clone` error: Reorder the double-dashed cli arguments to let Click mangle the correct parameter name
* `quota` type error: Always provide a valid binary-size string, with the "unconstrained" default zero

Author: achimnol

changes/137.fix

@@ -0,0 +1 @@
+Fix naming and defaults of new vfolder creation parameters `cloneable` and `quota` (added in APIv6) for both the functional SDK and the CLI arguments

src/ai/backend/client/cli/vfolder.py

@@ -92,11 +92,11 @@ def list_allowed_types():
               help='Folder\'s innate permission. '
                    'Group folders can be shared as read-only by setting this option to "ro".'
                    'Invited folders override this setting by its own invitation permission.')
-@click.option('-q', '--quota', metavar='QUOTA', type=ByteSizeParamCheckType(), default=None,
+@click.option('-q', '--quota', metavar='QUOTA', type=ByteSizeParamCheckType(), default='0',
               help='Quota of the virtual folder. '
                    '(Use \'m\' for megabytes, \'g\' for gigabytes, and etc.) '
                    'Default is maximum amount possible.')
-@click.option('--allow-clone', '--cloneable', type=bool, is_flag=True,
+@click.option('--cloneable', '--allow-clone', type=bool, is_flag=True,
               help='Allows the virtual folder to be cloned by users.')
 def create(name, host, group, host_path, usage_mode, permission, quota, cloneable):
     '''Create a new virtual folder.

src/ai/backend/client/func/vfolder.py

@@ -37,7 +37,7 @@ async def create(
         group: str = None,
         usage_mode: str = 'general',
         permission: str = 'rw',
-        quota: str = None,
+        quota: str = '0',
         cloneable: bool = False
     ):
         rqst = Request('POST', '/folders')