)]}'
{"tempest/config.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"e2c22b3c679b7ebf75b3b949bfb6596a1b7aaa76","unresolved":false,"context_lines":[{"line_number":1036,"context_line":"    cfg.BoolOpt(\u0027heat\u0027,"},{"line_number":1037,"context_line":"                default\u003dFalse,"},{"line_number":1038,"context_line":"                help\u003d\"Whether or not Heat is expected to be available\"),"},{"line_number":1039,"context_line":"    # TODO(gmann): Need to keep this config option as Congress is using this in"},{"line_number":1040,"context_line":"    # Kilo and Liberty branch and Ceilometer Tempest plugin is only present"},{"line_number":1041,"context_line":"    # since Mitaka."},{"line_number":1042,"context_line":"    cfg.BoolOpt(\u0027ceilometer\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_5ef360d6","line":1039,"updated":"2016-04-13 16:29:28.000000000","message":"Normally we\u0027ve just removed this if it\u0027s not used in-tree. The congress plugin should just depend on the new ceilo plugin to get this option if it relies on it. (although tbh they can treat it\u0027s absence as it being set to false) But, I\u0027m fine with lockstepping it","commit_id":"4c9d613b0ea8f08f8b461cbf3d308708be6d44b6"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"89f90b67165308a4b863f93f8fd1dbf11b7b2f8e","unresolved":false,"context_lines":[{"line_number":1036,"context_line":"    cfg.BoolOpt(\u0027heat\u0027,"},{"line_number":1037,"context_line":"                default\u003dFalse,"},{"line_number":1038,"context_line":"                help\u003d\"Whether or not Heat is expected to be available\"),"},{"line_number":1039,"context_line":"    # TODO(gmann): Need to keep this config option as Congress is using this in"},{"line_number":1040,"context_line":"    # Kilo and Liberty branch and Ceilometer Tempest plugin is only present"},{"line_number":1041,"context_line":"    # since Mitaka."},{"line_number":1042,"context_line":"    cfg.BoolOpt(\u0027ceilometer\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a18252c_9d0fac2d","line":1039,"in_reply_to":"5a18252c_5ef360d6","updated":"2016-04-13 23:42:59.000000000","message":"Yea, agree on this.\nyea, absence of config option(registered by other plugin) should be considered as False. They do for murano option.\n\nAfter this I will check if they are ok to skip tests for liberty/kilo and we can remove this accordingly.","commit_id":"4c9d613b0ea8f08f8b461cbf3d308708be6d44b6"}],"tempest/services/telemetry/json/alarming_client.py":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"f27a5283e40a3460ff638404d0d4dfb373cf831a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1a122d0e_bd0d9139","updated":"2016-04-22 11:17:06.000000000","message":"Should we move this client to the tempest/lib/services directory and only after that delete this file?","commit_id":"e4796f8de446aaaafa83902d1fb2d613331436cf"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"9a4e972adfc8d8c4ce9e9a9116f933ece19ccca4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bab6814e_02128cb4","in_reply_to":"1a122d0e_78c5de3b","updated":"2016-05-18 09:30:41.000000000","message":"If sahara wants to test integration with Ceilometer, it just needs to add the ceilometer plugin in the configuration of its jenkins jobs.","commit_id":"e4796f8de446aaaafa83902d1fb2d613331436cf"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"e918645b30aaf4c5c2152f04ab864acf983bc530","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1a122d0e_ced6bef0","in_reply_to":"1a122d0e_bd0d9139","updated":"2016-04-22 20:43:13.000000000","message":"The tempest lib clients are only gonna be for things with in tree tests (at least for right now, we can consider expanding this in the future) For right now the plugins will own the service clients","commit_id":"e4796f8de446aaaafa83902d1fb2d613331436cf"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"e50046d1317ae608c4979449ae991f7813bc4e23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1a122d0e_ccbcfcdd","in_reply_to":"1a122d0e_ced6bef0","updated":"2016-04-23 12:46:26.000000000","message":"Oh, thanks, Matt, for clarification. I didn\u0027t know it.","commit_id":"e4796f8de446aaaafa83902d1fb2d613331436cf"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"3f409dcefd344d45813e59207a5d5d7af416dbc9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1a122d0e_78c5de3b","in_reply_to":"1a122d0e_ced6bef0","updated":"2016-05-02 14:59:51.000000000","message":"so what if other service test needs ceilometer client?\n\nand what if some core service like sahara has its own plugin, if we remove its client in tempest, then when ceilometer want to test the integration with sahara, the only left way is use python-saharaclient, I think such way is not recommended in tempest developer document","commit_id":"e4796f8de446aaaafa83902d1fb2d613331436cf"}],"tempest/services/telemetry/json/telemetry_client.py":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"f27a5283e40a3460ff638404d0d4dfb373cf831a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1a122d0e_1d13e55d","updated":"2016-04-22 11:17:06.000000000","message":"ditto","commit_id":"e4796f8de446aaaafa83902d1fb2d613331436cf"}]}