summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Ignore prlimit argument on Windowsnewton-eol3.14.1stable/newtonLucian Petrut2016-10-032-5/+32
| | | | | | | | | | | | | The built in 'resource' module is used in order to apply process resource limits. This is not available on Windows. For the moment, we'll ignore this argument and log a warning message. In the future, we may provide a portable implementation for this feature. Change-Id: I6a82916eb68fbf8737b45a65f71cae1f835d12d1 Closes-Bug: #1627766 (cherry picked from commit 70ff5511053bc7070c071f46978b1ff59b89c4b5)
* Update .gitreview for stable/newtonDoug Hellmann2016-08-291-1/+2
| | | | Change-Id: I0ddb8bd93f33c447a246d8546885576e5b6f7175
* Updated from global requirements3.14.0OpenStack Proposal Bot2016-08-131-1/+1
| | | | Change-Id: Iaef3d3ff7c565991ed4eaf00a8ee2bdfe80c2347
* Fix external lock tests on WindowsDenis Buliga2016-08-081-42/+55
| | | | | | | | | | | | | | | At the moment, those tests use fcntl to acquire file locks, which will fail on Windows. This change addresses this issue by adding some platform checks, using the appropriate functions when handling file locks. Note that we've avoided running private methods from fasteners. Also, we now avoid using os.fork, which won't work on Windows. Instead, we spawn new processes using multiprocessing.Process, using queues for communication between processes. Change-Id: I9839b9033c814280f6d1b53c5ed2643fd2bf8bf8
* Updated from global requirements3.13.0OpenStack Proposal Bot2016-07-291-1/+1
| | | | Change-Id: I2e9ff687f0408795147e5b8c4769aa3a21fa87bd
* Fix parameters of assertEqual are misplacedyanheven2016-07-232-39/+41
| | | | | | | | | | | | Many assertEqual sentences don't follow assertEqual(expected, actual), These misplaces have 2 impacts: 1, giving confusing messages when some tests failed. 2, mislead other developers, new test modules may follow these wrong pattern. This patch fix all of them. Change-Id: I00bb4801c208d0eb8078e8a34ae52f9bcfe0b1a6 Closes-Bug: #1604213
* Add Python 3.5 classifier and venvChangBo Guo(gcb)2016-07-142-5/+2
| | | | | | | | Now that there is a passing gate job, we can claim support for Python 3.5 in the classifier. This patch also adds the convenience py35 venv. This commit also removes py34 specific venv. Change-Id: I50ddda7d96c77e7db60aa0b42e8e3b701dadf404
* Updated from global requirements3.12.0OpenStack Proposal Bot2016-07-091-2/+2
| | | | Change-Id: I40bc3ca015baff9c95499d395168a41eef1d3499
* Imported Translations from ZanataOpenStack Proposal Bot2016-07-011-0/+30
| | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: I3657f625009612562b2a40e7295089cf54151846
* Updated from global requirementsOpenStack Proposal Bot2016-06-302-3/+3
| | | | Change-Id: I75f0c00c84523ac027b9f68e3980d7de2815ee02
* Imported Translations from Zanata3.11.0OpenStack Proposal Bot2016-06-111-0/+101
| | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: I324acd475c84727b21aa9905c1184add135eb0cd
* Imported Translations from Zanata3.10.0OpenStack Proposal Bot2016-06-043-120/+19
| | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: Ifa683e2c40e33c05a4b6f6be514bb8a879179ee8
* Updated from global requirementsOpenStack Proposal Bot2016-06-032-3/+3
| | | | Change-Id: Ia49ba770943fee9f3e74cf860a2f77a7a3c14ae6
* Add reno for releasenotes managementKirill Bespalov2016-05-2010-0/+314
| | | | | | | An initial patch to add reno and create a base directory for release notes Change-Id: Ie3f6653a27a49d8b2fced3f399b1d4c8f0c95b7c
* Merge "Add timeout option to ssh_execute"3.9.0Jenkins2016-05-142-3/+13
|\
| * Add timeout option to ssh_executezhangsong2016-05-122-3/+13
| | | | | | | | | | | | | | | | | | Add an optional timeout parameter to ssh_execute() method of oslo_concurrency.processutils. Timeout is None means infinity, which is the default. Change-Id: I45532799ecd23ff8c5572036d06725e71965b2b5 Closes-Bug: #1580826
* | Add doc/ to pep8 checkYuriy Taraday2016-05-122-2/+2
| | | | | | | | | | | | | | We had unused imports there, this would prevent us from letting more of such errors in. Change-Id: Ie78c18dd3b9ab34370e47eba98b32740e0c83e1d
* | Remove unused import statementChangBo Guo(gcb)2016-05-121-2/+1
| | | | | | | | | | | | | | We don't use fnmatch and fileinput, so remove them. Change-Id: Ic49f3a03f4ccafcd02c549a6bdb0a3175f9e07e1
* | Merge "Fix wrong import example in docstring"Jenkins2016-05-111-2/+2
|\ \
| * | Fix wrong import example in docstringChangBo Guo(gcb)2016-05-111-2/+2
| |/ | | | | | | Change-Id: I0abc5403851f9ab2f78cdf26d29b82c3ec76ec25
* | Trivial: ignore openstack/common in flake8 exclude listChangBo Guo(gcb)2016-05-111-1/+1
|/ | | | | | | The directory openstack/common doesn't exist any more. So remove it from flake8 exclude list. Change-Id: Ieb4a5aac33b69f4cd7a4c0761836cb6bee087430
* Merge "processutils: add support for missing process limits"3.8.0Jenkins2016-05-043-11/+85
|\
| * processutils: add support for missing process limitsDaniel P. Berrange2016-04-193-11/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The original commit adding support for process limits only wired up address space, max files and resident set size limits. This is not sufficient to enable nova to protect qemu-img commands against malicious images. This commit adds support for the remaining limits supported by python: core file size, cpu time, data size, file size, locked memory size, max processes and stack size. Related-bug: #1449062 Change-Id: I164c4b35e1357a0f80ed7fe00a7ae8f49df92e31
* | Updated from global requirementsOpenStack Proposal Bot2016-04-301-1/+1
| | | | | | | | Change-Id: Icf2fd29b3d8fa5b8ab678d4085fd72970122c64e
* | Imported Translations from ZanataOpenStack Proposal Bot2016-04-218-50/+50
|/ | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: I65f5fc2140e9a12057343f89596f97648315e647
* Remove direct dependency on babelJoshua Harlow2016-04-181-1/+0
| | | | | | The dependency on oslo.i18n already forces this requirement. Change-Id: I4839b63ec9c8005f38231ee11f433a4c334e96d5
* Updated from global requirementsOpenStack Proposal Bot2016-04-131-1/+1
| | | | Change-Id: Ie01f18f40bc35f7dc57c083014c94b9a949a6ef6
* Updated from global requirementsOpenStack Proposal Bot2016-04-081-1/+1
| | | | Change-Id: Ib2b3d0985a0c39e6fcb894dedb956c420a8f7fc6
* Updated from global requirementsOpenStack Proposal Bot2016-04-071-1/+1
| | | | Change-Id: Iaf46167e8c39c5f546c7d86b85d84de6f500b1e8
* Add a few usage examples for lockutilsJoshua Harlow2016-03-171-0/+51
| | | | | | | | There was a complaint that the usage examples in oslo.concurrency are lacking (which they are) so this addresses some of those concerns by adding a few usage examples (and descriptions). Change-Id: I0738b898f1235e83a600c7732d70058b3fd3860f
* Merge "Revert "Use tempfile.tempdir for lock_path if OSLO_LOCK_PATH is not set""Jenkins2016-02-291-4/+1
|\
| * Revert "Use tempfile.tempdir for lock_path if OSLO_LOCK_PATH is not set"Yuriy Taraday2016-02-261-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 5021ef82fd8f0323b82d6d010bff9dab8a0cbcec. That commit set default location for lock files to a world-writable directory (like /tmp) which violates security precaution stated right in the help string of lock_path config variable: lock_path should be writable by the user of the process using it only. There is some history behind this: - when we decided to not do tempfile.mkdtemp if lock_path were not set (because it was broken), it broke upgrades for all components (starting with Cinder; - it was suggested to provide the same default value: https://review.openstack.org/60274 but it was not accepted because it weakens security; - there was a thread about this in ML, please read it for more history: http://lists.openstack.org/pipermail/openstack-dev/2013-December/021055.html it ended up with understanding that only a project using the library can provide sane default value for lock_path, not the library itself. Change-Id: I801004ad3c83862849696f00c6e7ca9877fd496b
* | Merge "Use tempfile.tempdir for lock_path if OSLO_LOCK_PATH is not set"Jenkins2016-02-261-1/+4
|\ \ | |/
| * Use tempfile.tempdir for lock_path if OSLO_LOCK_PATH is not setSean M. Collins2016-02-251-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DevStack spends a lot of time setting lock_path in a bunch of locations. http://codesearch.openstack.org/?q=oslo_concurrency%20lock_path&i=nope&files=&repos= The default for lock_path is an environment variable OSLO_LOCK_PATH. http://codesearch.openstack.org/?q=OSLO_LOCK_PATH&i=nope&files=&repos= Since the default is to consult an environment variable, which may not even be set, this means that really there is no default. If you do not set it - you end up getting errors like: http://paste.openstack.org/show/488108/ Ideally - a library should have a reasonable default, where if a user doesn't set something, the default will at least work. So, let's use Python's built in tempfile module, which has fairly complex rules for determining a sane directory to place temporary items in. https://docs.python.org/2/library/tempfile.html#tempfile.tempdir Change-Id: I6906af43bc0255cd215f2d9584ea000c81f5880e
* | Updated from global requirementsOpenStack Proposal Bot2016-02-261-1/+1
|/ | | | Change-Id: I1650fd8adb65777de950fe25a4585a9559fb9ecb
* Updated from global requirements3.6.0OpenStack Proposal Bot2016-02-201-1/+1
| | | | Change-Id: I535d094e5097b3b20cba25828df860f643b84fa4
* Merge "Make ProcessExecutionError picklable"3.5.0Jenkins2016-02-202-4/+27
|\
| * Make ProcessExecutionError picklableAngus Lees2016-02-192-4/+27
| | | | | | | | | | | | | | | | | | | | | | Serialising/deserialising exceptions in python follows the protocol defined by pickle. It relies on the base Exception.__init__ setting self.args, and later resurrects exceptions with class(*args). This change rewrites processutils.ProcessExecutionError so it survives a pickle.dumps/loads round-trip. Change-Id: I9b8d104f60df868be7b808c72c932d08f1752777
* | Updated from global requirementsOpenStack Proposal Bot2016-02-191-1/+1
|/ | | | Change-Id: I34f0b21afc4eba3a2503e3021ac121180d588948
* Updated from global requirementsOpenStack Proposal Bot2016-02-101-1/+1
| | | | Change-Id: Ibeffb23a0ef4f94d2f8f18f2fdebb491f88b3943
* Merge "Add prlimit parameter to execute()"3.4.0Jenkins2016-02-013-0/+250
|\
| * Add prlimit parameter to execute()Victor Stinner2016-01-283-0/+250
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new oslo_concurrency.prlimit module which is written to be used on the command line: python -m oslo_concurrency.prlimit --rss=RSS -- program arg1 ... This module calls setrlimit() to restrict the resources and then executes the program. Its command line is written to be the same than the Linux prlimit system program. Add a new ProcessLimits class processutils: resource limits on a process. Add an optional prlimit parameter to process_utils.execute(). If the parameter is used, wrap the command through the new oslo_concurrency prlimit wrapper. Linux provides a prlimit command line tool which implements the same feature (and even more), but it requires util-linux v2.21, and OpenStack targets other operating systems like Solaris and FreeBSD. Change-Id: Ib40aa62958ab9c157a2bd51d7ff3edb445556285 Related-Bug: 1449062
* | Update translation setupAndreas Jaeger2016-01-2810-7/+7
|/ | | | | | | | | | | | | | | Follow new infra setup for translations, see spec http://specs.openstack.org/openstack-infra/infra-specs/specs/translation_setup.html for full details. This basically renames oslo.concurrency/locale/oslo.concurrency.pot to oslo_concurrency/locale/oslo_concurrency.pot. For this we need to update setup.cfg. Update also domain name in i18n.py. Change-Id: Ie774c4252f8481f2541ed0a2cad565d149dfcc72
* Updated from global requirementsOpenStack Proposal Bot2016-01-261-1/+1
| | | | Change-Id: I5d9484bae360143185fe6efbc2633ce9b01614b1
* Updated from global requirementsOpenStack Proposal Bot2016-01-241-1/+1
| | | | Change-Id: I3396829a7b7d3c31e6cae816aefca222d1048cbc
* Updated from global requirementsOpenStack Proposal Bot2016-01-231-1/+1
| | | | Change-Id: I26110041758b218b7ce93c884082042019ee1d0e
* Updated from global requirementsOpenStack Proposal Bot2016-01-181-1/+1
| | | | Change-Id: Ia560a40275adf735f529c29c7ced64003ca2709b
* Merge "Updated from global requirements"Jenkins2016-01-162-10/+10
|\
| * Updated from global requirementsOpenStack Proposal Bot2016-01-162-10/+10
| | | | | | | | Change-Id: I0302364cb544b7e94ddb3f4f314ff0975a5b8ff2
* | Imported Translations from ZanataOpenStack Proposal Bot2016-01-166-46/+49
|/ | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: I42faa1a3e1c6119c33301d83a64000dc32dceb9e