summaryrefslogtreecommitdiff
path: root/trove
Commit message (Collapse)AuthorAgeFilesLines
* Remove downgradeTrevor McCasland2016-10-0642-473/+9
| | | | | | | | | | | | Removed downgrade from all existing migrations. Modified a test that verifies that no migrations have a downgrade. Modified test _walk_versions to only do upgrades. Removed exceptions from pylint.config Related cross-project spec: https://review.openstack.org/152337 Closes-Bug: #1434103 Change-Id: I9a7c87ae3f4e2eff3a4a6df881d086d52062dbba
* Merge "Fix SafeConfigParser DeprecationWarning in Python 3.2"Jenkins2016-10-051-3/+8
|\
| * Fix SafeConfigParser DeprecationWarning in Python 3.2ChangBo Guo(gcb)2016-10-011-3/+8
| | | | | | | | | | | | | | | | | | | | | | SafeConfigParser is deprecated in Python 3.2 and log warning like " DeprecationWarning: The SafeConfigParser class has been renamed to ConfigParser in Python 3.2. This alias will be removed in future versions. Use ConfigParser directly instead." So use ConfigParser in Python 3.2+. Closes-Bug: #1618666 Change-Id: I30fe51324ffcc0afbd02799449daee8f628634b6
* | Merge "Modify parameters of spelling mistakes"Jenkins2016-10-041-2/+2
|\ \
| * | Modify parameters of spelling mistakesjiansong2016-09-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Look should be written mock_set_task_status, mock_set_tast_status is the wrong way Change-Id: I549f48be860db3a859ce9308d7bd42ca817a5d3a reference:https://github.com/openstack/trove/blob/master/trove/tests/unittests/taskmanager/test_manager.py#L114
* | | Merge "Delete unnessary i18n"Jenkins2016-10-041-1/+0
|\ \ \
| * | | Delete unnessary i18njiansong2016-10-031-1/+0
| | |/ | |/| | | | | | | | | | | | | We import i18n but didn't use, so I think should be deleted. Change-Id: I5e0781b46d7aa6318428e92ccc506349cbe876cb
* | | Merge "Fix a typo in service.py and trove_testtools.py"Jenkins2016-10-042-2/+2
|\ \ \
| * | | Fix a typo in service.py and trove_testtools.pygengchc22016-09-292-2/+2
| | |/ | |/| | | | | | | | | | | | | | | | There are many spelling mistakes in in service.py and trove_testtools.py.The word 'overriden' should be 'overridden'. Change-Id: I2e9c88280de98b535f355cd34692ba0a9f423bb1
* | | Merge "Use more specific asserts in tests"Jenkins2016-10-047-10/+10
|\ \ \
| * | | Use more specific asserts in testsGábor Antal2016-09-297-10/+10
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I changed asserts with more specific assert methods. e.g.: from assertTrue(sth == None) to assertIsNone(*) or assertTrue(isinstance(inst, type)) to assertIsInstace(inst, type) or assertTrue(not sth) to assertFalse(sth). The code gets more readable, and a better description will be shown on fail. Change-Id: I8f2ce30e5d876fc298b8e1ff4504e5e34dc090ab
* | | Merge "Fix typos in description"Jenkins2016-10-041-2/+2
|\ \ \
| * | | Fix typos in descriptionHa Van Tu2016-10-031-2/+2
| | |/ | |/| | | | | | | | | | | | | | | | | | | This patch fixes typos in description. + typo 'conole' -> 'console' + typo 'considerred' -> 'considered' Change-Id: I7a53ccd4da5ec90c58b4364b72bd3ea8285234d4
* | | Merge "Fix Old style classes are deprecated and no longer available in Python 3"Jenkins2016-10-0411-18/+18
|\ \ \
| * | | Fix Old style classes are deprecated and no longer available in Python 3Luong Anh Tuan2016-09-3011-18/+18
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | [H238] Old style classes are deprecated and no longer available in Python 3 (they are converted to new style classes). In order to avoid any unwanted side effects all classes should be declared using new style. See the new-style class documentation for reference on the differences. http://docs.openstack.org/developer/hacking/#python-3-x-compatibility Change-Id: I79e2f0d7d53d398fcf5ba18cefc8318ee6e07426
* | | Fix some typos in doc and commentsChangBo Guo(gcb)2016-10-042-2/+2
| |/ |/| | | | | Change-Id: I7afe1d5851878fa392c46c7654e984e73de4246c
* | Add Apache 2.0 license to source fileCao Xuan Hoang2016-09-291-0/+12
|/ | | | | | | | | | | | As per OpenStack licensing guide lines [1]: [H102 H103] Newly contributed Source Code should be licensed under the Apache 2.0 license. [H104] Files with no code shouldn't contain any license header nor comments, and must be left completely empty. [1] http://docs.openstack.org/developer/hacking/#openstack-licensing Change-Id: I38f2f1b1429ea49fb02ae1c5cb3e68d547bc1783
* Merge "Perfect the mysql test"Jenkins2016-09-281-2/+10
|\
| * Perfect the mysql testjiansong2016-09-201-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In test_populate_users_intermingled,Should be mixed the above three test, should have a total of four records.One is the foundation,Other records with this relationship should be 1.same name,differ host. 2.differ name,same host. 3.same name,same host. 4.differ name,differ host. At the same time joined the differ name, differ the host test in two record test Change-Id: I92c94cc4c352e6ad96a95d64313807f160072aa4
* | Merge "Remove unused variables"Jenkins2016-09-281-3/+3
|\ \
| * | Remove unused variablesTony Xu2016-09-221-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | trove/tests/api/instances_actions.py:stderr = create_server_connection(self.instance_id) trove/tests/api/instances_actions.py: dbaas_flavor, baas_flavor_href = res trove/tests/api/instances_actions.py:_dontcare_ = res Change-Id: Ibafb9888e00e00a76bf1ccd6510ae85d26f8f8bf
* | | Merge "Taskmgr & guestagent use inconsistent topics"Jenkins2016-09-281-0/+1
|\ \ \
| * | | Taskmgr & guestagent use inconsistent topicsDoug Shelley2016-09-241-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During a create instance call, the Task Manager needs to pre-create the guestagent message queue so that it can send a "prepare" RPC before the guest is running. It creates a topic queue called "guestagent.<UUID>". When the guest starts up, it will attempt to create a topic queue called "guestagent". The overall result is that if the guest creates the queue it has a different name than the task manager expects. This could happen if the guestagent restarts after prepare. Both cases result in a queue being created called "guestagent.<instance uuid>" because oslo.messaging causes 3 queues per topic to be created (topic, direct and fanout) and it so happens that the topic queue from task mgr matches the direct queue from the guestagent, however, this doesn't appear to be intentional behaviour. This changes the guestagent topic name to match the taskmgr name. Change-Id: Ie20dab8e763fe8b5f1897debd5942a64afea15ee Closes-bug: 1627179
* | | | Merge "Use common methods for cluster validation"Jenkins2016-09-2812-126/+264
|\ \ \ \
| * | | | Use common methods for cluster validationPetr Malik2016-09-2212-126/+264
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | Use a common set of tested utility methods for validations in the API. Change-Id: Icd3ea6251ef2ea2dbdaa05ae07f78bc3bd965b16
* | | | Merge "Remove cfg and logging import unused"Jenkins2016-09-2826-75/+0
|\ \ \ \
| * | | | Remove cfg and logging import unusedCao Xuan Hoang2016-09-2326-75/+0
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | This patch removes cfg and logging import unused in trove Change-Id: Ic50a58123481faa37b8898565a597dcdb1f1fa46 Closes-Bug: #1618281 Closes-Bug: #1618347
* | | | Merge "Use assertIn instead of assertTrue(A in B)"Jenkins2016-09-2715-44/+44
|\ \ \ \ | |/ / / |/| | |
| * | | Use assertIn instead of assertTrue(A in B)Bin Zhou2016-09-1215-44/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Developers should use assertIn(A, B) instead of assertTrue(A in B ). TrivialFix Change-Id: I31e20fae988b71145ec7b82fa696a87c4dbcc4e3
* | | | Merge "Concentrated test_validate_xx_configuration to a new method"Jenkins2016-09-231-46/+51
|\ \ \ \
| * | | | Concentrated test_validate_xx_configuration to a new methodjiansong2016-09-221-46/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems,Test what name such as test_validate_xx_configuration, only have different parameters, so we can take them to focus on a new method, to reduce repetitive code. Closes-Bug: #1622939 Change-Id: I80ae25a37a862ca3315291a88e3dbc39c1a94fc8
* | | | | Merge "Add max_prepared_stmt_count to validation rules"Jenkins2016-09-234-0/+28
|\ \ \ \ \
| * | | | | Add max_prepared_stmt_count to validation rulesDoug Shelley2016-09-234-0/+28
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add max_prepared_stmt_count to the list of valid parameters that can be set on configuration groups for Mysql, Percona, PXC and MariaDB. Change-Id: Ib5522ba54bcb1ea5ace0deb9eb671dcafe2fc189 Closes-bug: 1624418
* | | | | Merge "Display flavor-ephemeral in trove flavor-list command"Jenkins2016-09-232-2/+3
|\ \ \ \ \
| * | | | | Display flavor-ephemeral in trove flavor-list commandjiansong2016-09-172-2/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds ephemeral column to flavor-list Change-Id: I537bc2f3578e1e99b22f701f70ab8d96fec8908e Closes-Bug:1617980
* | | | | Merge "Fix typos in validation-rules.json & service.py"Jenkins2016-09-222-2/+2
|\ \ \ \ \
| * | | | | Fix typos in validation-rules.json & service.pyzhangyanxian2016-09-202-2/+2
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | TrivialFix Change-Id: I48409a3aa5227ae87185a951090586eda5a3f551
* | | | | Merge "Fixup indentation errors"Jenkins2016-09-221-1/+1
|\ \ \ \ \
| * | | | | Fixup indentation errorsTony Xu2016-09-221-1/+1
| | |_|_|/ | |/| | | | | | | | | | | | | Change-Id: I568a8fda16d276a9c5d5fa222437e4d237a1000a
* | | | | Merge "Delete unused method verify_errors"Jenkins2016-09-221-13/+0
|\ \ \ \ \
| * | | | | Delete unused method verify_errorsjiansong2016-09-201-13/+0
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | We define this method is never used, I think maybe you can delete Change-Id: I1aceb0d7d73d261a82a6d6e7ed918c7aaf33de78
* | | | | Merge "Repeat definition blacklist"Jenkins2016-09-221-1/+0
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Repeat definition blacklistjiansong2016-09-201-1/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | The variable black_list is set, used, and then again set to its previous value which is redundant. Change-Id: I24029148f8538207cd734cdc01c992aeadc1c922
* | | | Merge "Skip 'eject valid master' replication test"Jenkins2016-09-211-3/+7
|\ \ \ \
| * | | | Skip 'eject valid master' replication testPeter Stachowski2016-09-211-3/+7
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test was skipped in the scenario runs however it was not disabled in the original api tests. Although it doesn't fail as often there it still happens, so the same change has been applied to it. See: https://review.openstack.org/#/c/368230 The test will also print out the bug number during each run as a reminder (using the new SkipKnownBug method). Change-Id: I931d6d72a70cc93dcd8248d9840eadf1160b9bab Related-Bug: #1622014
* | | | Merge "Fix Postgresql configuration test and guest-log"Jenkins2016-09-218-32/+174
|\ \ \ \ | |/ / / |/| | |
| * | | Fix Postgresql configuration test and guest-logPetr Malik2016-09-208-32/+174
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Move 'log_min_duration_statement' into non-dynamic values. Although the property itself is dynamic, it still requires restart because we don't provide default value for it in our template. - Add the 'ms' suffix to the 'log_min_duration_statement' time. Postgres by default returns the default suffix. Specifing the suffix explicitly fixes the comparison. - Change the validation rules for PostgreSQL long_min_duration_statement to string from int. - Enhance the ConfigurationManager to be able to apply system overrides *before* user overrides, as well as after. This allows the logging to be overridden by the user config values. - Added to the guest-log scenario test to make sure that the user log doesn't change after being disabled, and that it can be enabled again. Co-Authored-By: Petr Malik <pmalik@tesora.com> Co-Authored-By: Peter Stachowski <peter@tesora.com> Closes-Bug: #1624492 Change-Id: I059357d52fa24c609e45c2dc108d2b4ba77b3458
* | | Merge "Fix pylint Error:PGSQL_RECOVERY_CONFIG"Jenkins2016-09-191-1/+1
|\ \ \
| * | | Fix pylint Error:PGSQL_RECOVERY_CONFIGjiansong2016-09-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we should use pgsql_recovery_config, the method which has now been renamed to lowercase. The earlier code used a constant (upper-case) but in the change this one was likely missed. pgsql_recovery_config is an @property and should be referenced as such. This has nothing to do with case per-se, just a missing function/variable. Closes-Bug: #1624996 Change-Id: Icc9d794b45503340d988101fec710fae76f0b915
* | | | Merge "Fix call to to_mb"Jenkins2016-09-191-2/+1
|\ \ \ \ | |/ / / |/| | |
View Lorry Controller Status