summaryrefslogtreecommitdiff
path: root/ceilometerclient
Commit message (Collapse)AuthorAgeFilesLines
...
* drop v1 clientgordon chung2015-10-2111-1062/+0
| | | | | | | | | | we dropped v1 back in Juno. as icehouse is EOL and the client follows stable/branches, v1 is completely inaccessible and can be removed Change-Id: If4c6c07e65a6c47f47c59de6c99e8c40ef4e63cb Co-Authored-By: ZhiQiang Fan <aji.zqfan@gmail.com> Closes-Bug: #1389331
* Merge "print sample id for query-samples"Jenkins2015-10-202-13/+13
|\
| * print sample id for query-samplesZhiQiang Fan2015-10-152-13/+13
| | | | | | | | | | | | | | | | | | Provide sample id can make it possible for end user to query particular sample by its id, this is needed because we don't show everything in the list. Change-Id: I06bfc2de1a4e9a24892b2e36b6d2137e1b38cb27 Closes-Bug: #1506745
* | fix typos in docstringZhiQiang Fan2015-10-193-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | 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-075-87/+208
|/ | | | | | | | | | | | 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
* Add statistic in rule informationMehdi Abaakouk2015-09-231-1/+2
| | | | | | | | This change displays the 'statistic' of an threshold rule. When the detail of rule is asked. Change-Id: I59d5a9603b7666da87d734a95c0ada758ae24ad3 Closes-bug: #1498901
* Change the word for clarityAtsushi SAKAI2015-09-151-4/+4
| | | | | | | | | | | | settable => configurable and moves the word "only" This is a suggestion from CLI reference review. https://review.openstack.org/#/c/221996/1 The CLI reference generation tool generates from command help message. Change-Id: I3a746351e922f33aba10e28adb54be03b784ccf7
* add limit support1.5.0gordon chung2015-09-095-9/+58
| | | | | | | | limit is now a mandatory condition when querying. this patch allows users to control this value. Change-Id: Ied5f80dc8cfe8b5f85e60fb8ca1d4a8f26663456 Closes-Bug: #1493511
* Merge "do not generate meter links on resource-list"Jenkins2015-09-092-6/+7
|\
| * do not generate meter links on resource-listgordon chung2015-09-082-6/+7
| | | | | | | | | | | | | | | | meter links are not accessible via resource_list and add significant overhead to query. we shouldn't generate these anymore. Change-Id: Ib20f410a70171aa09e4ebbbb748a5ce495fc6a43 Closes-Bug: #1493400
* | refactor: move 'repeat_actions' to common argsRyota MIBU2015-09-081-37/+5
| | | | | | | | Change-Id: I53f3af447a8f814388985f6e4ab57a8ffec18a2a
* | Support alarm-event-{create,update}Ryota MIBU2015-09-071-0/+63
|/ | | | Change-Id: Ic29d7d5fcf95ff29112bd42fed747c54181e3640
* Use new location of subunit2htmlRyota MIBU2015-09-041-1/+1
| | | | | | | | | The previous location is going away, this location is preferred. This patch is copied from https://review.openstack.org/#/c/218165/ . Change-Id: Idd9cc58a845e82bc679d49e5bfb22644a9c67364 Closes-Bug: #1489404
* Don't try to get aodh endpoint if auth_url didn't providedliu-sheng2015-08-252-6/+26
| | | | | | | | | | | | This change includes: * Allow users of ceilometerclient specifying aodh_endpoint explicitly * If aodh_endpoint didn't provided and users just provide ceilometer endpoint and token, no auth_url, we assume aodh is unavailable and avoid trying to get aodh endpoint. Change-Id: Idd0ce60058d23507a99b89926e461d19db267ca2 Closes-Bug: #1488290
* Merge "Add ceilometerclient support for api-no-pipeline"1.4.0Jenkins2015-08-104-4/+44
|\
| * Add ceilometerclient support for api-no-pipelineliu-sheng2015-06-264-4/+44
| | | | | | | | | | | | | | | | | | | | | | * Add --direct flag for ceilometer sample-create and sample-create-list commands to support this feature * Add parameter for ceilometerclient.v2.samples.create() and ceilometerclient.v2 samples.create_list() to support this feature Partially implements: blueprint api-no-pipeline Change-Id: Ia3d787bbeaf360ddc198cb4ec183d83a5e76203a
* | tenant_id not required with keystone v3Rohit Jaiswal2015-07-313-9/+10
| | | | | | | | | | | | | | | | | | | | 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-226-60/+259
| | | | | | | | | | | | | | | | 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
* | Fix unit tests failing caused by new Mock releaseIlya Tyaptin2015-07-101-6/+11
| | | | | | | | | | | | | | | | Since the mock 1.1.1 calling of function 'assert_called' and 'assert_called_once' raises the AttributeError. In current patchset calling of these functions is fixed Change-Id: I5c21bb3f6cfd8cc095a277ead80794e97bb63c72
* | Merge "Typo: Show an sample => Show a sample"Jenkins2015-07-091-1/+1
|\ \
| * | Typo: Show an sample => Show a sampleaviau2015-06-251-1/+1
| |/ | | | | | | Change-Id: I17ca56142870313681c8ae9410db4173c2946f65
* | Merge "Add alarm severity in alarm-history output"Jenkins2015-07-032-1/+2
|\ \
| * | Add alarm severity in alarm-history outputRohit Jaiswal2015-06-242-1/+2
| |/ | | | | | | | | | | | | | | | | | | | | Info about alarm's severity is missing from the detail column for creation and deletion type of alarm-history call. This fix adds that. Change-Id: I669126596ef7391530e7822c2807ffca7c6a1de1 Closes-Bug: 1468295
* | Change default values from [] to NoneRoman Vasilets2015-06-252-2/+6
|/ | | | | | | | | Its well known fact that python default values initialized only once at the stage if creation function. So if we call function with parameter foo=[] as default value and change foo.append("bar") at the next call of the function default value would be ["bar"]. It will save. Change-Id: Iba9e55bc2d4dfd811ce7d068b6a9cceae7c000d6
* Merge "Explicit error for wrong resource metadata format"1.3.0Jenkins2015-06-182-1/+16
|\
| * Explicit error for wrong resource metadata formatliu-sheng2015-06-172-1/+16
| | | | | | | | | | | | | | | | | | When creating sample with wrong resource metadata format, Ceilometer CLI will return a ValueError about json decoding, this change provide a more explicit error info. Change-Id: I18e757ddb319fbf0ffad13c07528355785ea5e9a Closes-Bug: 1465193
* | Removes whitespace from CLI queriesRohit Jaiswal2015-06-162-1/+15
|/ | | | | | | | | | Strips leading and trailing whitespaces from CLI query field and value parameter before building url. Closes-Bug: 1464694 Change-Id: Ibe99747e22afe05eee187d59aaf0603f58e91373
* Add capability for creating array of samplesAleksey Ripinen2015-06-084-0/+105
| | | | | | | | | | Added method for creating array of samples to samples manager. Added shell command for creating array of samples. Added tests for this changes. It is made for Rally tests that use big data arrays. So now only one function may be called instead of numerous calls simple create function. Change-Id: I01ba4f0c0db40c95ef72dadaa34b3fafc034e417
* Merge "Catch missing 404 exceptions in client"Jenkins2015-06-033-6/+16
|\
| * Catch missing 404 exceptions in clientJuan Antonio Osorio Robles2015-05-183-6/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Whenever an entity wasn't found, the ceilometerclient would issue a NotFound exception. Which is fine, except that it's not the exception that was being expected in the client at all. This exception was defined in the apiclient in the openstack/common directory. So, instead this exception is now being caught in the manager, and now the appropriate exception is being raised. There were some functions where, if the entity was not to be found, a 404 exception would be raised, which was never caught by the client since it was expecting the function to return None, instead of raising. This CR fixes that. Closes-Bug: #1451833 Change-Id: I47b34af2df9c85f86ba1f7f4d7b2951f8d27f96c
* | Merge "Drop use of 'oslo' namespace package"Jenkins2015-06-037-10/+10
|\ \
| * | Drop use of 'oslo' namespace packageDoug Hellmann2015-06-037-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Oslo libraries have moved all of their code out of the 'oslo' namespace package into per-library packages. The namespace package was retained during kilo for backwards compatibility, but will be removed by the liberty-2 milestone. This change removes the use of the namespace package, replacing it with the new package names. The patches in the libraries will be put on hold until application patches have landed, or L2, whichever comes first. At that point, new versions of the libraries without namespace packages will be released as a major version update. Please merge this patch, or an equivalent, before L2 to avoid problems with those library releases. Blueprint: remove-namespace-packages https://blueprints.launchpad.net/oslo-incubator/+spec/remove-namespace-packages Change-Id: I1f539cf9f5d6f48e10cac381c13eeb5fa314abd4
* | | Merge "[unittest] Increase client and shell modules cover"Jenkins2015-06-033-13/+339
|\ \ \ | |/ / |/| / | |/
| * [unittest] Increase client and shell modules coverIlya Tyaptin2015-05-113-13/+339
| | | | | | | | | | | | | | Add tests to increase unittest coverage of ceilometerclient.client, ceilometerclient.shell, ceilometerclient.v2.shell. Change-Id: I83a64ee31d6068a5903ef85ec85d5780828a24d7
* | Merge "move capabilities ut code to unit dir"Jenkins2015-05-291-0/+0
|\ \
| * | move capabilities ut code to unit dirZhiQiang Fan2015-04-231-0/+0
| | | | | | | | | | | | | | | | | | | | | The unit test code for capabilities command are left alone, it should be under unit directory Change-Id: I6c9d4e75b44aeba8bd5e79b3c9cf531b928499b8
* | | Merge "Use oslo_utils instead of deprecated oslo.utils"Jenkins2015-05-294-5/+5
|\ \ \
| * | | Use oslo_utils instead of deprecated oslo.utilsDina Belova2015-05-114-5/+5
| | |/ | |/| | | | | | | 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
* | Merge "Added missing ceilometer CLI tests"Jenkins2015-05-071-5/+42
|\ \
| * | Added missing ceilometer CLI testsraiesmh082015-04-141-5/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds following missing Ceilometer CLI tests - 1. help 2. bash completion In addition to this following optional arguments are also verified. 1. --debug It also modifies existing code to check outputs of different tests. Earlier these were uploaded under - https://review.openstack.org/#/c/100190/ Change-Id: I2a00918e4ecbaac7d08f07ebaaa587dde7d8eee5
* | | remove useless event unit test codeZhiQiang Fan2015-05-061-2/+0
| | | | | | | | | | | | | | | | | | | | | In v2.test_shell.ShellEventListCommandTest, we specify traits twice, the first one should be removed. Change-Id: Iedd7c6161165816b9e76c44ef0f2ae6bf6016204
* | | Merge "Allow changing project-id and user-id by alarm-threshold-update"Jenkins2015-05-042-4/+7
|\ \ \
| * | | Allow changing project-id and user-id by alarm-threshold-updateJiří Suchomel2015-04-292-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ceilometer alarm-threshold-update should support updating project-id and user-id, but the update function was silently ignoring changes of these options. Change-Id: I33231bff44b90f3c58e5ef1646349b6dc63a3ce7 Closes-Bug: 1395932
* | | | show raw details of eventgordon chung2015-04-221-1/+1
| |_|/ |/| | | | | | | | | | | | | | add ability to show raw details of event. Change-Id: I7068eed3647b7068d3286d3d205ed4c9f4372122
* | | Merge "print user friendly error message for alarm update time constraints"Jenkins2015-04-162-2/+20
|\ \ \ | |/ / |/| |
| * | print user friendly error message for alarm update time constraintsZhiQiang Fan2015-04-022-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, if we update an alarm with wrong time constraint which doesn't get name defined, then the shell only prints a very simple string 'name'. This is because our code assume name field has always been specified, however it is not true, then KeyError exception will be raised but not handled well, finally user only gets an implicit message. This patch uses dict.get() for name field, and sends request (may be broken) to ceilometer api, then extracts error message from response. Change-Id: I086c4ec790acc22767ba7f5e43dbcf73f3af5dff Closes-Bug: #1439207
* | | Merge "Add CLI for Capabilities REST API"1.1.0Jenkins2015-04-165-0/+145
|\ \ \
| * | | Add CLI for Capabilities REST APIZhiQiang Fan2015-04-045-0/+145
| |/ / | | | | | | | | | | | | | | | Implements blueprint capability-cli Change-Id: I23fe9c0bc9c27e04e3e0133fc84da7cf1f87321f
* | | Add timeout for keystoneclient sessionZhiQiang Fan2015-04-143-3/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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