summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Updated from global requirements2014.2.b2OpenStack Proposal Bot2014-07-222-2/+2
| | | | Change-Id: Ic983d2b1bf5e602b6ae762b5ef37d6384023c5a1
* Add tests.unit.db.sqlalchemy.test_migrate to py33Felipe Reyes2014-07-201-0/+1
| | | | | | | This test module is running fine in Python 3.3, so it's being added to tox.ini Change-Id: I9e40e24ef081fb6299fa2e1b3dfe777f391ead21
* Merge "Fix "tests.unit.test_excutils" test with python 3.4"Jenkins2014-07-181-1/+4
|\
| * Fix "tests.unit.test_excutils" test with python 3.4Romain Soufflet2014-07-181-1/+4
| | | | | | | | | | | | Mocking time.time() failed because python 3.4 did not pass "self" argument. Change-Id: Icb9874250766eff20b8da0844a4fae33816088cb
* | Inherit MockWithCmp from Mock instead of MagicMockFelipe Reyes2014-07-182-1/+20
|/ | | | | | | | This patch changes the base class for MockWithCmp and implements necessary methods to compare instances, because mock.MagicMock in python 3 raises NotImplemented exception on __lt__, __gt__, etc. Change-Id: I56f774a5bc27726dae66acb70bf12bc618cb96fb
* Merge "Add Kilo release name to versionutils"Jenkins2014-07-181-0/+2
|\
| * Add Kilo release name to versionutilsMorgan Fainberg2014-07-171-0/+2
| | | | | | | | | | | | | | Add the newly minted Kilo release name for the upcoming K cycle to the versionutils module. Change-Id: Ie1bada1ddca8110d25345b80e1686d9e3f0d3bd3
* | Merge "Fix tests.unit.reports.* with python3.4"Jenkins2014-07-176-31/+32
|\ \
| * | Fix tests.unit.reports.* with python3.4Romain Soufflet2014-07-166-31/+32
| | | | | | | | | | | | | | | | | | Sort all reports alphabetically to avoid dict key order issues in python2 or python3. Change-Id: I3ceeba831ad93980ecf889e28bfdabf6204a9489
* | | Merge "Add default log level for websocket"Jenkins2014-07-161-1/+1
|\ \ \ | |/ / |/| |
| * | Add default log level for websocketabhishekkekane2014-07-091-1/+1
| | | | | | | | | | | | | | | | | | | | | Added default log level as warning for websocket module. blueprint: update-to-latest-websockify Change-Id: I4d9945ed9488d84276b1f76994f712ca8792a821
* | | Merge "Make logging_context_format_string optional in log.set_defaults"Jenkins2014-07-162-14/+11
|\ \ \
| * | | Make logging_context_format_string optional in log.set_defaultsJoe Gordon2014-07-152-14/+11
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | Instead of supporting setting logging_context_format_string to None, which will break things. Support not specifying logging_context_format_string in log.set_defaults so projects can set the default_log_level Change-Id: I932d5b2148a57c5b59ab3f071e53c87ef43b6a6e
* | | Merge "Fix deletion of cached file for policy enforcer"Jenkins2014-07-164-2/+36
|\ \ \
| * | | Fix deletion of cached file for policy enforcervponomaryov2014-07-114-2/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Policy ENFORCER's method 'clear' does not really delete cached file as written in its docstring. This commit adds 'delete_cached_file' to fileutils and makes 'clear' method of ENFORCER use it. Change-Id: I855d5f2001ac721249217d9fbd782c64af9db874 Closes-Bug: #1340843
* | | | Merge "Allow deprecated decorator to specify no plan for removal"Jenkins2014-07-162-5/+70
|\ \ \ \ | |_|/ / |/| | |
| * | | Allow deprecated decorator to specify no plan for removalMorgan Fainberg2014-07-122-5/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow the @deprecated decorator to specify functionality as deprecated but not slate any release for removal of the deprecated functionality. Change-Id: Iff24553ec83c4a8b0faf4b6381b6fe1589a1c089
* | | | Merge "cfgfilter has graduated, remove it"Jenkins2014-07-154-613/+0
|\ \ \ \
| * | | | cfgfilter has graduated, remove itMark McLoughlin2014-07-134-613/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cfgfilter is included in the oslo.config 1.4.0.0a2 release. No projects use the version in oslo-incubator, so we don't have to worry about waiting for projects to adopt it. Change-Id: I5bb5bd39c64538f6d5914c28a018e76cde0c0e8d
* | | | | Merge "log: make set_defaults() tests clean up properly"Jenkins2014-07-151-0/+8
|\ \ \ \ \ | |/ / / / | | / / / | |/ / / |/| | |
| * | | log: make set_defaults() tests clean up properlyMark McLoughlin2014-07-131-0/+8
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A call to log.set_defaults() changes the 'default' attribute on the option definitions statically defined in the log module. Currently, the tests for this function do not restore the original default values when the test completes. This only becomes an issue when the tests are called in a sequence like this: - test_change_default_log_level() sets default_log_levels=['foo=bar'] - test_error_notification() subsequently calls log.setup() which tries to do logger.setLevel('bar') because it's using the bogus default from the previous test Change-Id: Id020483049f46b78f834381f37d4ad6c75d49d25
* | | Add JUNO as a target to versionutils moduleMorgan Fainberg2014-07-122-4/+12
|/ / | | | | | | | | | | | | Add the JUNO release to the versionutils module for the @deprecated decorator. Change-Id: I2d4325a0e395739ba2d157d22f7c70d3492b02cd
* | Fix tests.unit.reports.test_base_report with python3.4Romain Soufflet2014-07-091-3/+2
|/ | | | Change-Id: I02c60b4deb01946b10aee460bd609be568ea20c4
* Fix exception message in openstack.common.processutils.executevponomaryov2014-07-081-2/+1
| | | | | | | Exception message in 'execute' function has wrong value for module and function. Remove it as wrong and redundant. Change-Id: Ia5bb4180dbf0df92bfb986c2ddf8327d5bc08e7c
* Merge "Ability to customize default_log_levels for each project"Jenkins2014-07-072-14/+30
|\
| * Ability to customize default_log_levels for each projectDavanum Srinivas2014-07-052-14/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | Add a new parameter default_log_levels to log.set_defaults. We need to default its value to None since this is a backwards incompatible change otherwise. However, if someone is not setting a specific value, we do need to set the usual default levels exposed DEFAULT_LOG_LEVELS public variable so others can use the default log levels to construct their custom defaults. Change-Id: I49a82c5b3446784d254ca558f51b6c3e5c2028eb Closes-Bug: #1297950
* | Backport code for i18n to check lazy at runtimeJames Carey2014-07-072-65/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently in the oslo-incubator version of i18n support the TranslationFactory generates a direct translation method based on the passed lazy parameter. This means that when the default is assigned to _ during its intial import, lazy isn't specified, which defaults to False, so the TranslationFactory returns a method that does not support lazy translation. The _ method is not changed until enable_lazy is called, thus any import of _ prior to that will cache the initial method that does not support lazy translation even when it has been enabled. This problem was exposed by the cinder test case: cinder/tests/test_wsgi.py test test_cinder_exception_with_localized_explanation when the file: cinder/api/middleware/fault.py had the line: from cinder.openstack.common.gettextutils import _ added to it. This behavior is corrected in the i18n library such that the generated method checks the global _lazy.USE_LAZY to determine if lazy translation should be used in commit: Iae22668 Check the lazy flag at runtime Note that this removes the lazy parameter from the signature of install and the TranslationFactory to match the library. Change-Id: If1832b012d4b72708d0850367a1ec55cb0cd63e8
* | Merge "Updated from global requirements"Jenkins2014-07-072-4/+4
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2014-07-072-4/+4
| | | | | | | | | | | | Change-Id: I885991acc06cf903604d3d6ab59284ac337a04af
* | | Merge "Added missing jsonutils.dump() function"Jenkins2014-07-072-0/+13
|\ \ \ | |/ / |/| |
| * | Added missing jsonutils.dump() functionIhar Hrachyshka2014-07-072-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | The function is missing from the module, meaning that some module consumers can't use jsonutils as a complete replacement for stdlib json module. Change-Id: Ia678d251a3984c959ae6c9d7ad70d33b197da4ff Closes-Bug: 1338571
* | | Merge "Python 3: enable tests/unit/test_log.py"Jenkins2014-07-063-2/+8
|\ \ \ | |_|/ |/| |
| * | Python 3: enable tests/unit/test_log.pyCyril Roelandt2014-07-043-2/+8
| | | | | | | | | | | | Change-Id: I2ae861b73cf32c4f732a31ae93f7a86b05890fbb
* | | Delete 'deprecated' directoryRomain Soufflet2014-07-041-0/+0
|/ / | | | | | | | | | | Not used anymore. Change-Id: Ief39415e432e166e334755d65fb063f0264468b1
* | Merge "Allow test_lockutils to run in isolation"Jenkins2014-07-031-0/+1
|\ \
| * | Allow test_lockutils to run in isolationThomas Herve2014-07-031-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While trying to port oslo to python 3, we discovered that test_lockutils wasn't working when not run with the global test suite. The fix can be tested running: $ python -m openstack.common.lockutils python -m testtools.run \ tests.unit.test_lockutils Change-Id: I51461f63cdd8e7eb6f8b6fdfa838748a582ecdad
* | | Allow test_service to run in isolationThomas Herve2014-07-031-0/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | While trying to port oslo to python 3, we discovered that test_service wasn't working when not run with testr. SIGHUP tests expect processes to be a daemon, which is not the case when run directly in process. The fix can be tested running: $ python -m openstack.common.lockutils python -m testtools.run \ tests.unit.test_service Change-Id: Ie45813668e658789cc133387fce7cd643ec42bb1
* | Merge "Changes calcuation of variable delay"Jenkins2014-07-021-4/+4
|\ \
| * | Changes calcuation of variable delayZhongyue Luo2014-07-021-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the loopingcall.FixedIntervalLoopingCall class, an warning is logged when 'delay' is equal or less than zero. Commonly delayed time is considered something over due from a deadline which is a positive value. Also a task delayed zero seconds is not considered delayed but finished on time. Therefore this patch fixes the calculation of the variable 'delay' and omits a warning log when 'delay' is zero Change-Id: I9dbae0838e6c03fa0eaa79e7332e650b02ed9c77
* | | Merge "update new requests logger to default WARN"Jenkins2014-07-021-1/+2
|\ \ \
| * | | update new requests logger to default WARNSean Dague2014-07-011-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The requests library changed the name of their logger, so now the terrible logging is back in all the projects. Set the new logger level to WARN as well as the old name. Change-Id: Ifbc22d097952cae4537fd97a96985bbb42c1b76c
* | | | Merge "Use timestamp in loopingcall"Jenkins2014-07-022-18/+22
|\ \ \ \ | | |/ / | |/| |
| * | | Use timestamp in loopingcallZhongyue Luo2014-07-022-18/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All time related data in loopingcall are in seconds. Therefore creating datetime objects only adds overhead to the module. This patch replaces all datetime objects with utc timestamps in loopingcall. The rational of using timeutils was disscussed here, https://review.openstack.org/#/c/16310/1..5/openstack/common/loopingcall.py,cm which was to use set_time_override to avoid actual time delays in test scripts. Now that set_time_override is deprecated, we no longer need to use timeutils. A lambda function returning time.time() was declared due to mock collisions with time.time() called in the logging module. Changes to test scripts were made accordingly. Change-Id: I38749f3c7bfff23164481b2341d56154ba737765
* | | | Mark gettextutils obsoleteDoug Hellmann2014-07-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The first version of oslo.i18n has been released, so we can mark gettextutils as obsolete. Change-Id: I56ea6ecc2cba5f3b4614189bbfa366efff29e50e
* | | | Merge "Use lockutils wrapper to set OSLO_LOCK_PATH for run_tests.sh"Jenkins2014-07-021-1/+1
|\ \ \ \
| * | | | Use lockutils wrapper to set OSLO_LOCK_PATH for run_tests.shIan Wienand2014-06-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use lockutils wrapper to get OSLO_LOCK_PATH set for tests. This is required for run_tests.sh to pass Change-Id: I276b07ca41582c04899b5fbc0632c056463ab4c7 Closes-Bug: #1303584
* | | | | Merge "Remove `processutils` dependency on `log`"Jenkins2014-07-021-6/+5
|\ \ \ \ \
| * | | | | Remove `processutils` dependency on `log`Flavio Percoco2014-07-021-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the processutils module depends on openstack.common.log. This patch proposes to remove this dependency and use the standard library logging module instead. The downside of this patch is that it removes the dependency on the log module but adds a new dependency on the strutils module because of the `mask_password` call. Change-Id: I245750f9f397a231891514f6ed1ea9626513cbbb
* | | | | | Merge "Move `mask_password` to strutils"Jenkins2014-07-024-296/+301
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Move `mask_password` to strutilsFlavio Percoco2014-07-024-296/+301
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `make_password` operates on strings to hide passwords based on some regular expressions. Since this function is not directly related to logging but string, this patch proposes moving the function into the strutils module. Change-Id: I9fe7f56cf6681099783f81d0a1b219ff10e95ace