)]}'
{"ceilometer/hacking/checks.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"9809a28ac435396a4bedb02a819b387b0d60ffb0","unresolved":false,"context_lines":[{"line_number":29,"context_line":"import re"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"# TODO(zqfan): When other oslo libraries switch over non-namespace\u0027d"},{"line_number":33,"context_line":"# imports, we need to add them to the regexp below."},{"line_number":34,"context_line":"oslo_namespace_imports \u003d re.compile("},{"line_number":35,"context_line":"    r\"from[\\s]*oslo[.](concurrency|config|utils)\")"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa81d914_fd773a00","line":32,"updated":"2015-01-29 10:10:50.000000000","message":"@Lena Novokshonova\n\nyou have reverted this to PS4","commit_id":"c8a6866b5ea2a942ba4809f752cb876df6a159aa"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"30a091b6839771714a73475f21389828fdb4d126","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    # TODO(zqfan): When other oslo libraries switch over non-namespace\u0027d"},{"line_number":34,"context_line":"    # imports, we need to add them to the regexp below."},{"line_number":35,"context_line":"    oslo_namespace_imports \u003d re.compile("},{"line_number":36,"context_line":"        r\"from[\\s]*oslo[.](concurrency|config|utils)\")"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    # ignore openstack.common since they are not maintained by us"},{"line_number":39,"context_line":"    if \u0027ceilometer/openstack/common/\u0027 in filename:"}],"source_content_type":"text/x-python","patch_set":9,"id":"da86d52c_25c23734","line":36,"updated":"2015-02-05 05:24:37.000000000","message":"What about other oslo namespace? Could we loosen the RE here to cover all oslo namespaces, and temporarily disable C300 check for now? After we get all oslo changes merged, we could turn on this again.","commit_id":"354b40494cb2d3f63c5a0ee394085fa73ff91282"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"63b3084dcf55695d3dcdd02fe53070abae739834","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    # TODO(zqfan): When other oslo libraries switch over non-namespace\u0027d"},{"line_number":34,"context_line":"    # imports, we need to add them to the regexp below."},{"line_number":35,"context_line":"    oslo_namespace_imports \u003d re.compile("},{"line_number":36,"context_line":"        r\"from[\\s]*oslo[.](concurrency|config|utils)\")"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    # ignore openstack.common since they are not maintained by us"},{"line_number":39,"context_line":"    if \u0027ceilometer/openstack/common/\u0027 in filename:"}],"source_content_type":"text/x-python","patch_set":9,"id":"da86d52c_25fa3743","line":36,"in_reply_to":"da86d52c_25c23734","updated":"2015-02-05 05:41:43.000000000","message":"The main reason of this patch is easily discover the affected files when migrate and block new code contains deprecated namespace.\n\nif we disable it, then CI cannot check the accidentally introduced deprecated namespace, and the meaning of this patch is gone\n\nif aother module has done the migration and merged into our project, then that fix patch should add the new module to this pattern to enable CI check","commit_id":"354b40494cb2d3f63c5a0ee394085fa73ff91282"}]}