summaryrefslogtreecommitdiffstats
path: root/openstack/common
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | kombu's fanout_cast_to_server was calling wrong methodChris Behrens2012-11-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make it call the common amqp method fanout_cast_to_server vs cast_to_server. Fixes bug 1074113 Change-Id: If06bde93dab94c6030465a5e7ba329d8936c69e3
* | | | | Move utils.execute to its own module.Michael Still2012-11-013-88/+124
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Also move the exceptions raised to being local to the executils module. Change-Id: Ibb2445f92840b9ce4b52373b0b09adb3d6a4a897
* | | | Merge "Don't log exceptions for GreenletExit and thread_done"Jenkins2012-11-011-11/+12
|\ \ \ \
| * | | | Don't log exceptions for GreenletExit and thread_doneAngus Salkeld2012-11-011-11/+12
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1) Greenletexit is normal so don't log it. 2) logging in a linked callback is just asking for problems. (we don't want to cause a context switch in this callback). Change-Id: Ia9bb1cabc65a742fc49beb379548eb680b326e58
* | | | Merge "Removes redundant string cast"Jenkins2012-11-011-1/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Removes redundant string castZhongyue Luo2012-10-311-1/+1
| |/ / | | | | | | | | | | | | | | | Removes uuid.UUID.hex result string casting Change-Id: I4b0ed3ec1e4dc4a1ade149a539c5068ac154ebf7
* | | Merge "move nova.common.deprecated to openstack-common"Jenkins2012-10-311-0/+18
|\ \ \
| * | | move nova.common.deprecated to openstack-commonSean Dague2012-10-311-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1070511 nova has a common deprecation path which lets you turn deprecation warnings into fatal errors via config. Make this a function of the openstack common log to make it available to other projects in a standard way. Include exception directly in logging module. Update exception to work correctly now that it's not based off of nova's exception class. Change-Id: I699e776c99c429e60dfb5cd6c0cdbb661bae0ce8
* | | | Add a missing comma in a docstring.David Ripton2012-10-311-1/+1
| |/ / |/| | | | | | | | Change-Id: I1f75c7da1ab1543637198ecbb80a81b39ad35fde
* | | Merge "Use queue_arguments for fanout consumers too."Jenkins2012-10-301-0/+1
|\ \ \
| * | | Use queue_arguments for fanout consumers too.Sam Morrison2012-10-261-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1071528 queue_arguments (currently only x-ha-policy for now) were only set for topic consumers, this adds support for rabbit HA for fanout consumers too. Change-Id: I0d5c0393b49cf249262d93a64f6348caa76ef8dd
* | | | Merge "Import order clean-up"Jenkins2012-10-308-10/+10
|\ \ \ \
| * | | | Import order clean-upZhongyue Luo2012-10-298-10/+10
| | |_|/ | |/| | | | | | | | | | | | | | | | | | Reorder imports by full module path Change-Id: If619b4b50aa052401f7fa65df57b74d4ad1b65a6
* | | | Merge "Fix a logic error in stable version calculation."Jenkins2012-10-301-3/+5
|\ \ \ \ | |/ / / |/| | |
| * | | Fix a logic error in stable version calculation.Monty Taylor2012-10-261-3/+5
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Milestone information in blank for stable branches, so there is no milestone prefix. However, we were blindly concatenating the milestone version prefix onto the datestamp. Fixes bug 1059634. Change-Id: I6cff4ee7f6c1dc970397b66fd2d15fa22b0a63a3
* / / cfg: fix required if option has a dashJulien Danjou2012-10-261-1/+1
|/ / | | | | | | | | | | | | | | | | If an option has a dash in it and is required, the check fails because it tries to self._get() on the name (with dash) rather than the dest (with underscore). Change-Id: I6448019f70f98bc2e58a325d0cf9ce88b8bb085b Signed-off-by: Julien Danjou <julien@danjou.info>
* / Fix missing import in lockutils.Michael Still2012-10-231-0/+1
|/ | | | | | This showed up while integrating these changes into nova. Change-Id: I48d846fc761c45cf97480a49c99d20817f1dea2d
* Move nova's util.synchronized decorator to openstack common.Michael Still2012-10-182-0/+267
| | | | | | | In the end I needed to port utils.ensure_tree as well. Resolves bug 1063230. Change-Id: I6e6fa8201de2cac3f17e6c60d7b16f7df7c64116
* Fix a couple of file handle leaks, using with statements.David Ripton2012-10-171-9/+11
| | | | | | | | | In practice unclosed file handle leaks are not a huge deal in CPython code, because reference counting means they get closed automatically in a timely manner. But they're bad style, and they can become a real problem if the code is run with a version of Python that doesn't use reference counting. Change-Id: Ie54b979e26ffc9dd405871ee07c50b304cb36c67
* Merge "Revert "fanout subscriptions in impl_zmq""Jenkins2012-10-162-78/+20
|\
| * Revert "fanout subscriptions in impl_zmq"Eric Windisch2012-10-112-78/+20
| | | | | | | | | | | | | | | | | | | | This changed the message format without consideration for versioning. If this is resubmitted, it will be only after message versioning is introduced. This reverts commit 032bf345ea593e97fcc07e23c1725ba5f4b612d0. Change-Id: I4f4326b0ba78699b5f149cc2d2822235e9a30462
* | Merge "Revert "Support declare_topic_consumer in impl_zmq""Jenkins2012-10-161-29/+0
|\|
| * Revert "Support declare_topic_consumer in impl_zmq"Eric Windisch2012-10-111-29/+0
| | | | | | | | | | | | | | | | | | | | | | | | Depends on reverted patch. Not used by any code or project. This was an implementation of an internal AMQP-driver method. Will determine long-term requirement or replacement for declare_topic_consumer at Grizzly sumit. This reverts commit d5095d7effe79c1dcf19a67ff03e00a454662830. Change-Id: I48883c5011ebccba3e6f987ae48bff871d016bd1
* | Fix WSGI Service init's call to incorrect super methodKiall Mac Innes2012-10-141-1/+1
| | | | | | | | Change-Id: I7554beefafc409a5504b3e33ac31b4394fe46533
* | Merge "Revert "Add support for finer-grained policy decisions""Jenkins2012-10-121-150/+2
|\ \
| * | Revert "Add support for finer-grained policy decisions"Mark McLoughlin2012-10-111-150/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 3fc46892 After a productive discussion here: http://lists.openstack.org/pipermail/openstack-dev/2012-October/thread.html#1566 It appears that we don't yet have a really compelling use case for this and folks are worried about the extra complexity it brings. I think it's safe to say there's a consensus that we shouldn't proceed with this idea yet. We can always revisit this again later if needs be.
* | | Merge "Added is_newer_than function"Jenkins2012-10-111-0/+5
|\ \ \ | |_|/ |/| |
| * | Added is_newer_than functionSirisha Devineni2012-10-091-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added is_newer_than function to compare if the provided time is newer than current time for specified number of seconds Fixes bug 1036343 Change-Id: Ic133b0e7e2337b6b0fdad244ded6a93f8db48379
* | | Remove an unneeded 'global'Kevin L. Mitchell2012-10-091-1/+0
| |/ |/| | | | | | | | | | | | | If we're not changing the value of a global variable, there's no need to use the 'global' statement. This removes an unnecessary 'global' statement from the register() function/decorator. Change-Id: I0d4388f28acf3c9dc5969d86d74f384f1f3d0474
* | Add support for finer-grained policy decisionsKevin L. Mitchell2012-10-091-2/+150
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint fine-grained-policy Add support to the new policy language for finer-grained policy decisions: policy checks can, using the "case" construct, return values other than simply "True". This means that a policy rule can be written that allows for fine shades of permissions; for instance, a rule could return the values, "fulladmin", "projectadmin", or False, and the caller can use these values to decide whether to allow all modifications, only some modifications, or no modifications at all. Change-Id: Id2bbee5c34052692af12e4a028256131ed2747fc
* | Add a 'not' operator to the policy langageKevin L. Mitchell2012-10-091-1/+41
| | | | | | | | | | | | | | | | | | Implements blueprint fine-grained-policy Inverting the sense of a check was not possible with the list-of-lists syntax, but it clearly makes sense to support it. Change-Id: Ibd92cd75a279efdafec16a26f9aec33f39614b5c
* | Add a new policy languageKevin L. Mitchell2012-10-091-4/+270
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint fine-grained-policy Add a new policy language with "and" and "or" operators to replace the old list-of-lists syntax. New '@' and '!' operators are also added. This new language will enable us add more advanced features than the old syntax would have allowed. Backwards compat support for the old list-of-list syntax is retained. Change-Id: I872cb6abf6f8051c3ff502a0fc7590cff4f63a25
* | Remove deprecated policy engine APIsMark McLoughlin2012-10-091-150/+1
| | | | | | | | | | | | | | | | | | Implements blueprint fine-grained-policy Remove the deprecated Brain/enforce API and support for using functions as check types. Change-Id: Idf79999ad402b2c79072d0547d60c7b0bfa20d5a
* | Rewrite the policy engine from scratchKevin L. Mitchell2012-10-091-180/+502
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint fine-grained-policy Complete rewrite of the policy engine, with careful thought given to backwards-compatibility. Policy rules are now represented internally by a tree of Check objects. A new API is added for parsing and enforcing rules: set_rules(Rules.load_json(data, default_rule=...)) result = check(rule, target, credentials) reset() The old Brain/enforce API is deprecated and will be removed soon. A new API is also added for registering new check types: @register("key") class KeyCheck(Check): def __call__(self, target, creds): return self.match in creds Support for using functions as check types is deprecated and will also be removed soon. Change-Id: I2951a0de3751bd2ec868e7a661070fed624e4af2
* Merge "Fix missing import of `rpc.dispatcher` in `rpc.service`"Jenkins2012-10-021-4/+5
|\
| * Fix missing import of `rpc.dispatcher` in `rpc.service`Kiall Mac Innes2012-10-021-4/+5
| | | | | | | | Change-Id: I7bdd5fbff1addef4a855aa72f3551930630299f5
* | Added a missing `cfg` import in service.pyKiall Mac Innes2012-10-021-0/+2
|/ | | | Change-Id: I7e98686cb72ba6287b93746267390cc79f18bb6d
* Change wsgi Server class to ServiceAngus Salkeld2012-09-291-18/+28
| | | | | | | | | Also: - use WritableLogger from common/log.py Part of blueprint service-infrastructure Change-Id: If41e592167bd4c308dd2a5d0226fcf3d8318f90d
* Ignore the stackforge jenkins email addressDoug Hellmann2012-09-281-2/+2
| | | | | | | | | | | Update generate_authors to ignore the email address for the stackforge jenkins as well as the main openstack jenkins, to ignore commits in the history of projects that were on stackforge before moving over to the main server (such as ceilometer). Change-Id: I86d93ba0f358d5d33278847ba8de039642dd54b7 Signed-off-by: Doug Hellmann <doug.hellmann@dreamhost.com>
* Merge "Log config on startup"Jenkins2012-09-271-0/+4
|\
| * Log config on startupAngus Salkeld2012-09-261-0/+4
| | | | | | | | | | | | | | Part of blueprint service-infrastructure Change-Id: I60698800231e5de349fdbcf9f9090e5b31727640 Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
* | Merge "Integrate eventlet backdoor"Jenkins2012-09-271-0/+2
|\|
| * Integrate eventlet backdoorMark McLoughlin2012-09-261-0/+2
| | | | | | | | | | | | Part of blueprint service-infrastructure Change-Id: Ia0511ae62497d95771ca3a894379486741e223ba
* | Merge "Add the rpc service and delete manager"Jenkins2012-09-273-93/+78
|\|
| * Add the rpc service and delete managerAngus Salkeld2012-09-263-93/+78
| | | | | | | | | | | | Part of blueprint service-infrastructure Change-Id: Ie00efcb1c974d699ac6b2a38285abef1b295f3ce
* | Add exchange_name parameter to TopicConsumerJulien Danjou2012-09-272-6/+11
| | | | | | | | | | | | | | | | | | | | | | In Ceilometer, we need to consume messages from different exchanges of type 'topic', so being able to specify the exchange name rather than using the default in TopicConsumer is necessary. This fixes bug #1055483 Change-Id: Ibbaec8496f24c0b221cf489258af1e7fe362e84a Signed-off-by: Julien Danjou <julien@danjou.info>
* | Extracted parse_host_port into network_utils.Eugene Kirpichov2012-09-263-46/+70
| | | | | | | | Change-Id: I77bcbf03a18659cfa62e99da9ba2136f8348022b
* | LOG.exception() should only be used in exception handlerJohannes Erdfelt2012-09-251-9/+10
| | | | | | | | | | | | | | When used outside of an exception handler it will print the message and then 'None', which isn't useful at all. Change-Id: If687a764dea4c7dc0046d7c3054318f2a751a2ea
* | Merge "Don't deprecate rabbit_host and rabbit_port"Jenkins2012-09-251-3/+4
|\ \
| * | Don't deprecate rabbit_host and rabbit_portMark McLoughlin2012-09-251-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Where a single RabbitMQ broker node is used (rather than a cluster of nodes) it is simplest for users to just set the rabbit_host option. Often users will not need to specify a port number at all. Similarly, where a cluster is used, it is simplest for the user to just specify a list of node addresses in rabbit_hosts and assume that all nodes are listening on the port specified in rabbit_port. Change-Id: I4227541676dffc50cd6116225de0cec0afa36a41