mention server migrate is 'offline', and mention live migration and how
it is different.
Change-Id: Ia9af8e4a21989c39deaf2f6f7e1039b3fc4a0eef
Signed-off-by: Pavlo Shchelokovskyy <[email protected]>
This patch adds the '--cluster' optional argument to the 'volume
migration' command. This allows users to migrate volumes to a
destination cluster instead of a specific host, which is particularly
useful in Active-Active configurations.
The '--cluster' option requires Cinder API microversion 3.16 or
higher. The '--host' and '--cluster' options are mutually exclusive;
one of them must be provided for the migration to start.
Change-Id: Ibd45ac35e39a2a9f88a39f8041c06c4469727098
Signed-off-by: hongp <[email protected]>
SDK returns generators, not lists, and we do not see errors until
they are iterated. Force early iteration by wrapping then in a list
to ensure we actually see the HTTP 403 errors we were expecting.
Change-Id: I0ab72e587bf4e16ae877db7a81023a226124e4d5
Signed-off-by: Stephen Finucane <[email protected]>
Nothing was setting haz_v1_api which means it was not possible
for these to run. Even if it had been, getting a cloud with the
v1 image API up and running is no easy feat nowadays.
Change-Id: Ic2ba82aedae4dbcae6313150335ee1c7412ce7b3
Signed-off-by: Stephen Finucane <[email protected]>
testtools was recently bumped to a version that is typed, which means we
now see type hints for that library. As a result, we now see issues with
variables defined in tests via setUpClass. Note that the issue lies with
mypy and not testtools, however: mypy doesn't adding class variables via
assignment [1].
[1] https://github.com/python/mypy/issues/8723
Change-Id: I8b09846ff8fc6ee51ba03531f5b41039282ee1c0
Signed-off-by: Stephen Finucane <[email protected]>
This avoids the need to duplicate our dependency list in multiple
places and allows us to take advantage of tox's dependency management
infrastructure, to ensure we always get the latest and greatest version
of a package allowed by upper-constraints.
While here, we also update the versions of the remaining pre-commit
hooks and change the indentation of the tox file to align with the two
indent spacing used for other SDK projects. This makes copy-pasting
easier.
Change-Id: Ibde8ecda673b2346c82aab68d4f4b49be08414ae
Signed-off-by: Stephen Finucane <[email protected]>
This flag is called "cascade" in the Cinder API.
The flag "purge" doesn't really communicate an obvious
meaning in the context of volume deletion. It also
has the danger of implying some kind of behavior about
volume wiping that does not exist.
Rename this flag to "--cascade" and preserve "--purge"
as a hidden flag for compatibility.
Change-Id: I8de27811222c17155697073fb9c512746d009266
Signed-off-by: Eric Harney <[email protected]>
Co-authored-by: Stephen Finucane <[email protected]>
In order for python-openstackclient to support image imports to
mutliple stores at the same time an update is needed to the --all-stores
argument used by the client whereby the argument is explicitly set to contain
a boolean value. This change makes the argument do what it's supposed to and
do it in a way consistent with the API contract exposed by Glance.
Amend tests to support the change in type of the --all-stores option.
Change-Id: If5a72ca3ca68656555b5eb478e104d43f419c77e
Closes-Bug: 2138903
Signed-off-by: Piotr Sipika <[email protected]>
Co-authored-by: Stephen Finucane <[email protected]>
When creating or listing projects, search for parent project in the
same domain as the child project.
Change-Id: I1912f06df353a64eb0573f080ac9ab067cb90632
Signed-off-by: 0weng <[email protected]>
Fix `openstack quota show --usage` to correctly display resource usage
and reservations by applying proper name normalization for corresponding
sections of data.
Previously, name normalization was applied only for "limits" which is
the root section, leaving 'usage' and 'reservation' sections untouched.
Change-Id: Id14fe894b30a74b9b8d78b00c3d4ff151f8b4210
Closes-bug: #2137636
Signed-off-by: Andriy Kurilin <[email protected]>
When using 'openstack image set --project <project> --accept <image>',
the command incorrectly changed the image owner. The --project parameter
when used with membership flags should only identify which member's
status to update, not change ownership.
Closes-Bug: #2136795
Change-Id: I1044b51f38000fb5339740bc40c7f8645c794402
Signed-off-by: Abhishek Kekane <[email protected]>
When a user is created without a password then no parameter called
'password' should be submitted to the keystone API. This removes the
incorrect 'password' with null being supplied.
Closes-Bug: 2136148
Change-Id: If1c2eb5db360764a5f7660ce4e5353da85b6d3da
Signed-off-by: Doug Goldstein <[email protected]>
In change I53d9058273748ecd4d4eecec5f7291d5f38ce5ab we added custom
Command classes for typing purposes. However, the Tap-as-a-Service code
merged around the same time and was missed. Correct this.
Change-Id: I3a9fe20b5b8eb54708644527538f27396f29b476
Signed-off-by: Stephen Finucane <[email protected]>
Even though the comment here attributed this to stevedore, it was in
fact the use of pkg_resources that changed things.
Change-Id: I35377dd7d773024aa6423b72b1412e11b1b6f2e4
Signed-off-by: Stephen Finucane <[email protected]>
Change Ibe1cd6461d2cb78826467078aa17272f171746aa removed support for the
v1 volume API. We should have removed this check at the same time.
We also remove some god-awful monkey patching that references v1
cinderclient but in practice modified all clients.
Change-Id: I3727fd9238df966b3bc59812c5efcf3398da5c72
Signed-off-by: Stephen Finucane <[email protected]>
We make a lot of use of typing.Any just to get this over the line. We
can come back to this later.
Change-Id: I03c18b0b44f210b2ad3e4012344d521fb85cae97
Signed-off-by: Stephen Finucane <[email protected]>
These are effectively identical to the osc-lib variants except they
include the attributes that the OSC shell implementation will set on
this during shell init. This helps from a typing perspective.
Change-Id: I53d9058273748ecd4d4eecec5f7291d5f38ce5ab
Signed-off-by: Stephen Finucane <[email protected]>
