summaryrefslogtreecommitdiff
path: root/ceilometerclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* Fix to enable meter-links on resource-list CLIRohit Jaiswal2015-10-232-1/+50
| | | | | | | | | | | 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-091-1/+37
| | | | | | | | limit is now a mandatory condition when querying. this patch allows users to control this value. Change-Id: Ied5f80dc8cfe8b5f85e60fb8ca1d4a8f26663456 Closes-Bug: #1493511
* do not generate meter links on resource-listgordon chung2015-09-081-4/+4
| | | | | | | | 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
* 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-251-0/+9
| | | | | | | | | | | | 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-102-1/+33
|\
| * Add ceilometerclient support for api-no-pipelineliu-sheng2015-06-262-1/+33
| | | | | | | | | | | | | | | | | | | | | | * 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-311-1/+2
| | | | | | | | | | | | | | | | | | | | 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-223-27/+116
| | | | | | | | | | | | | | | | 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
* | Add alarm severity in alarm-history outputRohit Jaiswal2015-06-241-0/+1
|/ | | | | | | | | | | 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
* Merge "Explicit error for wrong resource metadata format"1.3.0Jenkins2015-06-181-0/+10
|\
| * Explicit error for wrong resource metadata formatliu-sheng2015-06-171-0/+10
| | | | | | | | | | | | | | | | | | 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-161-0/+14
|/ | | | | | | | | | 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-082-0/+72
| | | | | | | | | | 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-031-1/+1
|\
| * Catch missing 404 exceptions in clientJuan Antonio Osorio Robles2015-05-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "[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 "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-041-2/+4
|\ \ \ | |_|/ |/| |
| * | Allow changing project-id and user-id by alarm-threshold-updateJiří Suchomel2015-04-291-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "print user friendly error message for alarm update time constraints"Jenkins2015-04-161-0/+17
|\ \ \ | |/ / |/| |
| * | print user friendly error message for alarm update time constraintsZhiQiang Fan2015-04-021-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-162-0/+101
|\ \ \
| * | | Add CLI for Capabilities REST APIZhiQiang Fan2015-04-042-0/+101
| |/ / | | | | | | | | | | | | | | | Implements blueprint capability-cli Change-Id: I23fe9c0bc9c27e04e3e0133fc84da7cf1f87321f
* | | Add timeout for keystoneclient sessionZhiQiang Fan2015-04-142-2/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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/+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
* | ceilometerclient insecure argument no longer worksMatthew Edmonds2015-04-071-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
* Set auth_plugin in __init__Chris Dent2015-03-201-0/+5
| | | | | | | | This avoids auth_plugin being undefined when calling self.get_base_parser without self.main having been called. Closes-Bug: #1434264 Change-Id: Ic3a898ccecb865ff4e88d82538bcb04001d623db
* Merge "support specify user-id when create sample and alarm"Jenkins2015-03-201-14/+22
|\
| * support specify user-id when create sample and alarmZhiQiang Fan2015-03-201-14/+22
| | | | | | | | | | | | | | | | | | | | The change of I9ba984f804aecee5b63e5a86e699733040dee245 enables us to specify os-user-id in CLI, but it shadows the user-id when creating sample and alarm, this patch fixes it as what we've done for project-id. Change-Id: Ic3ce5dc8e1aea49b3e41fe495be4921fbf85ea5a ref-change: I0ce2416dccd61eb50584799e6df0b8c45d44cdda
* | Add a post_test_hook for gate-run functional testsChris Dent2015-03-181-0/+50
| | | | | | | | | | | | | | | | This does the necessary to gather up the results and publish them properly for the gate job. Based on similar changes done for python-novaclient. Change-Id: Ib1f63aed9425fc8418fb9ddfe32f2d53f4b7c2b2
* | Add cli functional tests from tempestChris Dent2015-03-163-0/+72
|/ | | | | | | | | | | | | | | | | test_readonly_ceilometer is migrated from ./tempest/cli/simple_read_only/telemetry A new 'functional' target is available in tox. In order for these tests to pass OS_PASSWORD, OS_AUTH_URL, OS_USERNAME and OS_TENANT_NAME need to be set in the environment before running the tests, e.g: source src/devstack/openrc demo demo && tox -efunctional Subsequent patches will remove tests from tempest and add thes tests to project-config for gate checks. Change-Id: I3ce5ddbe471d70e8775ec5a1cc908a95a335b5e7
* Move unit tests into their own subdirectoryChris Dent2015-03-1327-23/+23
| | | | | | | | | | | | | | This is in preparation of moving functional tests currently in tempest in-tree. No surprises in this change: * move files * update the local imports Same number of tests run, all still pass. Change-Id: Ibfeac44421c5f36740b1377868f5ec469b2893f5
* Merge "Fixes bug with Client function not setting up SSL params"Jenkins2015-03-131-4/+11
|\
| * Fixes bug with Client function not setting up SSL paramsSrinivas Sakhamuri2015-03-121-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
* | Merge "Support unicode for alarm"Jenkins2015-03-111-5/+12
|\ \ | |/ |/|
| * Support unicode for alarmZhiQiang Fan2015-02-061-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We're using ceilometer.common.utils.print_dict() for displaying alarm, but assume only contain ascii characters, which will cause UnicodeEncodeError when alarm's name or description has unicode characters. The oslo-incubator.cliutils.print_dict() is more friendly to unicode, but it has no sort option, and has different line wrap strategy. Since the upstream improvement can be low progress, this patch adds unicode support based on current code. Change-Id: Ia58d5813c6f80cb8b44bf7636e1efc0cdd6e6d5b Closes-Bug: #1412300
* | Enable specified project_id in CLI commandsZhiQiang Fan2015-02-251-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After we port to oslo-incubator.apiclient, the project_id can no longer be specified for alarm-{,threhsold,combination}-{create,update} and sample-create, this is because client.AuthPlugin registers a CLI argument named --os-project-id which will shadow the project-id argument. Since os-project-id is used for Keystone V3 API, we should not remove it from AuthPlugin, so this patch moves the dest of project_id to {alarm,sample}_project_id, and adds a decorator to restore shadowed project_id field when we call v2 client. Change-Id: I0ce2416dccd61eb50584799e6df0b8c45d44cdda Closes-Bug: #1393489
* | Corrected the errors in sample-show and sample-createLianhao Lu2015-02-121-3/+7
|/ | | | | | | | | Patchset I67152c636526dad3ec27e06058ff73ad969ae2b9 broke the sample-show and sample-create when introducing the new sample API support. This patch fixes those. Change-Id: I3e36df61cf672b44c376c4a18dc808d565315bf6 Closes-Bug: #1421040
* Merge "add --no-traits for event-list"Jenkins2015-02-051-6/+55
|\
| * add --no-traits for event-listZhiQiang Fan2015-02-051-6/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we print traits list for each event when run event-list command, but that is a lot of information. After system run for a few days, the event-list command prints so much that it is not convenient to know the whole event-list even with less command. This patch adds a simple argument called --no-traits, it is False by default, event-list will not print traits if this argument is specified. Since this argument is boolean type, no need to assign a value, if it is specified, then it is actived. Change-Id: Ib69209fef5967a18f094f2d4665fa189e4ca3e6a
* | Add Sample API supportZhiQiang Fan2015-02-052-61/+258
|/ | | | | | | | | | Sample API has been implemented in Ceilometer for a long time, but CLI is lack of such support, this patch implements Sample CLI. Implements blueprint cli-samples-api Change-Id: I67152c636526dad3ec27e06058ff73ad969ae2b9 DocImpact
* Add severity field to alarm CLIPradeep Kilambi2015-01-262-30/+39
| | | | | Change-Id: I699bea4859d4f0a5832ced5ba74ba7fb26c122ef Partially-Implements: blueprint ceilometer-alarm-level
* Merge "Fix H105 pep8 error"Jenkins2015-01-203-6/+0
|\
View Lorry Controller Status