summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add local bindep.txtstable/rockyElod Illes2019-10-211-0/+9
| | | | | | | | | | | | | As it was announced [1] global bindep-fallback.txt was removed and now projects need to have a local bindep.txt to be able to install binary dependencies for testing. In test jobs the script tools/test-setup.sh is called which requires mysql and postgres servers and clients to be installed. [1] http://lists.openstack.org/pipermail/openstack-discuss/2019-June/007272.html Change-Id: I76df1464d7bd7af76afa619766c1721afc1d169e
* OpenDev Migration PatchOpenDev Sysadmins2019-04-193-9/+9
| | | | | | | | | | | | | | | This commit was bulk generated and pushed by the OpenDev sysadmins as a part of the Git hosting and code review systems migration detailed in these mailing list posts: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html Attempts have been made to correct repository namespaces and hostnames based on simple pattern matching, but it's possible some were updated incorrectly or missed entirely. Please reach out to us via the contact information listed at https://opendev.org/ with any questions you may have.
* Replace openstack.org git:// URLs with https://Ian Wienand2019-03-244-6/+6
| | | | | | | | | | | | | | | | | This is a mechanically generated change to replace openstack.org git:// URLs with https:// equivalents. This is in aid of a planned future move of the git hosting infrastructure to a self-hosted instance of gitea (https://gitea.io), which does not support the git wire protocol at this stage. This update should result in no functional change. For more information see the thread at http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html Change-Id: I93816d96eccf37d76aadcfa1a31dfd5d94f87264
* import zuul job settings from project-configDoug Hellmann2018-09-091-17/+29
| | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to complete step 1 of moving the zuul job settings out of project-config and into each project repository. Because there will be a separate patch on each branch, the branch specifiers for branch-specific jobs have been removed. Because this patch is generated by a script, there may be some cosmetic changes to the layout of the YAML file(s) as the contents are normalized. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I1105ddc52952b4ef9b559b5b82bc052c574790d1 Story: #2002586 Task: #24342
* Update UPPER_CONSTRAINTS_FILE for stable/rockyOpenStack Release Bot2018-08-091-1/+1
| | | | | | | | | | | | | | | | | | The new stable upper-constraints file is only available after the openstack/requirements repository is branched. This will happen around the RC1 timeframe. Recheck and merge this change once the requirements repository has been branched. The CI system will work with this patch before the requirements repository is branched because zuul configues the job to run with a local copy of the file and defaults to the master branch. However, accepting the patch will break the test configuration on developers' local systems, so please wait until after the requirements repository is branched to merge the patch. Change-Id: Ib983dead60556d060e76bc554c92f049ea30102c
* Update .gitreview for stable/rockyOpenStack Release Bot2018-08-091-0/+1
| | | | Change-Id: Ia829c651f43b58b73c31ea6cfaa01aee63c4bffb
* Fix replication failure when Swift isn't availablerocky-em10.0.0.0rc110.0.0Dariusz Krol2018-08-026-1/+30
| | | | | | | | | | | | | | | * add Swift token verification in create instance method in Trove API when slave_id is given * catch ConnectionError exception in verify_swift_auth_token * add Swift token verification to guestagent before backup and restore * add new exception representing Swift connection error * set missing fault information when replication snapshot fails * mock verify auth token method in replication and restore unit tests Closes-Bug: #1395523 Change-Id: I6a21ba2ba890a82875f9b6dae3c6b93bc9fdb4b0 Signed-off-by: Dariusz Krol <d.krol@samsung.com>
* Merge "More reliable gpg keys importing in DIB elements"Zuul2018-07-268-90/+44
|\
| * More reliable gpg keys importing in DIB elementsZhao Chao2018-07-168-90/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're still seeing apt-key failed to import gpg keys these days during the images building in the gate jobs, the problem is keys.gnupg.net and keyserver.ubuntu.com are both not stable according to [1] and [2], it's better to adopt pool.sks-keyservers.net instead and with simple retries. To reduce code duplication, this common apt-key importing function is also moved to ubuntu-guest as an environment snippet. [1] https://www.gnupg.org/faq/gnupg-faq.html#new_user_default_keyserver [2] https://sks-keyservers.net/overview-of-pools.php Closes-Bug: #1579094 Change-Id: I0fe200d140f6f9c4d423dd498797a225e3295a71 Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | Merge "Sync the data fields of DB* class and table fields."10.0.0.0b3Zuul2018-07-239-46/+63
|\ \
| * | Sync the data fields of DB* class and table fields.zhanggang2018-07-179-46/+63
| |/ | | | | | | | | | | | | | | | | | | | | | | This update try to fix some problems: 1. define the 'id' both in _data_fields and _auto_generated_attrs. 2. some of tables don't have the 'id' field. 3. _data_fields of DBInstance miss some table fields. 4. Change the key of persisted_models as the same of table name to make it more readable. Change-Id: I0ad96824e8de978ff5b6766085549b1e1a2509cf Signed-off-by: zhanggang <zhanggang@cmss.chinamobile.com>
* | Merge "Migrate to Zuul v3 native job definitions"Zuul2018-07-2345-1730/+168
|\ \
| * | Migrate to Zuul v3 native job definitionsZhao Chao2018-07-1745-1730/+168
| |/ | | | | | | | | | | | | | | | | | | | | | | | | This patch migrates the legacy gate jobs to Zuul v3 native ones(a new trove-devstack role is created for all those legecy jobs, and the previous way of running gate jobs is still kept as we still cannot migrate the genade job to a Zuul v3 native one), following the documents below: https://docs.openstack.org/infra/manual/zuulv3.html https://docs.openstack.org/devstack/latest/zuul_ci_jobs_migration.html Change-Id: I10a1adae3a79b7322118fce180119f0ee5ff2e1b Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | Merge "[doc] Use openstack client command to replace others"Zuul2018-07-201-7/+1
|\ \
| * | [doc] Use openstack client command to replace othersQian Min Chen2018-07-121-7/+1
| | | | | | | | | | | | | | | | | | | | | | | | The neutron client commands will be deprecated, and 'nova net-list' has already been deprecated, so use openstack client commands to replace neutron client. Change-Id: Idb7d9782bb3af28cba7e318b0ed664641b5d731f
* | | Merge "Update auth_uri option to www_authenticate_uri"Zuul2018-07-193-3/+3
|\ \ \
| * | | Update auth_uri option to www_authenticate_uricaoyuan2018-04-113-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Option auth_uri from group keystone_authtoken is deprecated[1]. Use option www_authenticate_uri from group keystone_authtoken. [1]https://review.openstack.org/#/c/508522/ Change-Id: I46bb0acaf1dec9bb4b91d4a2180b3e7ed66741c2
* | | | Merge "Trivial: Update pypi url to new url"Zuul2018-07-192-5/+5
|\ \ \ \ | |_|_|/ |/| | |
| * | | Trivial: Update pypi url to new urlTovin Seven2018-04-202-5/+5
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | Pypi url changed from [1] to [2] [1] https://pypi.python.org/pypi/<package> [2] https://pypi.org/project/<package> Change-Id: I3470831e34b3bbeb2fb028b71312f51e5224b502
* | | Merge "py3.x: Fix usage of gettext.install"Zuul2018-07-125-14/+10
|\ \ \
| * | | py3.x: Fix usage of gettext.installCorey Bryant2018-06-195-14/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This needed to be updated to get py3.x to work. Change-Id: I987854c0266566e1e7722478dbb3e27bcffbb08b Closes-Bug: #1777440
* | | | Merge "Raise timeout for instance resizing checking job"Zuul2018-07-112-1/+15
|\ \ \ \
| * | | | Raise timeout for instance resizing checking jobZhao Chao2018-07-092-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Recently the trove-functional-mysql usually failed by timeout at the test_instance_returns_to_active_after_resize case, and the cause is creating and resizing instances could be quit slow(e.g. 1200+ seconds). This may be caused by the pressure of the CI system and should be further investigated. The patch is a quick fix for the problem by raising the timeout to 30 minutes(it's now 900 seconds by default, but instead it's 32 minutes for instance creating, so raising the value to 30 minutes should be reasonable). Partial-Bug: #1778837 Change-Id: Ib903a72b098499553978ec881843a7bbee25b45f Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | | | | Merge "Add volume_type to apischema"Zuul2018-07-101-1/+1
|\ \ \ \ \
| * | | | | Add volume_type to apischemazhanggang2018-06-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, trove had support pass volume_type during creation of instance/cluster, but the schema "volume" only contains the "size", it is nesscessary add the "type" to schema "volume". Besides, there is a "required: True" under "properities", it is unnecessary. Change-Id: Id7c4317b0ba62d751956ddb1baee70fe27bc15f4 Signed-off-by: zhanggang <zhanggang@cmss.chinamobile.com>
* | | | | | Merge "Fix invalid escape sequence warnings"Zuul2018-07-0326-62/+64
|\ \ \ \ \ \
| * | | | | | Fix invalid escape sequence warningsKrzysztof Opasiak2018-07-0226-62/+64
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Starting with Python 3.6, invalid escape sequences in string literals are now deprecated[1]. This influence also automatic style checkers like pycodestyle which starting with 2.4.0 complains about invalid escape sequences (W605)[2]. Let's fix all those warnings at once by using raw strings where possible and adding additional \ where not. Footnotes: 1 - https://docs.python.org/3/whatsnew/3.6.html#deprecated-python-behavior 2 - https://github.com/PyCQA/pycodestyle/pull/676 Change-Id: I009a366fd8342edfd30890df6fe8e1fca88bf3cc Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>
* | | | | | Merge "change pylint wrapper to let messages be prefixes"Zuul2018-06-291-6/+7
|\ \ \ \ \ \
| * | | | | | change pylint wrapper to let messages be prefixesDoug Hellmann2018-06-131-6/+7
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The messages for many of the errors reported by pylint have been extended to include "suggestions" (especially in the case of undefined variables or members). Rather than update all of the individual messages in the configuration file, just treat the messages as prefixes. Change-Id: I1f11c50e39dbd43c4a9b54ccaaccf03f5ff07aca Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* | | | | | Merge "Cleanup testrepository and os-testr requirements"Zuul2018-06-293-6/+2
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Cleanup testrepository and os-testr requirementsZhao Chao2018-04-183-6/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have migrated to stestr for a while, however the related requirements is not updated, this patch does the cleanup, also updates requirements for stestr. Change-Id: I9c8cf2f90ced2261293f91faefc3099740540fe7 Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | | | | | Merge "Update Trove's README"Zuul2018-06-201-11/+29
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Update Trove's READMEzhanggang2018-06-131-11/+29
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the README.rst, change the repo url from github to git.openstack.org, add link for trove-sepcs and trove-dashboard, add link for CONTRIBUTING.rst and HACKING.rst. Signed-off-by: zhanggang <zhanggang@cmss.chinamobile.com> Change-Id: If1c7d3467c7da261ac626325a44aa7eab37ae042
* | | | | Remove pycrypto from requirementsFan Zhang2018-06-146-8/+4
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove pycrypto completely from requirements and fix the related tests. Closes-Bug: #1749574 Depends-On: I5c0c1a238023c116af5a84d899e629f1c7c3513f Change-Id: Ibfedd9e2ab0a5e78959108112f57103a089f02d1 Signed-off-by: Fan Zhang <zh.f@outlook.com>
* | | | Merge "Switch to cryptography from pycrypto"Zuul2018-06-135-25/+47
|\ \ \ \
| * | | | Switch to cryptography from pycryptoZhao Chao2018-06-085-25/+47
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PyCrypto isn't active developed for quite a while, cryptography is recommended instead. This patch does this migration, but still keeps pycrytpo as a fallback solution. Random generation is also migrated to os.urandom as the cryptography document suggests: https://cryptography.io/en/latest/random-numbers/ Closes-Bug: #1749574 Change-Id: I5c0c1a238023c116af5a84d899e629f1c7c3513f Co-Authored-By: Fan Zhang <zh.f@outlook.com> Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | | | Add a hook for restore process to check if successfulZhao Chao2018-06-074-9/+88
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1.This is the same solution with the backup process. Previously we only check the stderr of the restore command, but this method is not reliable, and the changes during percona-xtrabackup 2.4.11: https://jira.percona.com/browse/PXB-1542 causes the gate jobs failed in creating a slave MySQL instance. The new hook currently is only used by InnoBackupEx restore runner with checking the exit status and stderr output of xbstream. 2.with[1] merged,this makes DIB_CLOUD_IMAGES more flexible, but it break trovestack build image, now we need to specify a more detailed path to DIB_CLOUD_IMAGES to get the appropriate content. [1]:https://review.openstack.org/#/c/568697/ Co-Authored-By: Zhao Chao <zhaochao1984@gmail.com> Co-Authored-By: zhanggang <zhanggang@cmss.chinamobile.com> Co-Authored-By: jiansong <jian.song@easystack.cn> Closes-Bug: #1771990 Change-Id: Ibb54d6b5953a775be339fb991a0771961d27eba2
* | | Merge "Fix dict iteration in PropertiesCodec"Zuul2018-04-191-1/+8
|\ \ \ | |_|/ |/| |
| * | Fix dict iteration in PropertiesCodecZhao Chao2018-04-161-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're updating a dict while iterating in PropertiesCodec, this leads to random failures of the py35 unittest gate job. As the logs of a zuul gate job won't be permanent, I wrote a simple script(though may be ugly) to testing this: $ python --version Python 3.5.2 $ cat /tmp/python35_dict_updating.py data_dict = {} def init_dict(): global data_dict for x in range(11): key = 'key' + str(x+1) data_dict[key] = x * 10 def update_dict(): global data_dict for k, v in data_dict.items(): data_dict.update({k: v+1}) def checking_failed(round): global data_dict FAILED = False for x in range(11): key = 'key' + str(x+1) origin = x * 10 current = data_dict[key] if (current - origin) != round: print('%s is %s, expecting %s' % (key, current, origin + round)) FAILED = True break return FAILED TEST_TIMES=5 PASSED = True for x in range(TEST_TIMES): init_dict() for round in range(1, 100): update_dict() if checking_failed(round): print("Failed at round %s" % round) PASSED = False break if PASSED: print("No failures in 5 100-round tests") $ python /tmp/python35_dict_updating.py key9 is 82, expecting 81 Failed at round 1 $ python /tmp/python35_dict_updating.py key2 is 12, expecting 11 Failed at round 1 $ python /tmp/python35_dict_updating.py key9 is 82, expecting 81 Failed at round 1 $ python /tmp/python35_dict_updating.py No failures in 5 100-round tests From the above testing results, we could see it's quite often that one of the item will be updated twice during the dict iteration. This is the reason why test_properties_file_codec failed sometimes. This fix is manually converting the result of dict.items to a list: $ sed -i 's/data_dict.items()/list(data_dict.items())/' \ /tmp/python35_dict_updating.py $ python /tmp/python35_dict_updating.py No failures in 5 100-round tests $ python /tmp/python35_dict_updating.py No failures in 5 100-round tests $ python /tmp/python35_dict_updating.py No failures in 5 100-round tests $ python /tmp/python35_dict_updating.py No failures in 5 100-round tests Closes-Bug: #1764321 Change-Id: Ia9fcfc6519b29f1a9508b79614c5e81456ad57b6 Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | | Run unittests under the low-constraints jobZhao Chao2018-04-161-16/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | | | The low-constraints job should run unittests by the original desgin, we didn't put the commands about running unittests inside the default testenv settings, so in fact the low-constraints job did nothing except creating a virtualenv. This patch fixes and also move the unittests commands to the default testenv settings like the other OpenStack projects. Change-Id: Ia344a14a9baf7574b043fc08332869083da6dc1e Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | Fix lower-constraints and uncap eventlet10.0.0.0b1Zhao Chao2018-04-122-61/+62
|/ | | | | | | | | | | | | | | | | | | | | | | The 'requirements-check' gate job will validate requirements.txt and test-requirements.txt against lower-constraints.txt, i.e. the minimum version should match between lower-constraints and requirements. The change in lower-constraints.txt is produced by fix-lower-constraints.py script from the requirements repo, and 'enum34' is added mannually. This patch also include the commit about 'uncap eventlet' to make the 'requirements-check' actually working, because the change has been already merged in the requirements repo. The original commit message by Doug Hellmann is kept as below for futher referrence: We will manage the eventlet version using constraints now. See the thread starting at http://lists.openstack.org/pipermail/openstack-dev/2018-April/129096.html for more details. Co-Authored-By: Doug Hellmann <doug@doughellmann.com> Change-Id: I61a6904ea433e570420ccd46281162ff174a5737 Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* Merge "Updated from global requirements"Zuul2018-03-291-1/+1
|\
| * Updated from global requirementsOpenStack Proposal Bot2018-03-261-1/+1
| | | | | | | | Change-Id: I9cae682c6b028752340dd0efb18a8980d12ec0b8
* | Merge "fix a typo"Zuul2018-03-292-3/+3
|\ \
| * | fix a typowangqi2018-03-272-3/+3
| |/ | | | | | | Change-Id: Ia7e965a0f94e61e580a4c40eda8dffd8d76d2d64
* | Merge "Mox removal for instances_resize API tests"Zuul2018-03-282-140/+270
|\ \
| * | Mox removal for instances_resize API testsZhao Chao2018-03-222-140/+270
| | | | | | | | | | | | | | | | | | | | | | | | This is the last file using mox3 for testing, so mox3 is also removed from test-requirements.txt. Change-Id: Ie70adc620df04bf4e0ca43ff4e3060b449d2e377 Signed-off-by: Zhao Chao <zhaochao1984@gmail.com>
* | | Merge "add lower-constraints job"Zuul2018-03-283-0/+180
|\ \ \ | |_|/ |/| |
| * | add lower-constraints jobDoug Hellmann2018-03-223-0/+180
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create a tox environment for running the unit tests against the lower bounds of the dependencies. Create a lower-constraints.txt to be used to enforce the lower bounds in those tests. Add openstack-tox-lower-constraints job to the zuul configuration. See http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html for more details. Change-Id: I5352398603e5755d46c9c033b05af637e379eea0 Depends-On: https://review.openstack.org/555034 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* | Merge "fix a typo in documentation"Zuul2018-03-231-1/+1
|\ \