summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Modify ceilometer client cmd line help info"1.0.9Jenkins2014-01-281-2/+2
|\
| * Modify ceilometer client cmd line help infoJia Dong2014-01-281-2/+2
| | | | | | | | | | | | | | | | | | | | The optional argument "--insufficient-data-action" is described as "... URL to invoke when state transitions to insufficient_data...", it was " URL to invoke when state transitions to unkown. " Closes-Bug: #1260343 Change-Id: I34db56bb22b693e752a971f81d7b8467bfc6c4c5
* | Merge "Remove unused import for print_function"Jenkins2014-01-281-2/+0
|\ \
| * | Remove unused import for print_functionZhiQiang Fan2014-01-281-2/+0
| |/ | | | | | | | | | | | | | | A minor fix to https://review.openstack.org/#/c/69234/ which forgets to remove the unused 'from __future__ import print_function' after remove all print function. Change-Id: Iedc925a66fa4dec209cf34ce28b45f768f7bc81d
* | Merge "Update client to display data type of traits"Jenkins2014-01-283-7/+37
|\ \
| * | Update client to display data type of traitsJohn Herndon2014-01-243-7/+37
| | | | | | | | | | | | | | | | | | fixes bug 1268032 Change-Id: I47dfd3208fb3f7e5e5e2b33bd9b00c688d8283cd
* | | Merge "Raise traceback on error when using CLI and -debug"Jenkins2014-01-282-3/+30
|\ \ \ | |_|/ |/| |
| * | Raise traceback on error when using CLI and -debugChmouel Boudjnah2014-01-192-3/+30
| | | | | | | | | | | | | | | | | | | | | When --debug is specified to the CLI don't just print the simple error message but actually reraise that error. Change-Id: Iac624dbb3490d29e32a9ea251c14766186df31db
* | | Merge "Remove ununsed httplib2 requirement"Jenkins2014-01-273-9/+0
|\ \ \
| * | | Remove ununsed httplib2 requirementChmouel Boudjnah2014-01-273-9/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is not needed anymore since the logging is handled by six.moves/httplib. Change-Id: I4de3cfd8d8d3c08b38e39fde12172bdb4de0cea0 Closes-Bug: 1272986
* | | | Merge "Remove print debugs statements."Jenkins2014-01-271-4/+0
|\ \ \ \ | | |/ / | |/| |
| * | | Remove print debugs statements.Chmouel Boudjnah2014-01-181-4/+0
| |/ / | | | | | | | | | Change-Id: I78a885ba98363ca309598294bc84604d05a55446
* | | Merge "return sample info when creating sample with CLI"Jenkins2014-01-272-1/+70
|\ \ \
| * | | return sample info when creating sample with CLIls11752014-01-232-1/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It will return nothing after creating a sample successfully with ceilometer CLI. The information of the sample created should be return, like other creating commands, such as "ceilometer alarm-create". Change-Id: I60229fefa91de83e4b715cf4f92d3650e0a0d162 Closes-bug: #1267713
* | | | Merge "Using common methods from oslo cliutils"Jenkins2014-01-272-27/+15
|\ \ \ \
| * | | | Using common methods from oslo cliutilsJia Dong2014-01-242-27/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some common methods in cliutils we can use in ceilometerclient: arg, env, unauthenticated, isunauthenticated. + Replaces utils.env to add alias env from cliutils. + Removes unused methods in utils: env To use methods from clituils. partially implements blueprint common-client-library-2 Change-Id: I295cc80635ed15c91e2832acb315e9deede09516
* | | | | Merge "Fix typos picked up by misspellings"Jenkins2014-01-272-2/+2
|\ \ \ \ \
| * | | | | Fix typos picked up by misspellingsZhiQiang Fan2014-01-232-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Joe Gordon has introduced a module named misspellings which can check typo automatically. $ pip install misspellings $ git ls-files | grep -v locale | misspellings -f - Change-Id: I9eca2e097b03468651814fa563e4380fd573205e ref: http://lists.openstack.org/pipermail/openstack-dev/2013-December/020975.html
* | | | | | Merge "Avoid discarding alarm-threshold-create --query option"Jenkins2014-01-243-3/+55
|\ \ \ \ \ \
| * | | | | | Avoid discarding alarm-threshold-create --query optionEoghan Glynn2014-01-233-3/+55
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1271913 The query is crucial to the alarm evaluator statistics query being well-formed, so must not be discarded. Change-Id: I5ff8dcb420b932fc58dfef3dadfcbc4a4dc5cf66
* | | | | | Merge "Enable hacking H233 rule"Jenkins2014-01-241-1/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Enable hacking H233 ruleZhiQiang Fan2014-01-231-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | H233: Check that all occurrences look like print functions, not print operator. Change-Id: Ida43c6d68c605f7b6aba32a3aa4155811b888ef2
* | | | | Merge "Add support for groupby in statistics for API v2"Jenkins2014-01-243-3/+62
|\ \ \ \ \
| * | | | | Add support for groupby in statistics for API v2Enol Fernandez2014-01-173-3/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds the groupby parameter to list method of Statistics and a new option "-g/--groupby" for the shell. Change-Id: I41cfa076bf73f8c01d4ee71cd70008ea231556c8 Closes-Bug: #1270094
* | | | | | Merge "Replace inheritance hierarchy with composition"Jenkins2014-01-243-20/+21
|\ \ \ \ \ \
| * | | | | | Replace inheritance hierarchy with compositionAndrey Kurilin2014-01-153-20/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the process of unification of the clients code we should use composition to allow easier replacement with common HTTPClient. bp common-client-library-2 Change-Id: I98d2bc89d5b1a36c30771923cd1cda7925e563d5
* | | | | | | Merge "abbreviating --meter-name to -m in alarm commands"Jenkins2014-01-241-4/+4
|\ \ \ \ \ \ \
| * | | | | | | abbreviating --meter-name to -m in alarm commandsls11752014-01-131-4/+4
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | related commands of alarm:alarm-threshold-update,alarm-update,alarm-create should support parameter --meter-name abbreviating to -m like other commands of ceilometer. Change-Id: I9fa9a7d14df3d93c3021f5004b444e72e1099c48 Closes-bug: #1268431
* | | | | | | Merge "Using common method 'bool_from_string' from oslo strutils"Jenkins2014-01-242-11/+8
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | |
| * | | | | | Using common method 'bool_from_string' from oslo strutilsllg82122014-01-222-11/+8
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using common method 'bool_from_string' from oslo strutils to replace utils.string_to_bool. partially implements blueprint common-client-library-2 Change-Id: I101eb89174ac229aa1025bc4aea8b1c0c5594705
* | | | | | Merge "replace assertTrue(isinstance) to assertIsInstance"Jenkins2014-01-231-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | replace assertTrue(isinstance) to assertIsInstancels11752014-01-181-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | some of tests use different method of assertTrue(isinstance(A, B)) or assertEqual(type(A), B). The correct way is to use assertIsInstance(A, B) provided by testtools Change-Id: Idd931d27b13a46800e27393a98da096d41442320 Closes-bug: #1268480
* | | | | Merge "Remove dependencies on pep8, pyflakes and flake8"Jenkins2014-01-191-4/+1
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Remove dependencies on pep8, pyflakes and flake8Dirk Mueller2014-01-161-4/+1
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | They should be determined by the hacking dependency implicitely. Change-Id: Iaa66d7baaa05350b6d6289976f807a5750b0ea8e
* | | | Merge "fix optional parameter of creating sample"Jenkins2014-01-171-1/+1
|\ \ \ \
| * | | | fix optional parameter of creating samplels11752014-01-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When creating sample without parameter "meter_name" in ceilometer CLI, The result is 'counter_name' ,the parameter "meter_name"(i.e.counter_name) isrequired,but it is not in the help messages of "ceilometer sample-create". Change-Id: I17ee19354b51cd5095677438359d036783d82b15 Closes-bug: #1267378
* | | | | Merge "Remove unused imports"Jenkins2014-01-161-3/+0
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Remove unused importsWu Wenxiang2014-01-111-3/+0
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | Remove unused imports in doc/source/conf.py Change-Id: I419f60dc7bb3a2e73961bab933867b171851eb18 Closes-Bug: #1268091
* | | | Merge "Supports bash_completion for ceilometerclient"Jenkins2014-01-152-0/+56
|\ \ \ \
| * | | | Supports bash_completion for ceilometerclientZhiQiang Fan2013-12-232-0/+56
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bash_completion feature can improve CLI user experience, projects like nova, keystone, and cinder already support it. NOTE: this patch just provides simple functionality, which means cache for IDs and names is not used (like nova). Change-Id: I1b04998a5e7c8818aed4b51f5fe5bf01efd0e14a Closes-Bug: #1260939
* | | | Merge "client looking at wrong cacert argument name"Jenkins2014-01-151-1/+1
|\ \ \ \
| * | | | client looking at wrong cacert argument nameGordon Chung2014-01-061-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix so client consistently uses 'os_cacert' arg value instead of either 'os_cacert' or 'cacert' Change-Id: If5648855aefc7af64178249c270a9ac66907e466 Closes-Bug: #1266472
* | | | Merge "Python 3: fix test_sample_list"Jenkins2014-01-151-3/+3
|\ \ \ \ | |_|_|/ |/| | |
| * | | Python 3: fix test_sample_listCyril Roelandt2014-01-101-3/+3
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change the counter_volume field of "fake" samples to get compatibility with both Python 2 and 3. In Python 2: >>> print(str(0.261666666666667)) 0.261666666667 In Python 3: >>> print(str(0.261666666666667)) 0.261666666666667 Change-Id: I4beed7aff7e89f891024975b9df9208c987dc87c
* | | Support the Event APIJohn Herndon2014-01-1018-35/+692
| |/ |/| | | | | | | | | | | | | | | | | | | | | This patch adds support for the currently available Event API, including events, event types, and traits. Add an optional data type to the query, ex: ceilometer list-events -q 'hostname=string::localhost" bp extend-client-operations Change-Id: Icea9bd67f8ee4ff2bf9da9ff6894218689580eb3
* | Merge "Use Resource() class from common Oslo code"Jenkins2014-01-101-51/+2
|\ \
| * | Use Resource() class from common Oslo codeekudryashova2014-01-081-51/+2
| |/ | | | | | | | | | | | | | | In the process of unification of the clients code we should reuse common functionality from Oslo. Resource() class from ceilometer duplicates Oslo funclionality, so we replace it with inheritance Change-Id: I4e4b83252cdc87e8484d15e351cd5dab08f3ed8a
* | Fix the ceilometerlient log curl request incorrectlyls11752013-12-201-1/+1
|/ | | | | | | | | | | | Have installed ceilometer on Ubuntu 12.04 (LTS) and added the keystone endpoint with the url (publicurl,internalurl and adminurl)like:http://192.168.83.7:8777/refer to the "OpenStack Installation Guide for Ubuntu 12.04 (LTS)". When using "ceilometer --debug meter-list",the ceilometerclient log request url with a double slash after 8777. Change-Id: I467704b1fe1d38fe0f0e928bd6f6efe3d8925845 Closes-bug: #1262498
* Python 3: use six.moves.zip() rather than itertools.izip()Cyril Roelandt2013-12-191-2/+1
| | | | | | This ensures compatibility with both Python 2 and Python 3. Change-Id: Id9838b7517f939e5ee3e3a8c37ae5ec2e9b1d18b
* Merge "Display message on HTTP 400"Jenkins2013-12-173-4/+66
|\