summaryrefslogtreecommitdiffstats
path: root/openstack
Commit message (Collapse)AuthorAgeFilesLines
* 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
|\
| * Support running uninstalled in a tarball dir.Monty Taylor2013-01-251-3/+10
| | | | | | | | | | | | | | | | | | | | Running tests from the directory of an expanded tarball is a reasonable thing to do, so support it by checking for PKG-INFO file if the pkg_resources version grab fails. Fixes bug 1103473 Change-Id: I2bf1fc39b070d3a2cdf6a3abe5e816c81d2916a0
* | Merge "Default lockutils to using a tempdir"Jenkins2013-01-291-3/+2
|\ \
| * | Default lockutils to using a tempdirJason Kölker2013-01-281-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Previously lockutils attempted to use __file__/../ as the lock path. This causes packages to fail if the user running the code doesn't have +w access to that dir. Remove the default so it will use mkdtemp instead * Fixes bug 1107950 Change-Id: Ie0cd6fac5403a53a768eb4a0ce7da602b3fef755
* | | Merge "Fix Pep8 Warning"Jenkins2013-01-291-1/+1
|\ \ \ | |/ / |/| |
| * | Fix Pep8 WarningDirk Mueller2013-01-281-1/+1
| | | | | | | | | | | | | | | | | | | | | Fixes E271 multiple spaces after keyword "return _run_shell_command(" Change-Id: Ic6b2c88a0b7cabd04341d073ef837a13c1bb7a29
* | | Add TTL to AMQP messagesBrent Eagles2013-01-283-10/+27
|/ / | | | | | | | | | | | | | | | | | | This patch sets a TTL derived from the timeout parameter used when invoking the RPC. Needed for: * Bug 1053614 Change-Id: Idaf9a2a972feac3f95f170c52632033b027b4414
* | Merge "Fixes "is not", "not in" syntax usage."Jenkins2013-01-283-7/+7
|\ \
| * | Fixes "is not", "not in" syntax usage.Zhongyue Luo2013-01-283-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | Replaced "not ... is" to "is not" Replaced "not ... in" to "not in" Removed a redundant parenthesis Change-Id: I9564ab1207ccdcb32d7c2bb9e8f29658b2232ff9
* | | Merge "Use testtools as test base class."Jenkins2013-01-281-68/+0
|\ \ \
| * | | Use testtools as test base class.Monty Taylor2013-01-241-68/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On the path to testr migration, we need to replace the unittest base classes with testtools. Replace tearDown with addCleanup, addCleanup is more resilient than tearDown. The fixtures library has excellent support for managing and cleaning tempfiles. Use it. Replace skip_ with testtools.skipTest Part of blueprint grizzly-testtools. Change-Id: I45e11bbb1ff9b31f3278d3b016737dcb7850cd98
* | | | Merge "Start adding reusable test fixtures."Jenkins2013-01-282-0/+37
|\| | | | |/ / |/| |
| * | Start adding reusable test fixtures.Monty Taylor2013-01-242-0/+37
| | | | | | | | | | | | | | | | | | | | | | | | Now that nova is using fixtures, we have grown some that can be used across projects. The mox/stubout fixture is a great example of this - and also can be used in the oslo tests themselves. Change-Id: Ia78019317a6772c5c66a875c948854248383f9ea
* | | setup: count revs for revno if there are no tagsMark McLoughlin2013-01-251-3/+17
| |/ |/| | | | | | | | | | | | | | | | | | | | | We currently use 'git describe --always' to figure out the number of commits there have been leading up to the HEAD commit. However, if there are no tags in the repo, git-describe just returns the sha hash of HEAD and we fail. In that case, use 'git rev-list HEAD | wc -l' so we can be #winning. Change-Id: I345e0ee32189504276b3dfd3367057ce1d4a2b06
* | Merge "UTC ISO8601 from timestamp"Jenkins2013-01-231-0/+5
|\ \
| * | UTC ISO8601 from timestampFlaper Fesp2013-01-211-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | Taken from glance xattr.py module. This function converts timestamp formated dates into a UTC Iso 8601 date. It is being required in multiple places now (mostly on clients printing dates) in order to make dates human readable. Change-Id: I6f19325a4c2df241a0ef76bd256280a2930d9265
* | | Import timeutils.is_soon from keystoneclientJulien Danjou2013-01-231-0/+13
| |/ |/| | | | | | | | | | | | | | | | | This is a function defined as will_expire_soon at least twice in python-keystoneclient, and it's actually quite handy. There is a comment on that code about moving to timeutils, and it actually sounds like a good idea since I'm likely to use it in Ceilometer, so here it is. Change-Id: Idf208ce9e7ce9048a38d98212b7f730c6b7c8288 Signed-off-by: Julien Danjou <julien@danjou.info>
* | Merge "Move logging config options into the log module"Jenkins2013-01-212-81/+67
|\ \
| * | Move logging config options into the log moduleMark McLoughlin2013-01-212-81/+67
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We learned a lesson in Nova - it's best to declare and use config options within a single module if possible. Globally declared and use config options grow like weeds and it becomes harder to find out if, where and how individual options are used. Strangely, in cfg itself, we randomly declare a bunch of logging options which are only used within the openstack.common.log module - let's move the options there and remove the CommonConfigOpts class before they become part of the API we commit to when oslo-config is released. A minor detail in the patch - the logfile and logdir options are already deprecated in favour of log_file and log_dir, but we never got around to removing all other traces of the deprecated options. Change-Id: I3913ea54465658d93dc56e014dfe5d911b0541d6
* | Use revno and git sha for pre-release versioning.Monty Taylor2013-01-182-11/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In discussions with ttx and zul, it became clear that the git describe info for our intra-release case wasn't providing any value and was providing confusion. Additionally, plain git sha's are not providing enough info. So, the scheme settled on that breaks the least amount of things and yet still reduces complexity is: if pre-version: # server projects try: use current tag except: use pre-version+revcount+gitsha else: use git describe | s/-/./ Additionally, we'll use a as a prefix for the revcount, b as the prefix for the milestone portion of a milestone tag, and rc as the prefix for rc's, so that for our releases, python version sorting works as expected. Change-Id: I6f0fe029d225afa8f435bc83216fc144c2402ae0
* | Add env var version override for packagers.Monty Taylor2013-01-181-0/+3
|/ | | | | | Fixes bug 1100805. Change-Id: I8c8ba8610ed2d9093e188c1412e9689314560015
* trivial pep whitespace fixSteven Hardy2013-01-181-1/+1
| | | | | | | | Fix multiple spaces after keyword pep8 error, causes our pep8 gate to fail (pep8 1.3.4) Change-Id: I67ab86ce17a1a91f8dab61efb18b83b22d2e1488 Signed-off-by: Steven Hardy <shardy@redhat.com>
* Merge "Remove write_requirements"Jenkins2013-01-181-9/+0
|\
| * Remove write_requirementsMonty Taylor2013-01-161-9/+0
| | | | | | | | | | | | | | Everyone hated this, so we decided to get rid of it at the last summit. Actually, come to think of it, I think we decied to get rid of it in San Fran. Change-Id: I7d93af1eb98a37ff018730669731a7ee22516279
* | Merge "ThreadGroup remove unused name parameters"Jenkins2013-01-182-7/+5
|\ \
| * | ThreadGroup remove unused name parametersSteven Hardy2013-01-162-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Thread/ThreadGroup constructors take a name parameter which is never used, so we can simplify things a bit by removing it. If anything is referencing the internal self.name it should be using its own name->threadgroup mapping (e.g via a dict or similar container class) Change-Id: I6877a2fcee60ce9fc087d38aa492d7c9d50ce97e Signed-off-by: Steven Hardy <shardy@redhat.com>
* | | Merge "use regexp to parse the mailmap"Jenkins2013-01-171-5/+6
|\ \ \ | |_|/ |/| |
| * | use regexp to parse the mailmapIonuț Arțăriși2013-01-161-5/+6
| | | | | | | | | | | | | | | | | | | | | this also makes parse_mailmap() work with the swift .mailmap and should be more readable Change-Id: I3abb2a18d655e507bee5f621ebc64421d943cf0b
* | | Stop using no longer valid -E option for pipJohannes Erdfelt2013-01-161-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1100403 Version 1.1 of pip dropped the -E option. It's not necessary anyway since running pip in a correctly setup virtualenv will automatically do the right thing. Change-Id: Ifec9f27b8dc169d594125a0808126abe77ed42e7
* | | Fixes an import order nitZhongyue Luo2013-01-161-1/+1
| |/ |/| | | | | Change-Id: Ic47deb67c5dad123c8937e3468d798eefe045280
* | Merge "Make tox run doctests"Jenkins2013-01-152-8/+10
|\ \
| * | Make tox run doctestsMark McLoughlin2013-01-152-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use 'nosetests --with-doctests' to run any doctests found. We currently only use doctests in a handful of places, but we may as well run them to ensure they work. Make the cfg doctests avoid using the global CONF since we would need to reset its state between each doctest. Fix the cliutils doctests to actually pass. Use 'nosetests --exclude-dir=tests/testmods' to avoid loading the modules from this dir while discovering doctests. The cfg unit tests rely on these modules not having been previously loaded. Change-Id: I19ad70767fa5c8352b994dc963b5d3a7c9d9eb95
* | | Merge "Allow rpc drivers to always be imported"Jenkins2013-01-152-10/+20
|\| |
| * | Allow rpc drivers to always be importedMark McLoughlin2013-01-152-10/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The qpid and zmq drivers current can't be import unless their respective libraries are installed and neither are currently listed in test-requires. This prevents us from being able to use nosetests --with-doctests because the import failure is seen as a test failure. Fix this issue by making the import always succeed and deferring failure until the module's APIs are actually used. This will also help Nova's sample config file generation since it will allow the generate-sample tool discover config options for these drivers even when the messaging libraries aren't installed. Change-Id: I30087fd33eef41d50d1a374cb5694dc15c43d1da
* | | Merge "Rework zmq setup and cleanup"Jenkins2013-01-151-26/+18
|\| |
| * | Rework zmq setup and cleanupMark McLoughlin2013-01-151-26/+18
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Register config options at import time and remove the register_opts() function. It seems this isn't being used. Add accessor methods for ZMQ_CTXT and matchmaker so that they are lazily created rather than created at import time. Remove the rpc_zmq_matchmaker override in the tests since MatchMakerLocalhost is the default anyway. We were setting it with an incorrect value, but only after the matchmaker had already been created. Change-Id: I2520252c64d70e4c4903e34d07952fed43e70ebe
* | Merge "Fix zmq socket.close() with eventlet 0.9.17"Jenkins2013-01-151-3/+7
|\ \ | |/ |/|
| * Fix zmq socket.close() with eventlet 0.9.17Eric Windisch2013-01-131-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A bug/change in eventlet 0.9.17 blocks the kwargs from close() from passing through to pyzmq. As this code was only explicitly passing defaults, no argument is necessary. This addresses file descriptor leaks reported in bug 1099185. This bug does not affect installations with eventlet<=0.9.16 The code will now log an error if a socket cannot be closed. Change-Id: Id438dab9744042328b7fac44693d02e116d8c1d9
* | Merge "Add missing parameter to impl_fake notify function."Jenkins2013-01-151-1/+1
|\ \
| * | Add missing parameter to impl_fake notify function.Gary Kotton2013-01-141-1/+1
| |/ | | | | | | | | | | The envelope was missing. Change-Id: I3ab8f573e6c5020bd564ad78de29e6c285695f2c
* | Merge "Implement importutils.try_import."Jenkins2013-01-152-2/+10
|\ \
| * | Implement importutils.try_import.Dan Prince2013-01-142-2/+10
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a new (simple) importutils.try_import function which can return a module name or a default (by default this is None). This should help clean up some of our try: import foo except... blocks in modules. This commit also drops the dependency on python-extras which is no longer needed. Fixes LP Bug #1099501. Change-Id: I8591f79983bdad67a50d1aaae6dce8428dfce084
* | Merge "reusable ZmqProxy service initialization"Jenkins2013-01-151-0/+33
|\ \
| * | reusable ZmqProxy service initializationEric Windisch2013-01-131-0/+33
| |/ | | | | | | | | | | | | | | | | | | | | Move near-identical proxy setup code from oslo-zmq-receiver and test_zmq into ZmqProxy's consume_in_thread method Provides code reuse and simplifies the tests and receiver binary. Change-Id: I1324eacfa2a456599d5fd462b6476ddf659e95c4
* | Merge "call() to accept serialize and force_envelope"Jenkins2013-01-151-3/+6
|\ \ | |/ |/|
| * call() to accept serialize and force_envelopeEric Windisch2013-01-101-3/+6
| | | | | | | | | | | | | | | | | | | | The serialize and force_envelope options were being stripped from call(). Now, when call() is invoked, these options are passed and respected. Change-Id: I69355606b06ab2ec87765fe6ddba7baaa68e5754
* | Merge "Verbose should not enable debug level logging"Jenkins2013-01-132-4/+8
|\ \
| * | Verbose should not enable debug level loggingDavanum Srinivas2013-01-122-4/+8
| |/ | | | | | | | | | | | | | | | | | | | | | | | | Fixes LP #989269 Currently setting --verbose in will still allow DEBUG level message to be logged to python logger object. we need to check for --debug first (set DEBUG level), then --verbose (set INFO level) and if neither is set then set default to WARNING DocImpact Change-Id: Ic9e3cb5979b2d7283552ad3a461870373f45a239
* | Merge "zmq-receiver msg forwarding in greenthreads"Jenkins2013-01-121-20/+48
|\ \