summaryrefslogtreecommitdiff
path: root/ceilometerclient/tests/test_client.py
Commit message (Collapse)AuthorAgeFilesLines
* Add timeout for keystoneclient sessionZhiQiang Fan2015-04-201-0/+13
| | | | | | | | | | | | Currently, we only apply timeout on ceilometer api, but not on keystone api as well, this is not consistent. Keystone api may be slow, or worse, stuck, then ceilometerclient will not return timely. This issue has caused availability problem in our environment when ceilometer alarm service has been running for a long time. Change-Id: I0d5f82ff9cf4132a1de2f0b649908483326e116e Closes-Bug: #1436249 (cherry picked from commit 44af2d4087815d5d451305b549f91ab6130528c0)
* add region_name to auth plugin parametersZhiQiang Fan2015-04-201-0/+1
| | | | | | | | | | If endpoint is not specified, then auth plugin will request to keystone to get target endpoint, however, we don't provide region name for it, this will cause critical issue in multiple region scenario. Change-Id: I99ceafec25072988435018ca1affbd778ca9b22f Closes-Bug: #1439553 (cherry picked from commit bfb6d122fa99d94a61aea4432f74cbbf3f5d03cc)
* ceilometerclient insecure argument no longer worksMatthew Edmonds2015-04-201-0/+9
| | | | | | | | | The os_insecure argument needs to be parsed and passed to the auth plugin for non-shell usage. Change-Id: Id45a680396a9b646b273d971a0f6c274e95019a6 Closes-Bug: #1438750 (cherry picked from commit a5df77d8730284e278fde087e8bb3c2520c6ddb1)
* Fixes bug with Client function not setting up SSL paramsSrinivas Sakhamuri2015-04-161-4/+11
| | | | | | | | | | | There are instances where some applications (heat, rally) instantiate ceilometer client using Client function (instead of get_client) which doesn't use the insecure and verify params properly. This fix will address that. Change-Id: I2f6346496e6ee36b5cd97ec6d9a732431532ff96 Closes-Bug: #1428370 (cherry picked from commit e7b8c48c60e85378e1e1d3c30bfb4cfa79f3105a)
* Fix improper parameter setup for cacert and client certsSrinivas Sakhamuri2015-01-151-2/+16
| | | | | | | | | | | | | The client passes cacert incorrectly, CA cert need to be passed in verify parameter to requests library. Also at present, key file is not being used, which if it is supplied need to be passed as a tuple in cert parameter Python requests library relevant doc link http://docs.python-requests.org/en/latest/user/advanced/#ssl-cert-verification Change-Id: Ie71e760bce6a78ed1444373adfa0a133e82ec434 Closes-Bug: 1408372
* Merge "Enable --os-insecure CLI option"Jenkins2015-01-071-1/+1
|\
| * Enable --os-insecure CLI optionZhiQiang Fan2014-12-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | --os-insecure is not correctly passed to Keystoneclient because it expects a bool type but we assgin a string value to the insecure parameter, this patch fixes it by using oslo.utils.strutils.bool_from_string. --os-insecure is ignored by Ceilometerclient.v2.client because it expects parameter verify rather than insecure, this patch fixes it by converting insecure to verify if that field is not set. Change-Id: I545de167cdf8cfec6b5c67984b74327fb76c513a Closes-Bug: #1394449
* | Make methods static where it's possibleDina Belova2014-12-101-1/+2
|/ | | | Change-Id: If6c1fa05044e65ec1e72a3e5f63e6db826469a2d
* Merge "Fix wrong initialization of AuthPlugin for keystone v3"Jenkins2014-11-251-7/+34
|\
| * Fix wrong initialization of AuthPlugin for keystone v3ZhiQiang Fan2014-11-111-7/+34
| | | | | | | | | | | | | | | | Ceilometerclient doesn't pass domain related arguments to AuthPlugin, which will cause keystoneclient v3 cannot be used properly. Change-Id: I2554a36f214c0eae61b7ba6b8580c41b8f8c6370 Closes-Bug: #1391122
* | Fix timeout argument not treated as integerZhiQiang Fan2014-11-141-0/+35
|/ | | | | | | | | | | | Currently, if we specify --timeout 30 in CLI commands. we will get 'a float is required' error. Since we already specify timeout is ingeter in ceilometer.v2.client, this patch converts CLI argument timeout to integer. Note, if zero is passed to v2.client, we treat it as disable timeout. Change-Id: I213b6c24964095297cb4e87c25bb871ebd668bb4 Closes-Bug: #1391606
* Use HTTPClient from common Oslo codeekudryashova2014-07-311-23/+40
| | | | | | | | | In the process of unification of the clients code we should reuse common functionality from Oslo. bp common-client-library-2 Change-Id: I0e027c33ee42b6de032d33269caeea33e7837f40
* Update python-ceilometerclient to support Keystone V3 APIFabio Giannetti2014-06-261-1/+1
| | | | | | | | | | | | | | Keystone is now deprecating the V2 API in favor of the V3 API. The ceilometer client is now using the keystone session to delegate the discovery of the version that is going to be used, this eliminates the need of updating the client every time there is a new API version. DocImpact blueprint support-keystone-v3-api Change-Id: I6ebacce7adf70f24bdede9b920853ab9851071cf
* Avoid dead loop when token is string formatZhiQiang Fan2014-04-261-8/+10
| | | | | | | | | | The token needs to be a callable object in http.Client, so we assign a lambda expression to it when it is a string, but current implementation will cause dead loop since it is x = lambda: x. This patch fixes it and improves the corresponding test code. Change-Id: Id41edd705b46196404e5cb3483a290074bde12d0 Closes-Bug: #1301877
* Checking the type of auth_token, and fixing it if necessaryMarc Solanas2014-03-131-0/+55
When creating a new Client object by calling the get_client function, if the ceilometer_url is not passed as a parameter, but the token is passed, the auth_token is converted to a lambda function. However, when both the ceilometer_url and the auth_token are passed, this conversion does not take place. This fix checks if the auth_token is a lambda function or not. If it is not, it creates a new lambda function which returns the token. Without this check, if a token was passed as a string, the python client would throw and error, when trying to call self.auth_token() in ceilomenterclient/common/http.py. Change-Id: I3e564fcf17ce497546b75f0ab8c7ca0f6f2099df Closes-Bug: #1254186
View Lorry Controller Status