summaryrefslogtreecommitdiffstats
path: root/openstack
Commit message (Collapse)AuthorAgeFilesLines
...
| * Allow running test in uninstalled source tree.Monty Taylor2013-02-102-7/+7
| | | | | | | | | | | | | | | | It was pointed out that we missed the use-case that running tests in a fresh tree checkout should work - and we missed that by reaching too far into the setup module interface. Shame on us. Change-Id: Ic0056a3113b308f063d927484e1e4a44c9316a2d
* | Merge "Support testing args for LocalhostMatchMaker."Jenkins2013-02-112-7/+9
|\ \
| * | Support testing args for LocalhostMatchMaker.Eric Windisch2013-02-102-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The matchmaker should return the correct direct topics used by the RPC driver. The LocalhostMatchMaker was always returning 'localhost', even though our tests might use '127.0.0.1' instead. Now, the LocalhostMatchmaker will use whatever value it is told should be considered local. MatchMakerLocalhost and LocalhostExchange now take an optional host parameter. impl_zmq._get_matchmaker now passes arguments to the selected matchmaker module. The test_zmq now initializes the matchmaker, using the host '127.0.0.1' Change-Id: I8daa2c0668f1d717eb95ab56009612b8b60f0a15
* | | Merge "Add a fixture for dealing with mock patching."Jenkins2013-02-111-0/+51
|\ \ \ | |_|/ |/| |
| * | Add a fixture for dealing with mock patching.Monty Taylor2013-02-071-0/+51
| | | | | | | | | | | | | | | | | | | | | Quantum uses a pattern all over their test base, which can be collapsed down into a simple re-usable fixture. Change-Id: I5944505ce44ce8b79a685c3ea392f001307b5319
* | | Merge "Implements import_group"Jenkins2013-02-091-0/+18
|\ \ \
| * | | Implements import_groupZhongyue Luo2013-01-311-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created import_group method in cfg.py Added testcases Fixes bug #1107843 Change-Id: I427d4069dacbb6e586687370adbb08b5d50f7b63
* | | | Merge "Update HACKING.rst per recent changes"Jenkins2013-02-091-1/+1
|\ \ \ \
| * | | | Update HACKING.rst per recent changesZhongyue Luo2013-02-041-1/+1
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added "is not" usage with examples Fixed "not in" usage description https://review.openstack.org/#/c/20576/ and https://review.openstack.org/#/c/20879/ Fixes a previous "not X in Y" grep corner case Change-Id: I92ae642887de734a8562a528003d8092f5ff27d4
* | | | Merge "Exchanges should return directed topics."Jenkins2013-02-081-3/+3
|\ \ \ \
| * | | | Exchanges should return directed topics.Eric Windisch2013-02-011-3/+3
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Exchanges in the matchmaker are supposed to return a directed topic as the first element in returned tuples. Adds a test to ensure this is so. Change-Id: I6c62f325914faa19216d5362c6d8ed0288942110
* | | | Merge "Allow to_primitive to ignore datetimes"Jenkins2013-02-081-21/+13
|\ \ \ \
| * | | | Allow to_primitive to ignore datetimesJoe Gordon2013-02-071-21/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In preparation for having nova.db.api not return any sqlalchemy objects. nova.db.api will return only primitives, except for datetime.datetime objects. Uses functools.partial to make code DRYING Partially implements bp db-api-cleanup Change-Id: I9980d8c4e20b05bbe734cf90ac209e4e7e89befb
* | | | | Use importutils.try_import() for MySQLdbMark McLoughlin2013-02-071-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a common pattern we've adopted elsewhere, so use it here too. Change-Id: I2e0947b4857005b478e796e9c9a7c05ea1f0d926
* | | | | Minor tweak to make update.py happyMark McLoughlin2013-02-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fairly lame to do this instead of just fixing update.py, but we seem to do it this way everywhere else, so ... Change-Id: Ie1cebcfec57c6138b9b726d7d3bc6e0ea456f146
* | | | | Remove openstack.common.db.commonMark McLoughlin2013-02-073-58/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since this only contains exceptions that are either used in session.py or utils.py (not both), it seems sensible to move them into the modules where they are used. Also, remove the use of the Invalid base exception class since we don't seem to be making use of the base class anywhere by catching it rather than the more specialized exceptions. Change-Id: Ib05bb2e0a9494e1dc60c60b8eee0e76b5d2ee555
* | | | | Provide creating real unique constraints for columnsBoris Pavlovic2013-02-072-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Main issue is that we can't create unique constraint for columns, because we are using soft deletion of entries (set `deleted` column to True). The main idea is to use `deleted` columns to create unique constraint for columns. For example (`col1`, `deleted`). To make (`col1`, `deleted`) unique after entry deletion, we should assign the value of `id` to `deleted` column. Change type of `deleted` column from Boolean to table.id.type for all tables. Change models.soft_delete() method to assign table.id instead of True to `deleted` column. Change query.soft_delete() method to assign literal_column("id") instead of True blueprint db-unique-keys Change-Id: Ie1f67f49a5d085e6371efb63fc23a1c8b25d9464
* | | | | Fix missing wrap_db_error for Session.execute() methodBoris Pavlovic2013-02-071-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should add wrap_db_error for Session.execute() method, because Query.update() method will call Session.execute() and it is not inside Session.flush(), so exceptions from Query.update() wouldn't be wrapped. Add test for Session.flush() wrapper Add test for Session.execute() wrapper Fixes bug 1107890 Change-Id: I96894e502f9f279999b61a1e66469e34a07a022d
* | | | | Fix eventlet/mysql db pooling codeBrian Elliott2013-02-071-3/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Eventlet versions >= 0.10 return a tuple from db_pool.ConnectionPool().create(), not a scalar. Also, set the FOUND_ROWS client flag, which sqlalchemy appears to require to sanity check the number of rows touch. bug 1097992 Change-Id: I64e798660c506fcd13947a232cb7832db0318360
* | | | | Add missing DBDuplicateEntryMark McLoughlin2013-02-071-0/+7
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | The session.py part of nova's 8835866 was imported, but not the newly added exception. Change-Id: I0d79a8536f0c8f14cb062dc61c5fce100b63fd64
* | | | Add support for directly stringifying VersionInfo.Monty Taylor2013-02-071-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | glanceclient, and others, want to have a __version__ member in their top-level __init__.py. This leads to attempting to process the version information to realize the string at import time, rather than at request time. If we make VersionInfo a thing which can behave like a string, then, we should get the things we want without the operational weirdness. Change-Id: I3eb15006f50ed1155c85977aaae842677c5ec4bc
* | | | Merge "timeutils: considers that now is soon"Jenkins2013-02-061-1/+1
|\ \ \ \
| * | | | timeutils: considers that now is soonJulien Danjou2013-01-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a follow-up to Doug suggestion at https://review.openstack.org/#/c/20323/2/openstack/common/timeutils.py Change-Id: Ic318202234202e56054df1b5cc7f82a404c307bc Signed-off-by: Julien Danjou <julien@danjou.info>
* | | | | Merge "Emit a warning if RPC calls made with lock."Jenkins2013-02-063-59/+117
|\ \ \ \ \
| * | | | | Emit a warning if RPC calls made with lock.Michael Still2013-02-063-59/+117
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch will log a warning every time a RPC call is made while a lock is held if the caller has requested it. This should help us track down performance problems such as the security group refresh problem recently found in nova. RPC calls can emit a warning if called with check_for_lock=True and debugging is turned on. Sneaks up on bug 1063222. Change-Id: Ice94093efb3cb95dd58b31d6ba817c7d505c15af
* | | | | Merge "Be explicit about set_default() parameters"Jenkins2013-02-061-2/+4
|\ \ \ \ \
| * | | | | Be explicit about set_default() parametersMark McLoughlin2013-02-051-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We probably only want to support people using set_defaults() for a limited set of config options, so explicitly list those as parameters to the method. This is what we do for rpc and log options too. Change-Id: I3bfc5355e82eec17480b9547662e847dc0d74138
* | | | | | Merge "Remove duplicate DB options"Jenkins2013-02-061-6/+0
|\| | | | |
| * | | | | Remove duplicate DB optionsMark McLoughlin2013-02-051-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | sql_dbpool_enable and sql_connection_trace are declared twice. Looks like this was a merge conflict when sql_dbpool_enable changed from int to bool. Change-Id: I553c5beea916e0ee296648342324fd53167c073a
* | | | | | Merge "Eliminate gratuitous DB difference vs Nova"Jenkins2013-02-061-3/+2
|\| | | | |
| * | | | | Eliminate gratuitous DB difference vs NovaMark McLoughlin2013-02-051-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Looks like this was an abritrary cleanup done as part of the import. It conflicts with a more recent change in Nova. We can move to using try_import() once we've killed the Nova copy of this code. Change-Id: I5c67d3fbd846a262feb724e26b14447a76561ae6
* | | | | | Merge "Import sqlalchemy session/models/utils"Jenkins2013-02-066-0/+944
|\| | | | | | |/ / / / |/| | | |
| * | | | Import sqlalchemy session/models/utilsEric Windisch2013-02-056-0/+944
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring in session, base model, utilities, and tests for sqlalchemy from Nova. Add sqlalchemy to pip-requires and and python-mysql to test-requires. Partially implements blueprint common-db Change-Id: I3e0065cdac87e10c4e0742d66c293c72bb3acbb2
* | | | Merge "Mark password config options with secret"Jenkins2013-02-052-2/+4
|\ \ \ \
| * | | | Mark password config options with secretMichael J Fork2013-02-042-2/+4
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Config object supports masking values when writing out if the secret flag is set on the option definition. This change flags all oslo-incubator options containing a password. Change-Id: I78451c4fdfdb3d6e28a7bd5f35d1c4c9a1b2be99
* | | | Merge "Only unpack replies on caller."Jenkins2013-02-051-22/+18
|\ \ \ \
| * | | | Only unpack replies on caller.Eric Windisch2013-01-311-22/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Opening the messages is unnecessary in ZmqProxy, given that the msg_id is already in the (zmq-specific) message envelope. Additionally, by not opening the messages, we save a bit of complexity. This avoids having to deserialize and reserialize messages in ZmqProxy, a necessary improvement for trusted-messaging and should also have a markable performance benefit. Also cleans up msg_id handling a bit. Change-Id: I31f365b9e1040503206b1d8320a0266a8121c821
* | | | | Merge "Use bytes instead of str"Jenkins2013-02-051-4/+3
|\| | | | | |/ / / |/| | |
| * | | Use bytes instead of strEric Windisch2013-01-311-4/+3
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Clarifies that str is the wrong datatype for passing messages over ZeroMQ. This distinction will be more important with the eventual migration to Python 3.0, but (I hope) this also helps clarify WHY typecasting is performed before sending messages. Change-Id: I4dd4661a5730a24ba7b514520b0429b721b05e9c
* | | Support for ipv6 in wsgi.ServiceDavanum Srinivas2013-02-031-3/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable wsgi.Service to listen on ipv6 address by checking if the host specified is ipv6. Based on that set the appropriate family in the eventlet.listen api More tests for just the ipv6, ipv6 with app, ipv6+ssl with app to make sure everything is working fine Change-Id: I2772905128bdbc69dd0fafe4ced848f5c477d7c8
* | | Merge "Support for SSL in wsgi.Service"Jenkins2013-02-022-5/+139
|\ \ \
| * | | Support for SSL in wsgi.ServiceDavanum Srinivas2013-01-292-5/+139
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable support for SSL as well using code from glance. We have some new options for configuring the SSL support. test_app starts wsgi.Service with a test app, then creates a url to make sure the http requests are actually served properly test_app_using_router adds wsgi.Router and Mapper() to the mix along with using the wsgify annotation for serving the http requests Fixes LP# 979488 (partial) Fixes LP# 869884 (partial) DocImpact Change-Id: Iae47b13b50e00c102c8c36f4a3e73b24fa4e6303
* | | | Merge "Add _FATAL_EXCEPTION_FORMAT_ERRORS global."Jenkins2013-02-021-3/+8
|\ \ \ \
| * | | | Add _FATAL_EXCEPTION_FORMAT_ERRORS global.Dan Prince2013-01-301-3/+8
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new global variable to control when exception format errors are fatal. Also, updates the Oslo test base class to enable fatal exception format errors. The motivation for this change is to give projects that use openstack common a hook to enable exception format checking when testing. Change-Id: Id8f4a0946b1614c8e987976b79069532a2e8608f
* | | | Merge "Use importutils for matchmaker import."Jenkins2013-02-021-15/+1
|\ \ \ \
| * | | | Use importutils for matchmaker import.Eric Windisch2013-01-301-15/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Per Vish's suggestion. this should be simplier, cull redundant code, and otherwise cut the fat. Change-Id: I9fa3effdc8e9e2f3f124690ecf77ee8bb7d4d39e
* | | | Merge "Dict args safe processing."Jenkins2013-02-021-8/+9
|\ \ \ \ | |_|_|/ |/| | |
| * | | Dict args safe processing.Eric Windisch2013-01-281-8/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve safety of dict args in process() Change-Id: I0eae36ca6566c1b60f0a01ebee57ae8a7530fcb1
* | | | Fixes 'not in' operator usageZhongyue Luo2013-01-311-1/+1
| |/ / |/| | | | | | | | Change-Id: Ibac9853fc72e2452fa5e2b3a493bd03523faabfb
* | | Merge "Support running uninstalled in a tarball dir."Jenkins2013-01-301-3/+10
|\ \ \ | |_|/ |/| |
generated by cgit (git 2.34.1) at 2025-09-27 17:14:03 +0000