)]}'
{"specs/add-basic-ceilometer-tests.rst":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"0031d8d059066986c6eae25012e3f818366259ae","unresolved":false,"context_lines":[{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Now Ceilometer is the important metering/monitoring projects of OpenStack."},{"line_number":22,"context_line":"So it\u0027s necessary to include the basic tests into Tempest to make sure it works"},{"line_number":23,"context_line":"fine."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_21a21c99","line":22,"updated":"2014-06-04 14:35:21.000000000","message":"It may be worth saying that there basically aren\u0027t really any ceilo tests in tempest. Otherwise it doesn\u0027t really sound like there is a problem. :)","commit_id":"d9ec69c100f111f633fcac6de5451586c55698cd"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"0031d8d059066986c6eae25012e3f818366259ae","unresolved":false,"context_lines":[{"line_number":33,"context_line":"1. Initial Ceilometer Tempest integration"},{"line_number":34,"context_line":"-----------------------------------------"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"This point includes initial ceilometerclient testing code, base classes and"},{"line_number":37,"context_line":"configuration for the Telemetry tests. This step includes Telemetry client"},{"line_number":38,"context_line":"to be implemented as well."},{"line_number":39,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_8165b01f","line":36,"updated":"2014-06-04 14:35:21.000000000","message":"Do you mean CLI testing when you say ceilometer client testing? I think you should just explicitly outline that for this step you intend to add basic ceilometerclient support the CLI tests and the clients and base test classes for the API tests. Right now it\u0027s not exactly clear what this step is from this paragraph. (Although it\u0027s kind of obvious from the title)","commit_id":"d9ec69c100f111f633fcac6de5451586c55698cd"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"b1d0535b6d9a6757d8d68f2af8e3548b5e356580","unresolved":false,"context_lines":[{"line_number":33,"context_line":"1. Initial Ceilometer Tempest integration"},{"line_number":34,"context_line":"-----------------------------------------"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"This point includes initial ceilometerclient testing code, base classes and"},{"line_number":37,"context_line":"configuration for the Telemetry tests. This step includes Telemetry client"},{"line_number":38,"context_line":"to be implemented as well."},{"line_number":39,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_2aa2f198","line":36,"in_reply_to":"1ae5cdf2_8165b01f","updated":"2014-06-04 15:20:34.000000000","message":"oh, no, I meant adding the testing client code to the tempest - https://review.openstack.org/#/c/55276/ for the testing purposes","commit_id":"d9ec69c100f111f633fcac6de5451586c55698cd"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"0031d8d059066986c6eae25012e3f818366259ae","unresolved":false,"context_lines":[{"line_number":46,"context_line":"  * alarm-history API tests and alarming API itself"},{"line_number":47,"context_line":"  * notifications tests for the different types of the notifications (from"},{"line_number":48,"context_line":"    Nova, Cinder, Neutron, Swift, etc.)"},{"line_number":49,"context_line":"  * pollsters tests (with different services polling)"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_a11f0c6b","line":49,"updated":"2014-06-04 14:35:21.000000000","message":"are these the only 3 apis that will be tested in the API tests?","commit_id":"d9ec69c100f111f633fcac6de5451586c55698cd"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"0031d8d059066986c6eae25012e3f818366259ae","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  * notifications tests for the different types of the notifications (from"},{"line_number":48,"context_line":"    Nova, Cinder, Neutron, Swift, etc.)"},{"line_number":49,"context_line":"  * pollsters tests (with different services polling)"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Alternatives"},{"line_number":53,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_84013e07","line":50,"updated":"2014-06-04 14:35:21.000000000","message":"You included adding CLI test support in #1, did you want to outline what will be tested in the CLI tests in another section.","commit_id":"d9ec69c100f111f633fcac6de5451586c55698cd"}]}