| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
| |
The V2 implentation of replication, including:
- promote-to-replica-source
- eject-replica-source
Implements: blueprint replication-v2
Change-Id: I9075365a8fae754e29d27d6e371a1d8c8980e26b
Authored-By: Morgan Jones <morgan@parelastic.com>
Co-Authored-By: Peter Stachowski <peter@tesora.com>
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Python-troveclient 1.0.8 introduced a regression where the usage
of --bypass-url results in an AuthSystemNotFound exception.
Removing some redundant code, which was due to a previous merge
conflict, fixed the issue.
Closes-bug 1416120
Change-Id: I1c949f69ee08447ce88aeefa92c60f361ab54832
|
| |/
|
|
|
|
|
|
|
|
|
|
| |
Replaced last remaining reference to TROVE_URL in python-troveclient.
Currently the user along with OS_URL , or TROVE_URL , is used as part
of a hash for a unique directory for the bash completion cache. Since
I'm not sure if there is a reason why the legacy URLs are used here,
I've replaced TROVE_URL with NOVA_URL instead of OS_AUTH_URL which
might be more appropriate for this purpose.
Change-Id: Id30aeae992aecba3c1fbe53c2ce421157410ca20
Closes-Bug: #1238282
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The datastore parameter for configuration-create is documented as
optional but is required if there is no default datastore. Changed
documentation to handle this case.
Closes-bug 1365702
Change-Id: I8d7869b77eeba8c5be510093d42ff6486f529b6d
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The correct order of parameters to assertEqual is (expected,
actual). It makes the assertion message more meaningful when there is
a failure.
Change-Id: I1ab6933d2eee336b41f6ab791e32ad6eaa004a6b
Partial-Bug: #1277104
|
| |\ \ \
| |/ /
|/| | |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| | |
While testing the osprofiler code, certain grammatical inconsistencies
were found in the help/messages.
These have been fixed.
Change-Id: Ia0b81870e15a9bc118bcaa81e698ef0e44c96f94
|
| |\ \
| |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Part of the benefit of clients using the keystoneclient adapter to
manage the session object is that we can have a standard set of kwargs
that can be controlled from the adapter. This only works if the unknown
kwargs are passed through to the adapter.
Change-Id: I78f195ef739dbfe33a08f4dc1922b14ec77ae833
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
When volume support is disabled trove list does not contain include
a volume attribute in the response. Previously we would only add a
value for size when the volume attribute was present. This change
adds a value of '-' when there is no volume attribute. This makes
the behaviour consistent with the behaviour of nova list where the
task state has no value.
Change-Id: I75e5f7c493f3d2fe938ecc8663457953c58f6b02
Closes-Bug:1394591
|
| |/ /
| |
| |
| |
| |
| |
| |
| | |
The python-troveclient module still had references to strutils, and
gettextutils which are now obsolete.
Change-Id: Ia54a565c00966b0a1cb5f18c6e17e1237b2bfee6
Partial-Bug: #1380789
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| | |
The word 'simultaneously' was incorrectly spelled. This has been fixed.
Change-Id: Id231404b09b1d26274ec9be114d0f304aab02192
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When trying to create a troveclient.v1.client.Client without specifying
endpoint_type explicitly it will raise an AuthorizationFailure exception.
This was because troveclient.v1.client.Client defaulted endpoint_type=None
which overrides the troveclient.client.HTTPClient default value of 'publicURL'
to None. Fix was to default the endpoint_type to 'publicURL' in the
troveclient.v1.client.Client __init__ method.
Also updated the documentation and README.rst for creating a
trove client object because it was out of date and incorrect.
Change-Id: I64dfbaa4eefc73d01468470dbe9eba5e209ff157
Closes-Bug: 1409982
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When a nova flavor's id is a string and not an int, Trove will
return None for the flavor's id and instead expect clients
to use the str_id field. This updates the shell mask this
from the user and always present the relevant value as id, and
allow specifying either str_id or id for flavor-show.
find_resource() was updated to deal with ints and strings, not just
UUIDs and ints.
Relies on changes to Trove @ https://review.openstack.org/#/c/115811
Related-bug: #1333852
Change-Id: If08430b07b7e8b6a1737f3e71dba6a471de63794
|
| |\ \
| |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Allow CLI to take instance name as well as instance id for
sub-commands where instance can be specified. This commit
does not include the metadata-* commands as the backend for
that feature isn't merged. The spec will be adjusted to
reflect that they commands will be out of scope.
Affected sub-commands are:
backup-create
backup-list-instance
configuration-attach
configuration-default
configuration-detach
create (just for --replica_of)
database-create
database-delete
database-list
detach-replica
delete
resize-flavor
resize-instance
resize-volume
restart
root-enable
root-show
update
user-create
user-delete
user-grant-access
user-list
user-revoke-access
user-show
user-show-access
user-update-attributes
Unit tests were added to test passing in an instance object
for the affected sub-commands. A new set of unit tests were
added for databases.py (doesn't look like any existed before).
Change-Id: I5554a7d423990ff9cb24437a893d90d212b022b1
blueprint: add-instance-name-to-cli
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Reasons:
- trove show gives different output for id and names as input,
as explained in bug, with name it shows less information.
Changes:
- Returns the object found by id after finding the matching object
from the list of obtained instances, which results in getting
same attributes for output
irrespective whichever way the command is used.
Change-Id: Ib397d5791e9a6f10c10c2ab3160401c1c7cfa214
Closes-Bug: #1384055
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Author: Swapnil Kulkarni <swapnilkulkarni2608@gmail.com>
Co-Authored-By: Nikhil Manchanda <SlickNik@gmail.com>
Co-Authored-By: Craig Vyvial <cp16net@gmail.com>
Change-Id: I250777890a1f5240c5f14290cf02eb5a7b34b434
Closes-Bug: #966329
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
To be able to create profiling traces for Trove, client should be
able to send special HTTP header that contains trace info.
This patch is as well important to be able to make cross project
traces. (Typical case horizon calls Trove via python client, if
profiler is initialized, Trove client will add extra header, that
will be parsed by special osprofiler middleware in Trove api)
Don't worry no security issue here, trace information is signed
by HMAC key that is setted in api-paste.ini. So only person that
knows HMAC key is able to send proper header.
Main patch in Trove is: I580cce8d2b3c4ec9ce625ac09de6f14e1249f6f5
We prepared a common BP in oslo-spec due to integration change is
similar to all projects: I95dccdc9f274661767d2659c18b96da169891f30
Currently there are 2 other projects are using osprofiler: Glance &
Cinder, and some others are working in progress.
Change-Id: I5a76e11d428c63d33f6d2c2021426090ebf8340c
Signed-off-by: Zhi Yan Liu <zhiyanl@cn.ibm.com>
|
| |\ \ \ |
|
| | | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This feature provides the ability to associate flavor
types with datastore versions.The trove-mange util enables
adding/deleting this information.The trove-client has been
changed to include a new api call:
/{tenant_id}/datastores/{datastore}/versions/{version}/flavors.
This call will return all nova flavors if no entries are found
for the said datastore-version in datastore_version_metadata,
otherwise only the entries found are returned.
Change-Id: I89fc2db0078d7884b7b3fa91f5daec68c0132dfb
Implements: blueprint associate-flavors-datastores
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously, if 0 is passed as the min_size or max_size for a
configuration, the Trove client would ignore it and not put it in the
dict making up the request body. This changes that code to explicitly
check if this value is not None instead of ignoring it if its zero.
Co-Authored-By: Craig Vyvial <cp16net@gmail.com>
Fixes bug: 1398995
Change-Id: I7f1d196b00a612365576bfb82fbfd35d647588d4
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Updated trove client to support Keystone V3 API. The
Keystoneclient session object is used for authentication, retrieving
the service catalog and HTTP connection/session management.
Added additional CLI parameters for Keystone V3.
Added tests using requests_mock instead of mocking completely the
session object like the previous failed patch:
https://review.openstack.org/#/c/102315
Change-Id: Ib804c0a45b3b4d3f8670ec642b2c2fd1be6cd656
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Trove CLI appears to be broken (fresh checkout, devstack as well as
openstack gate). All commands fail with
ERROR: __init__() got an unexpected keyword argument 'http_log_debug'
Correcting the obvious place where this appears to be an issue is
causing additional failures and as it is time-critical to get this
back up and running, I'm proposing a reversal of the change from
3c47a1855b7ee1345dadfb00ae90a6279b7c424e while that change is
completely reworked.
I will ping trove core for an approval on this and if I don't get an
approval, I'll fast track this reversal.
Change-Id: Icac028cfafac57c7e3ef03d4277ae2b7833bf58f
Partial-Bug: 1391840
|
| |/
|
|
|
|
|
|
|
|
|
| |
Updated trove client to support Keystone V3 API. The
Keystoneclient session object is used for authentication, retrieving
the service catalog and HTTP connection/session management.
Added additional CLI parameters for Keystone V3.
Change-Id: I800c252234be2ff11d818cf7513c61c5431eb15c
Closes-Bug: 1323866
|
| |
|
|
|
|
|
|
|
| |
the refactoring in Ie1ac42d61e920ac8ea1ebab35c2db096fd313077
broke the responses for the aforementioned commands,
resulting in stacktraces.
Change-Id: Idb6d5677cb854b9dc76ad621910277b687c111a1
Closes-Bug: #1368982
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Replace the mysql-specific term 'slave' with the more general term
'replica'.
In order to merge properly the client currently handles both 'slave' and
'replica' in certain cases. The 'slave' support will be removed completely
once the trove runtime is updated to be in sync.
Change-Id: Id22d18a84a4ac104ff8af09959e8eb2ae0102e97
Co-Authored-By: Nikhil Manchanda <SlickNik@gmail.com>
Partial-Bug: 1360310
|
| |/
|
|
|
|
|
| |
add the CRUD calls of configuration parameters mgmt calls
partially implements blueprint configuration-parameters-in-db
Change-Id: Iaaea946bdbcab91ca02e5e2cbf956a3980a461ab
|
| |
|
|
|
|
|
|
|
|
|
|
| |
adds clusters support to the troveclient.
Co-Authored-By: Ranjitha Vemula <rvemula@ebaysf.com>
Co-Authored-By: Michael Yu <michayu@ebaysf.com>
Co-Authored-By: Mat Lowery <mlowery@ebaysf.com>
Partially implements: blueprint clustering
Change-Id: I6ed2c4c79a17fcf8f14c587cab6a8ec3acaf319f
|
| |
|
|
|
|
|
|
|
|
| |
As per the current code in the client, there is no option to detach
the configuration group of the instance. This change gives a user the
option to explicitly drop the config group of the instance as part of
the instance update call.
Change-Id: I7defd110549df87afd267d3ebe998904ff6b7483
Closes-Bug: 1359359
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Add new command to detach a replica from its replication
source.
Partially Implements: blueprint replication-v1
Co-Authored-By: Nikhil Manchanda <SlickNik@gmail.com>
Change-Id: Ieca67f042c6bcb33f2a4de1acbb330d3eefc0600
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Suppress links and improve display of slave information for the 'show' and
'create' commands. Also refactor shell.py to avoid repeating the code for
displaying an instance.
The revised print_instance method also avoids bug 1322822 where the order
in which attributes are queried can affect the display output.
Depends-On: If8a154083d0095606fb3ee115cc9b66ab788cbb4
Partially-Implements: blueprint replication-v1
Partial-Bug: 1322822
Change-Id: Ie1ac42d61e920ac8ea1ebab35c2db096fd313077
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Allows datastore versions to be hidden from all users except those
explicitly allowed by their tenant. Implemented via a visibility
attribute for datastore versions (defaulting to 'public'), and a
datastore version member table for controlling access to non-public
datastore versions by tenant.
This commit provides the client functionality for integration tests
related to the datastore-visibility feature.
Implements: blueprint datastore-visibility
Docimpact: change to datastore version payload
Change-Id: Ic8024a631530ba295688d7e49e895f50cb2f32ef
|
| |
|
|
|
|
|
| |
Update help text to indicate that --size may be a required parameter.
Change-Id: If1a40475a8d4a8e1c71c2c1e61a87bacb6e42e00
Closes-Bug: 1336807
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| | |
Added 'name' to the help for the CLI commands show and flavor-show
Change-Id: I845e648cbb371bf134d98de44e7f966172dc634a
Closes-Bug: #1322814
|
| | |
| |
| |
| |
| |
| |
| | |
This change only changes the wording of a help string. It should
be 'JSON' instead of 'json'.
Change-Id: I11796e8aa6e475790838db44206e8ed85ebb1505
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Reasons:
- currect code writes list of dict as they are,
without any formatting, actually _print_
method calls str() on each attribute,
this issue spoils human-readable form of the shell output.
Changes:
- updated "secgroup-list" (`rules` attributes was ignored);
- updated "secgroup-show" (`rules` attribute was ripped off);
- proposed new call "secgroup-list-rules <secgroup-id>";
Change-Id: Ic3f51b93af106c39de7eb3238f8bcffa3ea3cb95
Closes-Bug: #1300164
DocImpact: added new CLI command.
|