)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"d35be6534f1d3914717031f3e6fde6fcbc6a86ab","unresolved":false,"context_lines":[{"line_number":7,"context_line":"return sample info when creating sample with CLI"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It will return nothing after creating a sample successfully with"},{"line_number":10,"context_line":"ceilometer CLI.ceilometer client should return the information of the"},{"line_number":11,"context_line":"sample created. like other \"create\" commands,such as \"ceilometer"},{"line_number":12,"context_line":"alarm-create\"commands,such as \"ceilometer alarm-create\""},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAUH%2F%2FQFA%3D","line":10,"updated":"2014-01-15 21:35:04.000000000","message":"... CLI. Ceilometer ...","commit_id":"86d31aacb283f12cbfdec7332624dec76664c0a3"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"d35be6534f1d3914717031f3e6fde6fcbc6a86ab","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It will return nothing after creating a sample successfully with"},{"line_number":10,"context_line":"ceilometer CLI.ceilometer client should return the information of the"},{"line_number":11,"context_line":"sample created. like other \"create\" commands,such as \"ceilometer"},{"line_number":12,"context_line":"alarm-create\"commands,such as \"ceilometer alarm-create\""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I60229fefa91de83e4b715cf4f92d3650e0a0d162"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAUH%2F%2FQEs%3D","line":11,"updated":"2014-01-15 21:35:04.000000000","message":"sample created, like other \"create\" commands, such as \"ceilometer","commit_id":"86d31aacb283f12cbfdec7332624dec76664c0a3"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"d35be6534f1d3914717031f3e6fde6fcbc6a86ab","unresolved":false,"context_lines":[{"line_number":9,"context_line":"It will return nothing after creating a sample successfully with"},{"line_number":10,"context_line":"ceilometer CLI.ceilometer client should return the information of the"},{"line_number":11,"context_line":"sample created. like other \"create\" commands,such as \"ceilometer"},{"line_number":12,"context_line":"alarm-create\"commands,such as \"ceilometer alarm-create\""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I60229fefa91de83e4b715cf4f92d3650e0a0d162"},{"line_number":15,"context_line":"Closes-bug: #1267713"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAUH%2F%2FQEQ%3D","line":12,"updated":"2014-01-15 21:35:04.000000000","message":"duplication \u0027such as\u0027?","commit_id":"86d31aacb283f12cbfdec7332624dec76664c0a3"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"80feb0aabdf23f99709c9a22fc0c17aa71841369","unresolved":false,"context_lines":[{"line_number":9,"context_line":"It will return nothing after creating a sample successfully with"},{"line_number":10,"context_line":"ceilometer CLI.ceilometer client should return the information of the"},{"line_number":11,"context_line":"sample created. like other \"create\" commands,such as \"ceilometer"},{"line_number":12,"context_line":"alarm-create\"commands,such as \"ceilometer alarm-create\""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I60229fefa91de83e4b715cf4f92d3650e0a0d162"},{"line_number":15,"context_line":"Closes-bug: #1267713"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAUH%2F%2FPWo%3D","line":12,"in_reply_to":"AAAAUH%2F%2FQEQ%3D","updated":"2014-01-16 01:27:22.000000000","message":"thanks for your comments?it\u0027s my mistake, you are very careful !","commit_id":"86d31aacb283f12cbfdec7332624dec76664c0a3"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"06f9d294ca8d5a3b45bc0d73fc32bd51f5b75d2e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"return sample info when creating sample with CLI"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It will return nothing after creating a sample successfully with"},{"line_number":10,"context_line":"ceilometer CLI. The information of the sample created should be return."},{"line_number":11,"context_line":"like other creating commands,such as \"ceilometer alarm-create\""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I60229fefa91de83e4b715cf4f92d3650e0a0d162"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"AAAAUH%2F%2BsxE%3D","line":10,"updated":"2014-01-23 01:58:33.000000000","message":"s/sample created/created sample/\n\nthe end \u0027.\u0027 might be \u0027,\u0027?","commit_id":"ba4084672a53a9784243fb1eec9f0a477aa4f541"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"79bd109b19038146461c903b2f89b0d15137f09a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"return sample info when creating sample with CLI"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It will return nothing after creating a sample successfully with"},{"line_number":10,"context_line":"ceilometer CLI. The information of the sample created should be return."},{"line_number":11,"context_line":"like other creating commands,such as \"ceilometer alarm-create\""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I60229fefa91de83e4b715cf4f92d3650e0a0d162"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"AAAAUH%2F%2BslQ%3D","line":10,"in_reply_to":"AAAAUH%2F%2BsxE%3D","updated":"2014-01-23 02:46:57.000000000","message":"thanks","commit_id":"ba4084672a53a9784243fb1eec9f0a477aa4f541"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"06f9d294ca8d5a3b45bc0d73fc32bd51f5b75d2e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It will return nothing after creating a sample successfully with"},{"line_number":10,"context_line":"ceilometer CLI. The information of the sample created should be return."},{"line_number":11,"context_line":"like other creating commands,such as \"ceilometer alarm-create\""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I60229fefa91de83e4b715cf4f92d3650e0a0d162"},{"line_number":14,"context_line":"Closes-bug: #1267713"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"AAAAUH%2F%2Bsw8%3D","line":11,"updated":"2014-01-23 01:58:33.000000000","message":"space after \u0027,\u0027","commit_id":"ba4084672a53a9784243fb1eec9f0a477aa4f541"}],"ceilometerclient/v2/shell.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"c236ee96c64061f9018073e12c781c9615055c6c","unresolved":false,"context_lines":[{"line_number":117,"context_line":"                fields[k] \u003d json.loads(v)"},{"line_number":118,"context_line":"            else:"},{"line_number":119,"context_line":"                fields[arg_to_field_mapping.get(k, k)] \u003d v"},{"line_number":120,"context_line":"    sample \u003d cc.samples.create(**fields)"},{"line_number":121,"context_line":"    fields \u003d [\u0027counter_name\u0027, \u0027user_id\u0027, \u0027resource_id\u0027,"},{"line_number":122,"context_line":"              \u0027timestamp\u0027, \u0027message_id\u0027, \u0027source\u0027, \u0027counter_unit\u0027,"},{"line_number":123,"context_line":"              \u0027counter_volume\u0027, \u0027project_id\u0027, \u0027resource_metadata\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUH%2F%2FeLo%3D","line":120,"updated":"2014-01-13 15:04:36.000000000","message":"this actually may return a list of samples. we should process all of them.","commit_id":"17a8860a1ab1e1df79e8d3179932f5c29d258311"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"f363c7bc0d387236d41919b946ade4635b14b88c","unresolved":false,"context_lines":[{"line_number":117,"context_line":"                fields[k] \u003d json.loads(v)"},{"line_number":118,"context_line":"            else:"},{"line_number":119,"context_line":"                fields[arg_to_field_mapping.get(k, k)] \u003d v"},{"line_number":120,"context_line":"    sample \u003d cc.samples.create(**fields)"},{"line_number":121,"context_line":"    fields \u003d [\u0027counter_name\u0027, \u0027user_id\u0027, \u0027resource_id\u0027,"},{"line_number":122,"context_line":"              \u0027timestamp\u0027, \u0027message_id\u0027, \u0027source\u0027, \u0027counter_unit\u0027,"},{"line_number":123,"context_line":"              \u0027counter_volume\u0027, \u0027project_id\u0027, \u0027resource_metadata\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUH%2F%2FX68%3D","line":120,"in_reply_to":"AAAAUH%2F%2FaRw%3D","updated":"2014-01-14 14:17:33.000000000","message":"liusheng, you are right. CLI does not support batch creation right now.","commit_id":"17a8860a1ab1e1df79e8d3179932f5c29d258311"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"346b76991d6b2279994dae911ed3b6b2d8ce3141","unresolved":false,"context_lines":[{"line_number":117,"context_line":"                fields[k] \u003d json.loads(v)"},{"line_number":118,"context_line":"            else:"},{"line_number":119,"context_line":"                fields[arg_to_field_mapping.get(k, k)] \u003d v"},{"line_number":120,"context_line":"    sample \u003d cc.samples.create(**fields)"},{"line_number":121,"context_line":"    fields \u003d [\u0027counter_name\u0027, \u0027user_id\u0027, \u0027resource_id\u0027,"},{"line_number":122,"context_line":"              \u0027timestamp\u0027, \u0027message_id\u0027, \u0027source\u0027, \u0027counter_unit\u0027,"},{"line_number":123,"context_line":"              \u0027counter_volume\u0027, \u0027project_id\u0027, \u0027resource_metadata\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUH%2F%2FaRw%3D","line":120,"in_reply_to":"AAAAUH%2F%2FeLo%3D","updated":"2014-01-14 01:37:08.000000000","message":"hi gordon chung, thanks of your comments, I have check the codes of here,IMHO,I don\u0027t think this can return a list of  samples,though, here the sample is type of list(dict). the v2 API of meters can create a batch of samples,but CLI can not.  Even though the CLI can do this in the future,  return one sample is reasonable.like \"nova boot\", it just return one instance info when creating more than one instance.","commit_id":"17a8860a1ab1e1df79e8d3179932f5c29d258311"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"f363c7bc0d387236d41919b946ade4635b14b88c","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    fields \u003d [\u0027counter_name\u0027, \u0027user_id\u0027, \u0027resource_id\u0027,"},{"line_number":122,"context_line":"              \u0027timestamp\u0027, \u0027message_id\u0027, \u0027source\u0027, \u0027counter_unit\u0027,"},{"line_number":123,"context_line":"              \u0027counter_volume\u0027, \u0027project_id\u0027, \u0027resource_metadata\u0027,"},{"line_number":124,"context_line":"              \u0027counter_type\u0027]"},{"line_number":125,"context_line":"    data \u003d dict([(f, getattr(sample[0], f, \u0027\u0027)) for f in fields])"},{"line_number":126,"context_line":"    utils.print_dict(data, wrap\u003d72)"},{"line_number":127,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUH%2F%2FX50%3D","line":124,"updated":"2014-01-14 14:17:33.000000000","message":"we don\u0027t use the \u0027counter\u0027 term in ceilometer anymore. we should add some logic to rename these fields to name, unit, volume, and type.","commit_id":"17a8860a1ab1e1df79e8d3179932f5c29d258311"}]}
