)]}'
{"ceilometer/utils.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"fd7a3c5215608647a69e2c0e613e59cb949455f4","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"def setup_root_helper(conf):"},{"line_number":59,"context_line":"    global ROOTWRAP_CONF"},{"line_number":60,"context_line":"    ROOTWRAP_CONF \u003d conf.rootwrap_config"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"9a629dbe_a0d265fe","line":59,"updated":"2016-11-09 14:54:29.000000000","message":"why we replacing one global conf with another slightly less global conf? can\u0027t we just enforce people to pass it in to execute here: https://github.com/openstack/ceilometer/blob/ffc9ee99c10ede988769907fdb0594a512c890cd/ceilometer/ipmi/platform/ipmitool.py#L126","commit_id":"c71f79007a4626a123569b90c64e664748c8c2e2"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"ec197300dc58463f94aa1e3bbcedce8593d63f7c","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"def setup_root_helper(conf):"},{"line_number":59,"context_line":"    global ROOTWRAP_CONF"},{"line_number":60,"context_line":"    ROOTWRAP_CONF \u003d conf.rootwrap_config"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":18,"id":"9a629dbe_ea019707","line":59,"in_reply_to":"9a629dbe_a0d265fe","updated":"2016-11-09 15:56:46.000000000","message":"It was complicated to pass the conf to this, but I think I have cleanup enougth cfg stuffs on my branch I will retry this one.","commit_id":"c71f79007a4626a123569b90c64e664748c8c2e2"}]}