summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Add a gettextutils.install() helper functionMark McLoughlin2013-04-032-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-042-27/+276
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | 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-034-4/+4
|\ \ \
| * | | Fix IBM copyright stringsKurt Taylor2013-04-024-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Update IBM copyright strings to one consistant format Change-Id: I838321f9126d78818bae9ba0c79615b5f2d71226
* | | | Merge "Add PathFilter to rootwrap."Jenkins2013-04-032-0/+165
|\ \ \ \
| * | | | Add PathFilter to rootwrap.Roman Podolyaka2013-04-012-0/+165
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-022-3/+24
|\ \ \ \ | |_|/ / |/| | |
| * | | rpc: fix positional args in remote exceptionsKiall Mac Innes2013-03-312-3/+24
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-022-3/+3
|/ / | | | | | | | | | | | | 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
* | Merge "Change qpid_port to IntOpt"Jenkins2013-03-291-2/+2
|\ \
| * | Change qpid_port to IntOptAaron Rosen2013-03-281-2/+2
| | | | | | | | | | | | | | | | | | Fixes bug 1161679 Change-Id: I4b64fccbb7871dfab4fb74534514aa269c31f03d
* | | 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
* | Mark sql_connection with secret flagMichael J Fork2013-03-271-1/+2
| | | | | | | | | | | | | | | | | | sql_connection contains the password in some setups and marking as secret prevents accidental logging fixes bug 1160680 Change-Id: Ib2d00219ea40a010c62e6a29045309f030e6de28
* | Merge "Include PID in default logging_context_format_string"Jenkins2013-03-261-3/+3
|\ \
| * | Include PID in default logging_context_format_stringKiall Mac Innes2013-03-171-3/+3
| |/ | | | | | | Change-Id: I1d0458ecb1314c941d8c72a54c76f9e554ef4348
* | Merge "gettextutils: fix translation domain"Jenkins2013-03-261-1/+1
|\ \
| * | gettextutils: fix translation domainMark McLoughlin2013-03-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1076285 When oslo-incubator code gets copied into a project, the project's l10n infrastructure will extract translatable messages from the oslo code into the project's message catalog. Translations of those messages will not be found if we use 'openstack.common' as the translation domain. Instead, use 'oslo' as the translation domain and update.py will change this to the project name. Change-Id: I7ab46c4f860c9c307758ab63002ae410e4f975fe
* | | Merge "Remove deprecated Grizzly features"Jenkins2013-03-263-35/+2
|\ \ \
| * | | Remove deprecated Grizzly featuresJoe Gordon2013-03-253-35/+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 "Fix default values for Matchmaker IntOpts"Jenkins2013-03-261-2/+2
|\ \ \ \
| * | | | Fix default values for Matchmaker IntOptsJoe Gordon2013-03-251-2/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following should use ints not strings. * matchmaker_heartbeat_freq * matchmaker_heartbeat_ttl Fix bug 1159889 Change-Id: If2ce034d2a9876dbe6b232a1f5d73367d29074b0
* | | | Merge "Update to use flake8."Jenkins2013-03-2615-122/+141
|\ \ \ \ | |/ / / |/| | |
| * | | Update to use flake8.Monty Taylor2013-03-2416-123/+141
| |/ / | | | | | | | | | | | | | | | | | | 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 "Fix problem with setting SSL protocol in QPid"Jenkins2013-03-251-1/+1
|\ \ \
| * | | Fix problem with setting SSL protocol in QPidDavanum Srinivas2013-03-241-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | looks like we were setting the wrong param Fixes LP# 1158807 Change-Id: Id138926f737ce87618d625aa8b289c508a66aaf4
* | | Merge "Remove unused imports"Jenkins2013-03-251-1/+0
|\ \ \ | |/ / |/| |
| * | Remove unused importsZhongyue Luo2013-03-231-1/+0
| | | | | | | | | | | | Change-Id: Idf2235300fa2c1c73563ddcff6ce9e84bd11ea0e
* | | Merge "Include Co-authored-by entries in AUTHORS."Jenkins2013-03-232-4/+28
|\ \ \ | |/ / |/| |
| * | Include Co-authored-by entries in AUTHORS.Monty Taylor2013-03-222-4/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-222-3/+0
|\ \ \
| * | | Locking edge case when lock_path does not existGary Kotton2013-03-222-3/+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 "Eliminate multiple contexts in zmq call()"Jenkins2013-03-221-4/+9
|\ \ \ \
| * | | | Eliminate multiple contexts in zmq call()Eric Windisch2013-03-201-4/+9
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because we need compatibility with Grizzly (N-1 support), this should be reduced to two copies for Havana, before stripping down to a singular copy for I. The '-reply' request has its context key removed, with the msg option maintaining one extra copy of context until I. A TODO has been entered to clean this up following the Havana release. This fix eliminates an extra JSON double-deserialization routine. The double-serialization of context is still performed for Grizzly compatibility. fixes bug 1156956 Change-Id: I071e382ec9c7e0df3794fe9ca574a5722f8910a6
* | | | Merge "Improves Logging for for rpc method timeouts"Jenkins2013-03-224-29/+74
|\ \ \ \
| * | | | Improves Logging for for rpc method timeoutsMatthew Sherborne2013-03-224-29/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-226-46/+5
|\ \ \ \ \
| * | | | | Enable message envelope.Russell Bryant2013-03-216-46/+5
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-222-1/+70
|\ \ \ \ | |_|/ / |/| | |
| * | | Gracefully handle errors in logging config filesMark McLoughlin2013-03-222-1/+70
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-222-2/+21
|\ \ \
| * | | Allows wsgi server kwargs to be given.Alex Meade2013-03-182-2/+21
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "Adds pylint section in tox.ini to check for unused imports"Jenkins2013-03-222-1/+40
|\ \ \ | |_|/ |/| |
| * | Adds pylint section in tox.ini to check for unused importsZhongyue Luo2013-03-212-1/+40
| | | | | | | | | | | | | | | | | | Fixes bug #1157597 Change-Id: I77e0ddb1babf1eec0bb953b8bfa52cd1d415f709
* | | Remove detailed-errors from setup.cfg.Monty Taylor2013-03-211-1/+0
|/ / | | | | | | | | | | | | | | detailed-errors does not provide us any value in our use of runs, and it actually breaks now that we're using standard unittest protocol and testtools. Change-Id: Ie5a36eaf022e3f3d47904aad0ba2efdfc0d85b34
* | Merge "Removes unused imports in the tests module"Jenkins2013-03-2018-35/+0
|\ \
| * | Removes unused imports in the tests moduleZhongyue Luo2013-03-2018-35/+0
| |/ | | | | | | | | | | Fixes bug #1157596 Change-Id: I36d5484eaa2f0e21188eed6e70cc1ad785233d6a
generated by cgit (git 2.34.1) at 2025-09-27 04:37:17 +0000