summaryrefslogtreecommitdiff
path: root/ceilometerclient
Commit message (Collapse)AuthorAgeFilesLines
* fix gnocchi alarm create to use right field nameliberty-eol1.5.2stable/libertyPradeep Kilambi2016-01-261-1/+1
| | | | | | | Closes-Bug: #1509430 Change-Id: I4f3b60776230dec5ac0ee31369fbe95cb0a8cd71 (cherry picked from commit b97ef58628a824851ef07e4c0429723b08424214)
* Fix to enable meter-links on resource-list CLIRohit Jaiswal2015-10-233-1/+53
| | | | | | | | | | | This adds an optional param to resource-list CLI to enable meter-links. By default, meter links are not generated, this patch adds an optional param to generate links. Change-Id: I308b94c8a57833cf30b4815ecec2ff9a50e91af6 Closes-Bug: #1508709 (cherry picked from commit e4a62a20b1d072969e842c38b2885a46a20fc54c)
* 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
* | | add region_name to auth plugin parametersZhiQiang Fan2015-04-142-0/+3
| |/ |/| | | | | | | | | | | | | | | 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-142-1/+12
|\ \
| * | ceilometerclient insecure argument no longer worksMatthew Edmonds2015-04-072-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | 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