summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Move bin/ scripts to entrypoints.Monty Taylor2013-04-071-1/+1
| | | | | | | | | | | | | | | | | | | | | Instead of propogating more bin scripts, migrate these to be things that can be consumed as entrypoints. Change-Id: Ic01e0f16fe9e7634708fbb51499ccea3f4f40d63
* | | Merge "Add a gettextutils.install() helper function"Jenkins2013-04-041-2/+16
|\ \ \
| * | | Add a gettextutils.install() helper functionMark McLoughlin2013-04-031-2/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of fixing bug #995287 Every top-level script in every project needs to call gettext.install() so that a _() builtin function is installed and strings are translated using the correct translation domain. However, translations are always installed in the default localedir (which is commonly '/usr/share/locale') and, in cases like devstack, may be found in a project-specific localedir. To support such a use case we should have project-specific environment variables for overriding the default value of localedir - e.g. NOVA_LOCALEDIR. Add a new gettextutils.install() helper method to make this as easy as possible for projects to get right. Change-Id: I6c8549c8ff00797c96f2dd4b0b5266d18d77aa19
* | | | Copy recent changes in periodic tasks from nova.Michael Still2013-04-041-1/+178
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | The nova periodic task code has moved on. Bring oslo inline with the new code. The next step after this is to update Quantum (the only user of this code I can find) to the new interface, and then move nova across to using this code. Change-Id: I3f041ca9a6ba9f0e5be4b29198d16454c9797fef
* | | Merge "Fix IBM copyright strings"Jenkins2013-04-032-2/+2
|\ \ \
| * | | Fix IBM copyright stringsKurt Taylor2013-04-022-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Update IBM copyright strings to one consistant format Change-Id: I838321f9126d78818bae9ba0c79615b5f2d71226
* | | | Merge "Add PathFilter to rootwrap."Jenkins2013-04-031-0/+119
|\ \ \ \
| * | | | Add PathFilter to rootwrap.Roman Podolyaka2013-04-011-0/+119
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PathFilter is a type of filter that allows to check if path arguments of a command resolve to file system paths within given directories. Fixes bug 1098568. Change-Id: Ie2686ad2ff114075c6d8d804031b6e3fa60a43ca
* | | | Merge "rpc: fix positional args in remote exceptions"Jenkins2013-04-021-2/+23
|\ \ \ \ | |_|/ / |/| | |
| * | | rpc: fix positional args in remote exceptionsKiall Mac Innes2013-03-311-2/+23
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1160475. Positional arguments were dropped during the deserialization of valid remote exceptions, while keyword arguments are correctly supplied. Change-Id: I7b95fc4ed3fb9e5c75f5711ed6aace7aa5593727
* / / Fix the co-authored-by processing.Monty Taylor2013-04-021-1/+1
|/ / | | | | | | | | | | | | One of the problems with fakes is that sometimes you can fake away a bug and have happily passing tests and broken code. Change-Id: Ib544739699b63d3f5e80edc16e19377d45782334
* | Fix test_write_git_changelogAlessio Ababilov2013-03-281-8/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test changes current directory to a temporary one and ChangeLog file is created here. That's why the file doesn't reside in project's root directory. The correct behavior is to look for ChangeLog at cwd as is done in test_generate_authors for AUTHORS file. Use openstack.common.setup.__file__ to determine the root directory of openstack package. Change-Id: Ic7c9b01485a08b8921c88ac624e0aecd9409fa7c Fixes: bug #1161362
* | Merge "Remove deprecated Grizzly features"Jenkins2013-03-261-5/+2
|\ \
| * | Remove deprecated Grizzly featuresJoe Gordon2013-03-251-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that Havana has started, remove deprecated Grizzly features * deprecated alias list_notifier_drivers (for notification_driver) * openstack/common/notifier/rabbit_notifier.py (use rpc_notifier instead) Change-Id: I0dbb997ba774f58766bddf950049ec1e2d5b79de
| * | Merge "Remove unused imports"Jenkins2013-03-251-1/+0
| |\ \
| | * | Remove unused importsZhongyue Luo2013-03-231-1/+0
| | | | | | | | | | | | | | | | Change-Id: Idf2235300fa2c1c73563ddcff6ce9e84bd11ea0e
* | | | Update to use flake8.Monty Taylor2013-03-2414-114/+134
|/ / / | | | | | | | | | | | | | | | | | | flake8 is pluggable and handles pep8 and pyflakes, as well as configuration through tox.ini. It also removes the need for flakes.py. Change-Id: If5f7d8ad348b4fb8119fa4ec7b5e9d17bdc72a39
* | | Merge "Include Co-authored-by entries in AUTHORS."Jenkins2013-03-231-4/+20
|\ \ \ | |/ / |/| |
| * | Include Co-authored-by entries in AUTHORS.Monty Taylor2013-03-221-4/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by lines are the way we've decided to indicated shared authorship of a patch, so content from them should be included in the generated AUTHORS file. Fixes bug 1158319. Change-Id: I9dacf78c01f3ad74e696f16a7aa39edb98e8d185
* | | Merge "Locking edge case when lock_path does not exist"Jenkins2013-03-221-2/+0
|\ \ \
| * | | Locking edge case when lock_path does not existGary Kotton2013-03-221-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1158179 In the case that the lock_path does not exist and there is a contested resource then the process that was waiting on the lock will not be locking due to the fact that the directory was deleted. Change-Id: I75d720d4df499e85386d3e2cc86b927b017e12ac
* | | | Merge "lockutils: add a failing unit test"Jenkins2013-03-221-3/+22
|\| | |
| * | | lockutils: add a failing unit testMark McLoughlin2013-03-221-3/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a unit test which fails because of #1107950 - in this case, lockutils is deleting the lock directory every time the lock is dropped even though another process might have acquired a lock using that directory. Change-Id: Ic82409f9462e570bc102ab469334c53aafc6d7ac
* | | | Merge "Improves Logging for for rpc method timeouts"Jenkins2013-03-221-21/+39
|\ \ \ \
| * | | | Improves Logging for for rpc method timeoutsMatthew Sherborne2013-03-221-21/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch moves the traceback for an rpc timeout from inside an iterator, which gave a useless traceback, into the main flow of the program. It also adds the rpc method being called and the topic used to the exception's message. When the caller logs the message higher up the stack, the log information and traceback will be more useful. Finally it removes the timeout logging in the amqp.py module, in the spirit of bug #1137994 and https://review.openstack.org/#/c/23295/ Works towards: bug #1148516 Change-Id: I29a3b1b97c6114c4479e2b71c1257c4d72131535
* | | | | Merge "Enable message envelope."Jenkins2013-03-223-36/+2
|\ \ \ \ \
| * | | | | Enable message envelope.Russell Bryant2013-03-213-36/+2
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Grizzly had the ability to receive messages with an envelope, but did not send them. Now update the code to send them. Change-Id: I73aad7697cf83ad4aabb3c2058b7cc4f53f783c2
* | | | | Merge "lockutils: improve the external locks test"Jenkins2013-03-221-11/+13
|\ \ \ \ \ | | |_|/ / | |/| | |
| * | | | lockutils: improve the external locks testMark McLoughlin2013-03-221-11/+13
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use a separate directory for the lockutils external locks and for the test's flocks which are used to check that serialization is actually occurring. By using the same directory, we can't test the code in lockutils which auto-creates this directory because the dir gets deleted by a process by lockutils cleanup while another directory is using it for flocks. Also, assume the the handles directory has been created in the parent rather than having each child attempt to create it. Related, add a try/finally block so that when a child process throws an exception it immediately exits rather than deleting the temporary directories created by the parent. Change-Id: I32d7e8e05fb3f22cf38fa586f8bc97646c83f182
* | | | Merge "Gracefully handle errors in logging config files"Jenkins2013-03-221-0/+48
|\ \ \ \ | |_|/ / |/| | |
| * | | Gracefully handle errors in logging config filesMark McLoughlin2013-03-221-0/+48
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1154245 If the parameter to the --log-config option cannot be loaded by the logging.config.fileConfig() function, you get an exception like this raised: NoSectionError: No section: 'formatters' which doesn't do much to help users understand the error. Improve the situation by wrapping the error in a custom exception type giving this: LogConfigError: Error loading logging config /etc/logging.conf: No section: 'formatters' Also add some tests to check we raise this error under the most common failure conditions. Change-Id: I6ad2eb4867213b6356ce43d75da50218d1b498f6
* | | Merge "Allows wsgi server kwargs to be given."Jenkins2013-03-221-0/+19
|\ \ \ | |/ / |/| |
| * | Allows wsgi server kwargs to be given.Alex Meade2013-03-181-0/+19
| |/ | | | | | | | | | | | | | | | | This fix allows someone using openstack.common.wsgi to pass kwargs down to eventlet.wsgi.server when calling wsgi.run_server. Fixes bug 1156930 Change-Id: Id35232f68ee40c5435e157a834fc94d4bbd04970
* / Removes unused imports in the tests moduleZhongyue Luo2013-03-2018-35/+0
|/ | | | | | Fixes bug #1157596 Change-Id: I36d5484eaa2f0e21188eed6e70cc1ad785233d6a
* update OpenStack, LLC to OpenStack FoundationMark McClain2013-03-133-3/+3
| | | | | | | | | | fixes bug 1154745 The previous update (https://review.openstack.org/#/c/24103/) missed header files that contained "OpenStack, LLC". This change corrects the missed files to reflect the OpenStack Foundation. Change-Id: I9c6de265267485ef2c82ea7e6d8643e82134d102
* Merge "Sanitize input before creating IPC socket."Jenkins2013-03-131-0/+19
|\
| * Sanitize input before creating IPC socket.Eric Windisch2013-03-121-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | Sockets are created by the zeromq driver for the topic specified by each incoming message. Because the topic is arbitrarily supplied by the sender, path separators in the topic must be illegal. Fixes bug 1122763 Change-Id: Iccdb9b69e646bfe7665ee34c367fd4019db25f17
* | Fix Copyright Headers - Rename LLC to FoundationDavanum Srinivas2013-03-1126-26/+27
|/ | | | | | One code change, rest are in headers Change-Id: I73f59681358629e1ad74e49d3d3ca13fcb5c2eb1
* Merge "Add 'is' operator to extra specs ops."Jenkins2013-03-071-0/+30
|\
| * Add 'is' operator to extra specs ops.Avishay Traeger2013-03-071-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | Boolean values for capabilities don't work because extra_specs are all converted to unicode. The scheduler will then check, for example, if the boolean 'True' is equal to the unicode string 'True', and will always return False. This patch allows admins to specify '<is> True' in extra_specs, which will compare successfully to boolean True. Fixes bug: 1146306 Change-Id: Id0e6dcfb71eb0943a16bba551ec23c4d57206550
* | Simple code cleanupMatthew Sherborne2013-03-061-5/+3
|/ | | | | | | | | * Fix an exception that was raising a tuple * Remove unused imports * Remove unused exceptions * Remove extra blank lines Change-Id: I9127be991e9081dc173525c9b57ea297f389d16d
* support ISO 8601 micro-second precisionGuang Yee2013-03-041-0/+15
| | | | | | Bug 1134802 Change-Id: I9cc3c9d9324314d293f01f047882eb6be06e02dd
* Merge "Add simple in-memory cache from nova."Jenkins2013-03-011-0/+58
|\
| * Add simple in-memory cache from nova.Vishvananda Ishaya2013-03-011-0/+58
| | | | | | | | | | | | | | | | | | | | | | Nova's simple in-memory cache replicates the memcache interface, so clients can cache things in memcache or in memory using the same commands. Using memcached requires having the client library installed and the memcached_servers config option set. Callers can also pass in a list of memcached servers when they initialize the client. Change-Id: I831142a36797b04006cba4792df803e09f6fd69b
* | Merge "Redis-based matchmaker"Jenkins2013-03-013-31/+217
|\ \
| * | Redis-based matchmakerEric Windisch2013-03-013-31/+217
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduces a reference implementation of a matchmaker (based on redis) that supports dynamic host/topic registrations, host expiration, and hooks for consuming applications to acknowledge or neg-acknowledge topic.host service availability. Implements blueprint advanced-matchmaking Change-Id: I8608d2089fca118b0e369f2eb5c6aedacf6821fe
* | | Port safe parsing with minidom patches from NovaDavanum Srinivas2013-02-261-0/+101
| | | | | | | | | | | | | | | | | | | | | | | | Prevent attacks through xml entity expansion etc. Fixes LP# 1100282 Change-Id: I391531deac122697556c282184c8f8890ea66489
* | | Merge "openstack.common.setup: fails to get version from git"Jenkins2013-02-251-10/+35
|\ \ \
| * | | openstack.common.setup: fails to get version from gitDavanum Srinivas2013-02-251-10/+35
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | quantum run_tests.py fails because openstack.common.setup._get_version_from_git fails. It is because quantum unit tests run under quantum/tests/unit instead of git root dir. So the function should check parent dirs for .git. cinder folks seem to have hit this bug (1125416). ERROR: test_network_gateway_update (quantum.tests.unit.nicira.test_networkgw.NetworkGatewayExtensionTestCase) ---------------------------------------------------------------------- Traceback (most recent call last): File "quantum/quantum/tests/unit/nicira/test_networkgw.py", line 70, in setUp config.parse(args=args) File "quantum/quantum/common/config.py", line 99, in parse version='%%prog %s' % quantum_version.release_string()) File "quantum/quantum/openstack/common/version.py", line 63, in release_string self.release = self._get_version_from_pkg_resources() File "quantum/quantum/openstack/common/version.py", line 56, in _get_version_from_pkg_resources return setup.get_version(self.package) File "quantum/quantum/openstack/common/setup.py", line 334, in get_version raise Exception("Versioning for this project requires either an sdist" Exception: Versioning for this project requires either an sdist tarball, or access to an upstream git repository. Change-Id: I2e24c00b5ba8f35381cac081ff72d86ea0d75d19 Fixes: bug #1131162 and bug #1125416 Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
* | | Merge "Decode / Encode string utils for openstack"Jenkins2013-02-251-1/+33
|\ \ \