summaryrefslogtreecommitdiff
path: root/ceilometerclient/v2
Commit message (Collapse)AuthorAgeFilesLines
* [trivial] add a blank characterzhangguoqing2016-08-041-1/+1
| | | | | | | In ceilometer help event-list, the line "integer, floator datetime." should be "integer, float or datetime." Change-Id: Ib8976b5e580e2a95a4e04d7d95c815011127749c
* Switch to keystoneauthClenimar Filemon2016-06-011-6/+3
| | | | | | | keystoneclient.auth is deprecated in favour of keystoneauth. This patch updates Ceilometerclient accordingly. Change-Id: Id4b14cbccfabfeaa5322f7ae5702261d0ed9cca2
* Update to hacking 0.11.0ZhiQiang Fan2016-05-133-10/+0
| | | | Change-Id: I858a418fa5202ec4fc0fc17f462b983f7864bcac
* Merge "remove default value of repeat-actions"Jenkins2016-03-141-1/+0
|\
| * remove default value of repeat-actionsZhiQiang Fan2016-03-101-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I94468ff649dd4367848c74e94a3feb08494bb223 fixes problem that ceilometeclient incorrectly reset repeat-actions when update any attribute. I53f3af447a8f814388985f6e4ab57a8ffec18a2a reverts most of the above change, which leading to bug 1539092. Because repeat-actions has default value False, no matter we set it or not, it will replace this bit when we get alarm from API. The unit test doesn't cover such case, so this change still succeeds in CI test. We don't need to revert it again, because the redundant code for create and update is not needed, repeat-actions has default value (False) in API side, so the proper way is simply removing the default value in CLI side. Change-Id: I39c28294db3e55141bb8a2024a6ddfdf8acf5e0d Closes-Bug: #1539092
* | Decouple ceilometerclient without aodh services runningliusheng2016-03-081-0/+4
|/ | | | | | | | | Currently, the ceilometerclient will be broken if aodh services not running, it will be worse if users don't want alarming service and they must start the aodh services. Change-Id: Idf7468f28cee001262b67b6e110357ffc971535a Closes-Bug: #1550095
* Merge "make aggregation-method argument as a mandatory field"Jenkins2016-02-241-0/+1
|\
| * make aggregation-method argument as a mandatory fieldxialinjuan2016-01-221-0/+1
| | | | | | | | | | | | | | When create the gnocchi alarm, the aggregation-method is required. Closes-Bug: #1531602 Change-Id: I65b5038053d69dc727639fbfeaa5a23108845083
* | Merge "Enhances client to support unique meter retrieval"Jenkins2016-02-092-3/+14
|\ \
| * | Enhances client to support unique meter retrievalRohit Jaiswal2016-02-052-3/+14
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | meter list API is enhanced to support a unique param If specified as True, it returns a unique list of meters. This patch enhances the ceilometer-client to support the unique argument in the shell and the client bindings. DocImpact Closes-Bug: #1506959 Change-Id: I011ca725c25e1cdbe9891c3719f13f5cd34142ea
* | Fixing a word spellingsanjana2016-02-051-2/+2
|/ | | | | | TRIVIAL-FIX: correct the word used in the comment Change-Id: I060a63a1363b0d73dc9a089bcdf1f50d79cc943c
* Merge "Fix to disable meter-links from CLI"Jenkins2016-01-191-3/+0
|\
| * Fix to disable meter-links from CLIakanksha2016-01-081-3/+0
| | | | | | | | | | | | | | | | | | Removes the option of meter-links from CLI.Since meter-links are never shown. Change-Id: I3949bb6d77786926f22506d97d988f73fa614958 Closes-Bug: #1528318
* | fix project_id and user_id fields not set when create gnocchi alarmxialinjuan2016-01-171-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | When using command alarm-gnocchi-* the project_id and user_id are not set. The patch fix it and add related tests: 1.ceilometer gnocchi alarm create/update test cases 2.check gnocchi alarm create/update command args test cases 3.check update alarm without alarm-id 4.check shadowed args are restored Closes-Bug: #1506509 Change-Id: If6a8cfd9bc52b3eeff99dfcfdee2f4da7220674b
* | improve help docs for cli commandsIgor Degtiarov2016-01-141-7/+13
| | | | | | | | | | | | | | Adds default field into help docs for all lists cli commands. Closes-bug: #1527271 Change-Id: I3081f273bf7b5fd512b0bbd105b3dfd1d69772b9
* | Merge "Don't copy the auth_plugin for aodh"2.2.0Jenkins2016-01-121-38/+40
|\ \
| * | Don't copy the auth_plugin for aodhMehdi Abaakouk2016-01-071-38/+40
| |/ | | | | | | | | | | | | | | | | | | | | | | | | 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 "Restructuring a comment in Python-Ceilometerclient"Jenkins2016-01-111-5/+4
|\ \
| * | Restructuring a comment in Python-CeilometerclientSanjana Pai2015-12-141-5/+4
| |/ | | | | | | | | | | | | | | The comment describes the handling of HTTPNotFound exception about alarm, but it is ambiguous, this change make it more clear. Closes-Bug:#1523866 Change-Id: Id7be71dcda6693875222435e10cba929e7c81e56
* | Improve ceilometer alarm enabled input parameter validationapril2016-01-051-1/+2
|/ | | | | | | | | | | strict the input parameter of enabled in the command : ceilometer alarm-threshold-update <alarm-id> --enabled {True|False} This commit will improve the enabled value validation, if the input value is invalid such as "abde" "enabled", error info will be returned. Closes-Bug: #1531042 Change-Id: Ic553bc9ffa05a84575fc5626416c79fc84687bbe
* Merge "Ensure keystoneauth1 exception are raised"Jenkins2015-11-271-1/+5
|\
| * Ensure keystoneauth1 exception are raisedMehdi Abaakouk2015-11-201-1/+5
| | | | | | | | Change-Id: I808a2a32860a6fe6b9b1797dd59a0ebed08d7b0a
* | Merge "Catch exception raised by keystoneauth1"Jenkins2015-11-231-3/+8
|\ \ | |/
| * Catch exception raised by keystoneauth1Mehdi Abaakouk2015-11-201-3/+8
| | | | | | | | | | | | | | | | | | We must support session object that comes from keystoneauth1 and keystoneclient. This change adds code path for keystoneauth1 when it misses. Change-Id: I1164a0a0745605b6bc4fd1fc8fb2813c7e40fb9b
* | fix gnocchi alarm create to use right field namePradeep Kilambi2015-11-051-1/+1
|/ | | | | | Closes-Bug: #1509430 Change-Id: I4f3b60776230dec5ac0ee31369fbe95cb0a8cd71
* Update help message for alarm-gnocchi-resources-threshold-create2.0.0Rohit Jaiswal2015-10-281-1/+7
| | | | | | | | | | Help message for '--aggregation-method' in alarm-gnocchi-resources-threshold-create command is not precise. The problem is that 'avg' is not a valid aggregation method in gnocchi. This fix updates the help message. Change-Id: I513b6fb9db9f96bfdd10391b6acd5c95e278f601 Closes-Bug: #1506549
* Fix to enable meter-links on resource-list CLIRohit Jaiswal2015-10-231-0/+3
| | | | | | | | | | 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
* Merge "print sample id for query-samples"Jenkins2015-10-201-3/+3
|\
| * print sample id for query-samplesZhiQiang Fan2015-10-151-3/+3
| | | | | | | | | | | | | | | | | | 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-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | 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-57/+50
|/ | | | | | | | | | | | 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-094-8/+21
| | | | | | | | 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-091-2/+3
|\
| * do not generate meter links on resource-listgordon chung2015-09-081-2/+3
| | | | | | | | | | | | | | | | 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
* Don't try to get aodh endpoint if auth_url didn't providedliu-sheng2015-08-251-6/+17
| | | | | | | | | | | | 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-3/+11
|\
| * Add ceilometerclient support for api-no-pipelineliu-sheng2015-06-262-3/+11
| | | | | | | | | | | | | | | | | | | | | | * 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
* | Add support for client redirecting to aodh endpoint if availableliu-sheng2015-07-222-6/+39
| | | | | | | | | | | | | | | | 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 "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-031-1/+1
|\ \
| * | Add alarm severity in alarm-history outputRohit Jaiswal2015-06-241-1/+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
* | Change default values from [] to NoneRoman Vasilets2015-06-251-1/+4
|/ | | | | | | | | 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-181-1/+6
|\
| * Explicit error for wrong resource metadata formatliu-sheng2015-06-171-1/+6
| | | | | | | | | | | | | | | | | | 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-1/+1
|/ | | | | | | | | | Strips leading and trailing whitespaces from CLI query field and value parameter before building url. Closes-Bug: 1464694 Change-Id: Ibe99747e22afe05eee187d59aaf0603f58e91373