summaryrefslogtreecommitdiff
path: root/ceilometerclient/client.py
Commit message (Collapse)AuthorAgeFilesLines
* [bugfix]with keystone v3 Could not find domain: default.zhangjianfeng2017-05-091-6/+2
| | | | | | | | | | | | | | | when we use adminrc with keystone v3 not set OS_PROJECT_DOMAIN_ID OS_USER_DOAMIN_ID,default ceilometerclient code will set it to string default.this will lead error. value is often like fdc79c4b63e8492d98271412320e8672 so when change this code to set it as None. last we delete the user_domain_id and project_domain_id because it is not useful. closes-bug: 1679934 Change-Id: I95c994406ed7a0b15ba6446e80f01e4f6787ec20
* Adjust parameters order of tenant_xxx and project_xxxJeremy Liu2017-02-041-1/+1
| | | | | | We prefer project_xxx to tenant_xxx when authenticating. Change-Id: I438854e6c9a9ea397ae4e7f21919a82c872f9693
* Merge "panko redirect"Jenkins2017-01-231-1/+2
|\
| * panko redirectgord chung2017-01-111-1/+2
| | | | | | | | | | | | same as aodh but with panko_endpoint Change-Id: Iae7d60e1cf139b79e74caf81ed7bdbd0bf2bc473
* | Adding default project and domain if nothing is specifiedPrateek Khushalani2016-12-151-1/+9
|/ | | | | | Closes-Bug: 1543502 Change-Id: Iea57a6ed757bfa28eb09324378b5376667b0c120
* move old oslo-incubator code out of openstack/commonSteve Martinelli2016-11-041-3/+3
| | | | | | | | | | | | | | As part of the first community-wide goal, teams were asked to remove the openstack/common package of their projects if one existed. This was a byproduct of the old oslo-incubator form of syncing common functionality. The package, apiclient, was moved to a top level location and a new i18n module was created. There is no oslo.apiclient library or equivalent, the recommended solution is to move it in tree and maintain it there. Change-Id: Ia788313e0926dc872a87b090ef0a350898bfb079
* Make method import_versioned_module workLuong Anh Tuan2016-11-031-2/+3
| | | | | | | | | Update function import_versioned_module in Oslo.utils 3.17. This patch update to meet new version. For more information: http://docs.openstack.org/developer/oslo.utils/history.html Change-Id: Ia031b53b27fcea66dbf4f2798d58b51c53c2d0cc Closes-Bug: #1627313
* Fixes SSLError during cclient.meters.list() by https2.7.0Ekaterina Khripunova2016-10-171-2/+2
| | | | | | | | | | | | The problem causes the function _adjust_kwargs(kwargs) which creates a new dict client_kwargs with new-named keys. It gets 'os_insecure' key from kwargs and gives it key-name 'insecure'. But the bug is in using kwargs.get('insecure') <which is None> to produse value of 'verify'. Change-Id: If77b2d3c75beddcd1a0a82353b56c84b29184ec7 Closes-Bug: 1634027 Related-Bug: 1394449
* Merge "Switch to keystoneauth"Jenkins2016-06-231-8/+8
|\
| * Switch to keystoneauthClenimar Filemon2016-06-011-8/+8
| | | | | | | | | | | | | | keystoneclient.auth is deprecated in favour of keystoneauth. This patch updates Ceilometerclient accordingly. Change-Id: Id4b14cbccfabfeaa5322f7ae5702261d0ed9cca2
* | Ignore aodh_endpoint argumentrabi2016-06-071-0/+2
|/ | | | | | | | | When using keystone session, we seem to pass all **kwargs to SessionClient for metering. We should drop the 'aodh_endpoint' from kwargs. Change-Id: Iaf28d4a525d49f62a0ae635dc1102cc1f0308c74 Closes-Bug: #1589425
* Trivial: remove `None` as a redundant argument to dict.get()Chaozhe.Chen2016-02-221-1/+1
| | | | | | | `dict.get()` returns `None` by default, if a key wasn't found. Removing `None` as second argument to avoid redundancy. Change-Id: I898219110206e9c49348f9a36f07b5ba1766f0f1
* Only token or creds are required.Mehdi Abaakouk2016-01-131-9/+30
| | | | | | | | | When token is provided and not the endpoint, the client works, because keystoneauth/keystoneclient will use the service_type to discover the endpoint. Closes-bug: #1533712 Change-Id: I0cf1ba52f656a450d358b2356df67d004f9aba7d
* Don't copy the auth_plugin for aodhMehdi Abaakouk2016-01-071-75/+1
| | | | | | | | | | | | | We must not copy a keystone session object or auth_plugin object. This change simplifies the aodh redirection code by: * sharing the keystone session object between aodh and ceilometer client * creating a new AuthPlugin() for the aodh client instead of copying the ceilometer one and then change its content. Closes-bug: #1531452 Change-Id: I2b2195e32c5dd74136237f7166c9c0d325434611
* Merge "Token will be lost when constructing a ceilometer client"2.1.0Jenkins2015-11-301-1/+3
|\
| * Token will be lost when constructing a ceilometer clientxiangjun li2015-11-271-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | If parameters os_token and os_endpoint were passed to ceilometerclient.client.get_client to construct a ceilometer client, the token will be wrongly replaced with None in method ceilometerclient.client._adjust_kwargs, which will cause the authentication error when use the instance of ceilometer client later. This patch fixed this problem. Change-Id: Ib15cdd73b62b7f0b86d0bd6f19d344fd24a665f6 Closes-Bug: #1519729
* | Merge "Ensure metering is the default service_type"Jenkins2015-11-271-2/+2
|\ \
| * | Ensure metering is the default service_typeMehdi Abaakouk2015-11-201-2/+2
| | | | | | | | | | | | Change-Id: Ia5e924eb58aac7fd53e9fe4a3dbdee102ece3ad7
* | | Merge "Correct an api reference error"Jenkins2015-11-261-1/+1
|\ \ \ | |/ / |/| |
| * | Correct an api reference errorLianhao Lu2015-11-251-1/+1
| |/ | | | | | | | | | | | | Correct a nit doc string error which is populated into api reference doc. Change-Id: I53e2d5ea820891d90f522889743ce94802f0576e
* | Ignores endpoint_type if interface is providedMehdi Abaakouk2015-11-201-2/+2
|/ | | | | | | When a keystone session object is used, if endpoint_type and interface is provided, we must use interface and drops endpoint_type. Change-Id: I379e18f684aa5092f8adbeaebe98030053115767
* Remove double API requestsMehdi Abaakouk2015-11-201-1/+0
| | | | | | | | When a keystone session object is used API call are done twice. This change fixes that. Change-Id: I79815520b996bce954fe531193f1a0229ef30060
* fix typos in docstringZhiQiang Fan2015-10-191-6/+6
| | | | | | | | | | | | | patch I4804c45c2ba69982163503ba5d9d4eba86f904fe has some typos in docstring: * :param *args: should be :param args:, there is no need to use the leading star, meanwhile the star has special meaning in doc string which need to appear by pairs. * :param for this ceiloclient, actually this phase is part of last param's help string Change-Id: Ib7b964ffe064b805130953f6a6abca354d9eef79
* Move to keystone session objectMehdi Abaakouk2015-10-071-1/+108
| | | | | | | | | | | | ceilometer client is a rare client that doesn't allow to use a keystoneclient/keystoneauth session object. This change fixes that and will allow to remove the deprecated apiclient code in the future. Depends-On: I43223a8b38ff1f4ef30c1d2345a28de6ade51863 Depends-On: I7cbe0afa12c9d11ce2eaed1f4d29f41507e273f8 Change-Id: I4804c45c2ba69982163503ba5d9d4eba86f904fe
* tenant_id not required with keystone v3Rohit Jaiswal2015-07-311-2/+8
| | | | | | | | | | Ceilometer-client requires tenant_id when authenticating with keystone v3. This fix removes this requirement. Change-Id: I36bf3f7a75713976c56dd95b4fd22da4e64d01df Closes-Bug: 1440463
* Add support for client redirecting to aodh endpoint if availableliu-sheng2015-07-221-27/+104
| | | | | | | | Currently, alarm module of ceilometer has been decided to be splitted out of ceilometer tree. To keep compatibility for others upstream users (such as Heat), we need to redirect alarm related request to aodh endpoint. Change-Id: Id13b56eef55f807d5ad2026f225c93fd5585521e
* Merge "Use oslo_utils instead of deprecated oslo.utils"Jenkins2015-05-291-1/+1
|\
| * Use oslo_utils instead of deprecated oslo.utilsDina Belova2015-05-111-1/+1
| | | | | | | | Change-Id: Ie4c55f33d18294924701032031cf78539ee3b23b
* | Fix alarm-evaluator can't start when ssl was enabledLan Qi song2015-05-111-3/+3
|/ | | | | Change-Id: I8cd3f3c7e745107ad4e0cbed7ebdd247a34fbbbd Closes-Bug: #1453061
* Add timeout for keystoneclient sessionZhiQiang Fan2015-04-141-1/+3
| | | | | | | | | | | 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
* add region_name to auth plugin parametersZhiQiang Fan2015-04-141-0/+2
| | | | | | | | | 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
* Merge "ceilometerclient insecure argument no longer works"Jenkins2015-04-141-1/+3
|\
| * ceilometerclient insecure argument no longer worksMatthew Edmonds2015-04-071-1/+3
| | | | | | | | | | | | | | | | The os_insecure argument needs to be parsed and passed to the auth plugin for non-shell usage. Change-Id: Id45a680396a9b646b273d971a0f6c274e95019a6 Closes-Bug: #1438750
* | Merge "fix client docstring"Jenkins2015-04-091-0/+2
|\ \ | |/ |/|
| * fix client docstringZhiQiang Fan2015-04-061-0/+2
| | | | | | | | | | | | | | | | | | * fix broken docstring in ceilometerclient.client * remove non-existent `release` index * remove duplicated reference generation in ext/gen_ref.py Change-Id: Iad8c527e61628a406d5dd84adac875190b994fac Closes-Bug: #1440629
* | ceilometerclient fails with keystone v3 authMatthew Edmonds2015-03-311-0/+2
|/ | | | | | | | | | When attempting to use keystone v3 Password authentication, ceilometerclient is failing to pass the project information to keystoneclient, which results in an unscoped token that lacks the service catalog. Change-Id: I980ba19f4b6b85ffc54d044184970c4902e62af4 Closes-Bug: #1438753
* add in missing optionsgordon chung2015-03-191-1/+2
| | | | | | we missed a few options. this cleans up list. Change-Id: I9ba984f804aecee5b63e5a86e699733040dee245
* Fixes bug with Client function not setting up SSL paramsSrinivas Sakhamuri2015-03-121-30/+33
| | | | | | | | | | 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
* Merge "Allow all pep8 checks"Jenkins2015-01-201-2/+1
|\
| * Allow all pep8 checksDina Belova2015-01-121-2/+1
| | | | | | | | | | | | | | Fix H405 pep8 issue. H904 issue check was ignored, although there were no places in client code violating this rule. Change-Id: If3b84eb8abb03511d7f77cabfa2955f472a60f23
* | Merge "Fix improper parameter setup for cacert and client certs"Jenkins2015-01-161-1/+25
|\ \
| * | Fix improper parameter setup for cacert and client certsSrinivas Sakhamuri2015-01-151-1/+25
| |/ | | | | | | | | | | | | | | | | | | | | | | | | 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 "Remove redundant parentheses (except openstack.common)"Jenkins2015-01-131-1/+1
|\ \ | |/ |/|
| * Remove redundant parentheses (except openstack.common)Dina Belova2014-12-101-1/+1
| | | | | | | | Change-Id: I594234648a37d283559a0365969a7fe3609079f8
* | Support ceilometer-url and os-endpointZhiQiang Fan2015-01-081-20/+23
| | | | | | | | | | | | | | | | | | Ceilometerclient has messed up with (ceilometer-url and os-endpoint) and (os-auth-token and token). This patch marks ceilometer-url and auth-token as deprecated, and unifies the two pairs arguments. Change-Id: Id55e5b603d8075018cf45b9833d397173439b3ce Closes-Bug: #1396811
* | Enable --os-insecure CLI optionZhiQiang Fan2014-12-061-1/+3
|/ | | | | | | | | | | | | --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
* Support os-endpoint-typeZhiQiang Fan2014-11-271-2/+3
| | | | | | | | | | | | Ceilometerclient will always get publicURL even though we specify endpoint type via arguments, it is because we call kesytone session with wrong keyword argument, which should be interface rather than endpoint_type. see: https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/auth/identity/base.py Change-Id: Ic2b03bd900d913c7efd22c9b91dfaa79e0cdd4bc Closes-Bug: #1396493
* Fix wrong initialization of AuthPlugin for keystone v3ZhiQiang Fan2014-11-111-0/+8
| | | | | | | | Ceilometerclient doesn't pass domain related arguments to AuthPlugin, which will cause keystoneclient v3 cannot be used properly. Change-Id: I2554a36f214c0eae61b7ba6b8580c41b8f8c6370 Closes-Bug: #1391122
* Merge "keystone discovery fallback support"Jenkins2014-09-201-42/+41
|\
| * keystone discovery fallback supportgordon chung2014-09-101-42/+41
| | | | | | | | | | | | | | | | | | version discovery and sessions aren't supported by all external auth systems. this patch adds a fallback when discovery does not work. Closes-Bug: #1333961 Change-Id: Ib253c736e37f8dbc50881f9fc691bfdb4ab87d71
View Lorry Controller Status