delta/openstack/python-cinderclient.git/cinderclient, branch master

Merge "Add test coverage for shell snapshot-create w/ metadata"

2023-02-13T19:26:47+00:00

Merge "Add support for collect-timing option"

2023-02-13T18:53:56+00:00

Merge "remove simplejson requirement"

2023-02-10T20:11:04+00:00

Merge "Move print operations to shell_utils"

2023-01-26T18:13:28+00:00

remove simplejson requirement

2023-01-26T14:30:35+00:00

The json lib has been included in CPython for
ages, so the ImportError case never happens.

Change-Id: I669bc397323a0e83d4a9768b124a7d99bafaf38a

Python3.11: fix crashes in ShellTest

2022-12-21T13:31:33+00:00

In Python3.11, global flags must be placed right at the start of a
regular expression.

The following regex:

	r'.*?(?m)^Lists all volumes.',
must become:

        r'(?m).*?^Lists all volumes.',

However, since we are using re.MULTILINE, we actually do not need to use
a global flag.

This commit fixes the following tests in Python3.11:
- cinderclient.tests.unit.test_shell.ShellTest.test_help_arg_no_subcommand
- cinderclient.tests.unit.test_shell.ShellTest.test_help
- cinderclient.tests.unit.test_shell.ShellTest.test_help_on_subcommand
- cinderclient.tests.unit.test_shell.ShellTest.test_help_on_subcommand_mv

Closes-Bug: #1983047
Change-Id: If20abef109ddd7107c83b5886beb666a6550a640

Merge "Python3.11: Fix argparse-related test failures"

2022-12-12T23:13:37+00:00

Add test coverage for shell snapshot-create w/ metadata

2022-11-17T18:32:11+00:00

This path is not currently unit tested.

Change-Id: If38c6352f5e1f0a50e4a0d29fbdd5263ccba3b29

Move print operations to shell_utils

2022-11-15T15:12:41+00:00

Move more code to shell_utils that is only used
for shell operations.

The benefit of this is that the cinderclient library
becomes lighter-weight, because users of the lib
no longer have to import prettytable and extra code.

Change-Id: I7bf6bd91ee5746d1ad4bd4504f3a056d03ae86a9

Handle downgraded client for snapshot-create

2022-11-08T23:48:18+00:00

When a CLI user specifies --os-volume api-version 3.66, the shell
will execute the appropriate shell code, but if the server only
supports < 3.66, the client is automatically downgraded and correctly
uses the pre-3.66 SnapshotManager.create() method.

In that case, the 'force' parameter, which is technically not allowed
in mv 3.66 (but which silently accepts a True value for backward
compatibility), will have a value of None, which the pre-3.66 code
happily passes to cinder as '"force": null' in the request body, and
which then fails the Block Storage API request-schema check.

Handle this situation by detecting a None 'force' value and setting
it to its pre-3.66 default value of False.

Change-Id: I3ad8283c2a9aaac58c8d2b50fa7ac86b617e5dd3
Closes-bug: #1995883