)]}'
{"oslo_rootwrap/tests/test_rootwrap.py":[{"author":{"_account_id":9459,"name":".Haifeng Yan","email":"yanheven@qq.com","username":"yan"},"change_message_id":"204786377b565e6f7c93657dad29181bc14cf29f","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        privsep \u003d [\"privsep-helper\", \"--context\", \"foo\"]"},{"line_number":36,"context_line":"        filterlist \u003d wrapper.load_filters([])"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        # mock out get_exec because"},{"line_number":39,"context_line":"        with mock.patch.object(filters.CommandFilter, \u0027get_exec\u0027) as ge:"},{"line_number":40,"context_line":"            ge.return_value \u003d \"/fake/privsep-helper\""},{"line_number":41,"context_line":"            filtermatch \u003d wrapper.match_filter(filterlist, privsep)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_91722410","line":38,"range":{"start_line":38,"start_character":28,"end_line":38,"end_character":35},"updated":"2016-07-24 04:45:45.000000000","message":"imcomplete sentence?","commit_id":"37c2a041d33f0fdce7ce2832398c1f60f3ee8703"},{"author":{"_account_id":9459,"name":".Haifeng Yan","email":"yanheven@qq.com","username":"yan"},"change_message_id":"204786377b565e6f7c93657dad29181bc14cf29f","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"            self.assertIsNotNone(filtermatch)"},{"line_number":44,"context_line":"            self.assertEqual(filtermatch.get_command(privsep),"},{"line_number":45,"context_line":"                             [\"/fake/privsep-helper\", \"--context\", \"foo\"])"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"class RootwrapTestCase(testtools.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_f15f2073","line":45,"range":{"start_line":45,"start_character":57,"end_line":45,"end_character":64},"updated":"2016-07-24 04:45:45.000000000","message":"it should be assertEqual(expected, actual)","commit_id":"37c2a041d33f0fdce7ce2832398c1f60f3ee8703"}],"oslo_rootwrap/wrapper.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"608e13b78e1e2de0bd4c526d36e2a4ebf789d23c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            self.exec_dirs \u003d []"},{"line_number":50,"context_line":"            # Use system PATH if exec_dirs is not specified"},{"line_number":51,"context_line":"            if \"PATH\" in os.environ:"},{"line_number":52,"context_line":"                self.exec_dirs \u003d os.environ[\u0027PATH\u0027].split(\u0027:\u0027)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        # syslog_log_facility"},{"line_number":55,"context_line":"        if config.has_option(\"DEFAULT\", \"syslog_log_facility\"):"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_3852a713","line":52,"updated":"2016-07-27 11:45:44.000000000","message":"This is the path processing part of rootwrap. The rule being added in code is largely the same as what is in the cinder config file right now, I\u0027m not sure I understand what needs to be changed here to support venvs. Because that would imply that this wouldn\u0027t work with the existing rules.\n\nIf you could be more specific on that point, that would be great.","commit_id":"37c2a041d33f0fdce7ce2832398c1f60f3ee8703"}]}