summaryrefslogtreecommitdiff
path: root/trove
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Fix inspect.getargspec() deprecation warning"Zuul2023-04-182-2/+2
|\
| * Fix inspect.getargspec() deprecation warninglikui2023-04-182-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | DeprecationWarning: inspect.getargspec() is deprecated, use inspect.signature() or inspect.getfullargspec() [1] https://docs.python.org/3/library/inspect.html#inspect.getargspec Story: 2010694 Task: 47804 Change-Id: Ica14e66d2130dbc5949a6567673e68e8bb450061
* | Merge "Fixes permission problem when restoring backup"Zuul2023-03-221-29/+61
|\ \
| * | Fixes permission problem when restoring backupHirotaka Wakabayashi2023-01-271-29/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR changes the owner of the file used by the docker container that resets the mysql root password. Original problem is the failure to restore mysql:5.7.x backup. The error occurs because the container process tries to update the file that owned by the others. Story: 2010467 Task: 47017 Change-Id: I3d8ebaac4ee5e13af83c305ce28b51e4442d8c8b
* | | Use new get_rpc_client API from oslo.messagingTobias Urdin2023-03-061-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the new API that is consistent with the existing API instead of instantiating the client class directly. This was introduced in release 14.1.0 here [1] and added into oslo.messaging here [2] [1] https://review.opendev.org/c/openstack/requirements/+/869340 [2] https://review.opendev.org/c/openstack/oslo.messaging/+/862419 Change-Id: Ia0178e905654aaf0d05c7396867332ea5a661760
* | | Merge "Creates the mysqld extra configuration dir"Zuul2023-03-051-2/+4
|\ \ \
| * | | Creates the mysqld extra configuration dirHirotaka Wakabayashi2023-01-241-2/+4
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR makes guest-agent create the mysqld extra configuration dir. The original problem is that guest-agent fails to start mysql service with the following error message: ``` mysqld: Can't read dir of '/etc/mysql/mysql.conf.d/' (Errcode: 2 - No such file or directory) ``` Story: 2010543 Task: 47205 Change-Id: Ib1fcc64d5f27bf595d4b4c1d4e145136160bf842
* | | container_registry_password should be secretTakashi Kajinami2023-02-201-1/+2
|/ / | | | | | | | | | | | | | | | | | | The [guest_agent] container_registry_password option takes a credential to access a container registry, thus its value should be hidden from debug logs. Story: 2010594 Task: 47456 Change-Id: I8f595fff616606a96e70d1712a3a2e65c85b3476
* | Fixing tests with tox 4.2.6wu.chunyang2023-01-181-16/+29
| | | | | | | | | | | | | | | | | | | | | | 1. Remove basepython from testenv 2. this change also fix unittests because stevedore has dropped pkg_resources[0], after tox upgraded to 4.0.*, tox with usedevelop parameter installs trove in develop mode, hence stevedore unable to get the entrypoints from sdist[1]. [0]: https://opendev.org/openstack/stevedore/commit/d5297167e08468c75d2477f15004df61cf98e57e [1]: https://tox.wiki/en/2.8.1/config.html#confval-usedevelop=BOOL Change-Id: Ie65d099509e3cb7a61a09d975c58c1e95aa12b51
* | Fix the missing log request-idHa Minh Cong2022-11-281-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | This PS ensure oslo Context.update_store method is called, to ensure that thread-local context object is updated before Trove service render log Story: 2010451 Task: 46951 Change-Id: I5258853aed33cbcd26a5b6d68a5e5a4b9aa1fd48 Signed-off-by: Ha Minh Cong <conghm1@viettel.com.vn>
* | Fix deprecation warningsTakashi Natsume2022-10-1510-26/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the following deprecation warnings. * DeprecationWarning: Using the 'user' argument is deprecated in version '2.18' and will be removed in version '3.0', please use the 'user_id' argument instead * DeprecationWarning: Property 'user' has moved to 'user_id' in version '2.6' and will be removed in version '3.0' Change-Id: I18802ae1e2318eae3c62a26459178b259c728ac4 Signed-off-by: Takashi Natsume <takanattie@gmail.com>
* | Fix illegal shell characters18.0.0.0rc118.0.0wu.chunyang2022-09-144-0/+34
| | | | | | | | | | | | | | | | | | | | This change checks if the user input is legal. if illegal, we raise an InvalidValue excepiton. Story: 2010004 Task: 45128 Change-Id: Ib81646b8f8a01fcbc31d033ec205491b76a7b755
* | Fix compatibility with oslo.db 12.1.0wu.chunyang2022-09-141-0/+4
| | | | | | | | | | | | | | | | | | | | | | oslo.db 12.1.0 has changed the default value for the 'autocommit' parameter of 'LegacyEngineFacade' from 'True' to 'False'. This is a necessary step to ensure compatibility with SQLAlchemy 2.0. However, we are currently relying on the autocommit behavior and need changes to explicitly manage sessions. Until that happens, we need to override the default. Change-Id: I93ab8e0cce22a71efc08f73bde804d99a63b4a8d
* | Rename api_wsgi.py to app_wsgi.pyHirotaka Wakabayashi2022-09-021-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR changes wsgi script filename to app_wsgi.py. Original problem is that mod_wsgi script not found after devstack installation. This bug is created by #854356[1]. -- [1]: https://review.opendev.org/c/openstack/trove/+/854356 Story: 2010264 Task: 46163 Change-Id: I22883c8a8bb7e2818679d667b6868587a040a602
* | Rename app.wsgi to app_wsgi.pywu.chunyang2022-08-251-0/+1
| | | | | | | | | | | | | | | | | | | | 1. app.wsgi is a python script and should be end with py extension. 2. Generate trove-wsgi script which is needed by kolla to support https and ipv6. 3. Ignore build directory in git 4. update docs Change-Id: Ibbd6ecb8db25aa3a443d3f526f4cf6d418eb26f3
* | Fix the missing of guest-agent.conf in guest vmwu.chunyang2022-08-162-2/+7
| | | | | | | | | | | | | | | | This PS sets the injected_config_owner to ubuntu and the injected_config_group to root. Story: 2010231 Task: 46023 Change-Id: I77f9d0332a123594eae434c6e1cc795242e4022d
* | Fix docker start failed in guest-agentwuchunyang2022-08-093-3/+10
| | | | | | | | | | | | | | docker_insecure_registries is a StrOpt, but the default value is '[]' which is true for a string. Change-Id: I4b8841cc77324f274bc8210d328bc42ddef5d056
* | Merge "Adds a configuration to use a local docker registry"Zuul2022-07-202-0/+5
|\ \
| * | Adds a configuration to use a local docker registryHirotaka Wakabayashi2022-07-022-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR adds an configuration to use a local docker registry for development and test purpose. Using this option in development, * Developers can test images in local docker registry * Developers don't mind DockerHub's pull ratelimit * Developers can save bandwidth * Developers can test database quickly Story: 2010125 Task: 44793 Change-Id: I762702654bc4430b8432dd96bf869408b2947ff8
* | | Merge "Fix error when list database instances"Zuul2022-07-146-38/+54
|\ \ \
| * | | Fix error when list database instancesBo Tran2022-07-126-38/+54
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | When list database instances, you can't because we doing return with wrong self.addresses in db_info Story: #2010077 Task: #45568 Change-Id: I3275f605e03e6c74b744b29178de380ed2521425
* | | Merge "Uses userdata instead of personality files"Zuul2022-07-081-1/+1
|\ \ \ | |/ / |/| |
| * | Uses userdata instead of personality filesHirotaka Wakabayashi2022-06-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR uses userdata instead of personality files because of Nova's deprecates personality files[1] since the the NovaApi-2.57 microversion. Original problem is guest instance creation failure due to the lack of guestagent configuration files because of the recent code chage[2]. [1]: https://review.opendev.org/c/openstack/nova/+/522027 [2]: https://review.opendev.org/c/openstack/trove/+/823632 Task: 44821 Story: 2009934 Change-Id: Id6a6aae3de17c12001b1e9946a4c0ce28ff29f73
* | | Merge "Don't check task_status when creating a cluster"Zuul2022-06-061-1/+1
|\ \ \ | |/ / |/| |
| * | Don't check task_status when creating a clusterHirotaka Wakabayashi2022-04-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR allows load_simple_instance_addresse to get the ipaddr of an instance from neutron when creating a instance belongs to a cluster by skipping checking the task_status. load_simple_instance_addresses, which is a helper function to load ip addr information to Instance instances, currently checks task_status before getting the information from neutron. However, this logic is unnecessary and causes a logic error when instances belong to a cluster because the task_status of creating a cluster is determined after initializing a cluster, which happens in the next phase of load_simple_instance_addresses. The original reason of this code change is that the task status of a mariadb cluster hasn't change from BUILDING to NONE even after successfully creating instances belong to the cluster. Task: 45110 Story: 2009982 Change-Id: Id3234081fd3114e6536358cbdbf94b8280c8bc41
* | | Merge "Adds new configurations for injected file owner"Zuul2022-05-113-16/+63
|\ \ \
| * | | Adds new configurations for injected file ownerHirotaka Wakabayashi2022-05-023-16/+63
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR adds configuration options for file owner information of injected files in GuestVM to trove.conf. File owner information should not be hardcoded as "trove" because it depends on user environments. Original problem is guest instance creation failure due to the lack of guestagent configuration files because of the recent code chage[1]. That patch was introduced since Yoga. [1]:https://review.opendev.org/c/openstack/trove/+/823632 Task: 45239 Story: 2010021 Change-Id: Iaccfe84c912025ec6d96e66ffa6ad7a4b9537b6e
* | | Merge "Uses glance to get image_id when creating a cluster"Zuul2022-05-1111-19/+51
|\ \ \
| * | | Uses glance to get image_id when creating a clusterHirotaka Wakabayashi2022-04-2211-19/+51
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR uses glance to get a valid image_id when creating a cluster. Clusters API currently checks the "image_id" column in "datastore_versions" table, which has no valid image_id(NULL) since Victoria due to the following code and docs change[1]. The original problem of this patch is that nova api throws BadRequest exception with "Block Device Mapping is Invalid" message when creating a cluster. [1]: https://opendev.org/openstack/trove/commit/1d24b65052bbfb8f7dff5744c7bb44a58a2336c6 Task: 45004 Story: 2009982 Change-Id: I2a2932e261f47d9c8ec275bf9bd41b26983230c7
* | | Merge "instance model: fix extraction of ip addresses from instance record"Zuul2022-05-082-16/+15
|\ \ \ | |/ / |/| |
| * | instance model: fix extraction of ip addresses from instance recordAndrew Bogott2021-05-122-16/+15
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The instance.addresses value will be a dictionary of network: [addr_info] key/value pairs. addr_info is not itself the address but a dict that contains the address under ['addr']. The nova api changed quite a while ago so we need to catch up. In addition: this function previously returned a list of dictionaries; users of the function are clearly expecting a list of simple IPs, so return that instead. Change-Id: I9f7fed17b93e909f572949037b4e229d527e8fe4
* | Validates "network_id" param creating a clusterHirotaka Wakabayashi2022-04-092-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR changes Clusters API validator to validate a request parameter "network_id" instead of "net-id". When creating a cluster, Clusters API internally calls Instaces API to create a instance. Instances API has replaced the request parameter "net-id" with "network_id" by this commit[1]. The original problem is instance creation error when creating a instance that belongs to a cluster. Troves's Instances API has changed a parameter name from "net-id" to "network_id", but Clusters API and python-troveclient still uses the "net-id" parameter name. We need to fix it. [1]: https://opendev.org/openstack/trove/commit/535417240709694080255c064c9d84a5ab1d5372 Task: 44967 Story: 2009958 Change-Id: I7ee128afad80b92bdeabb45f719d19a99badd8a7
* | Adds docker daemon config to injected filesHirotaka Wakabayashi2022-03-232-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | This PR adds docker daemon config to injected files when docker_bridge_network_ip option is defined in trove.conf. Original problem is that Trove developers unable to transfer files between GuestVMs and Controller host when controller host address range is same with docker bridge network one. Task: 44805 Story: 2009928 Change-Id: I6ad7eb7b2a7fb0686604b69d0175ec0d06f60384
* | Merge "Removes the deprecated argument tenant from TroveContext"Zuul2022-03-201-2/+2
|\ \
| * | Removes the deprecated argument tenant from TroveContextHirotaka Wakabayashi2022-03-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tenant argument of RequestContext in oslo.context had been deprecated long time ago and it was finally removed in oslo.context-4.0.0. We should remove the tenant argument of TroveContext that derives from RequestContext and we should also update the requirements.txt in the master branch. The original problem of this patch is the Configurations API errors in following cases: * when getting a configuration. * when adding a new configuration parameter. Story: 2009933 Task: 44816 Change-Id: Idbd0507e6c87d6926f93806d0d1eef13f557d805
* | | Stop using deprecated functions in std Python libHirotaka Wakabayashi2022-03-202-2/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR stops using the following deprecated functions in std Python lib. * parser.readfp We should use parser.read_file(readline_generator(fp))[1]. * importing modules from collections directly. We should use collections.abc instead of using collections directly.[2] -- [1]: https://docs.python.org/3/library/configparser.html#configparser.ConfigParser.readfp [2]: https://github.com/python/cpython/pull/5460 Task: 44767 Story: 2009917 Change-Id: I61bf4299ad2acd8ee26b4aab66875b10287020e1
* | Removes the deprecated argument tenant from TroveContextHirotaka Wakabayashi2022-03-175-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | The tenant argument of RequestContext in oslo.context had been deprecated long time ago and it was finally removed in oslo.context-4.0.0. We should remove the tenant argument of TroveContext that derives from RequestContext and we should also update the requirements.txt in the master branch. Task: 44723 Story: 2009906 Change-Id: I69c7098cc0d61fbbba1dbf2eca87df0dd6fd70ba
* | Adapt to file injection deprecation in novaBo Tran2022-01-244-37/+91
| | | | | | | | | | | | Story: #2009770 Task: #44247 Change-Id: I34d47e709a9a7478cb6a85d6e6e37da3f92e40b0
* | Make more transparence with variable name from replica snapshotBo Tran2021-09-055-17/+22
| | | | | | | | | | | | Task: #43152 Story: #2009162 Change-Id: I469ad8b44640118883c903accfef0349e9ecb880
* | Allow regular user to get quotas16.0.0.0rc116.0.0Lingxian Kong2021-08-254-2/+93
| | | | | | | | | | | | | | | | | | The project user can query the project's own resource quota. Story: 2009140 Task: 43082 Change-Id: Iebac740e982a89fcf882a2cfc3e447ac53ee6656
* | Merge "Run reset master when setting up mysql replicas"Zuul2021-08-023-23/+43
|\ \
| * | Run reset master when setting up mysql replicasLingxian Kong2021-08-023-23/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Only run 'FLUSH PRIVILEGES;' when needed * Fix SqlClient contextmanager initialization Story: 2009091 Task: 42921 Change-Id: I05a07402388d214dc95071be709191f63af49411
* | | Merge "Replace deprecated import of ABCs from collections"Zuul2021-08-022-5/+5
|\ \ \
| * | | Replace deprecated import of ABCs from collectionsTakashi Kajinami2021-08-022-5/+5
| |/ / | | | | | | | | | | | | | | | | | | ABCs in collections should be imported from collections.abc and direct import from collections is deprecated since Python 3.3. Change-Id: Ic4c1cb4d9ce7595d1e684c26229a5148b5f4befa
* | | Show user network ID for getting instanceLingxian Kong2021-08-023-6/+89
|/ / | | | | | | Change-Id: Ia1e9112ae69e04f8c3e9e9d1b4a0189c743d7448
* | Use Block Storage API v3 instead of API v2Lingxian Kong2021-08-022-3/+3
| | | | | | | | | | | | | | | | Block Storage API v2 was deprecated during Pike cycle and is being removed during Xena cycle, and current v3 API should be used instead. Change-Id: Iac35c8a580b0e15c397cb5e78bb228fea9730f06 Ref: https://review.opendev.org/c/openstack/requirements/+/801759
* | Add periodic task to remove postgres archived wal filesLingxian Kong2021-07-2311-34/+268
| | | | | | | | | | | | | | | | | | | | | | | | * Added a periodic task for postgresql datastore to clean up the archived WAL files. * Added a check when creating incremental backups for postgresql. * A new container image ``openstacktrove/db-backup-postgresql:1.1.2`` is uploaded to docker hub. Story: 2009066 Task: 42871 Change-Id: I235e2abf8c0405e143ded6fb48017d596b8b41a1
* | Fix getting config option value for databaseLingxian Kong2021-07-143-5/+12
| | | | | | | | | | | | Change-Id: If2d3022a59ba535e1361a3ba9bc9dd983ca25256 Story: 2009051 Task: 42821
* | Merge "Allow admin user to get instances attached with any configuration"Zuul2021-07-132-8/+68
|\ \
| * | Allow admin user to get instances attached with any configurationLingxian Kong2021-07-132-8/+68
| | | | | | | | | | | | Change-Id: I5e5f0b99f79ba78c82cbb12482b9167ec69e50f7
View Lorry Controller Status