)]}'
{"tests/unit/test_inventory.py":[{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"e63f4a2fc2cd9824d560d468ea8f758c2695c81d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    def setUp(self, python_path\u003dNone):"},{"line_number":29,"context_line":"        super(TestInventoryBase, self).setUp()"},{"line_number":30,"context_line":"        if python_path:"},{"line_number":31,"context_line":"            self.fake_nodepool.python_path \u003d \u0027fake-python\u0027"},{"line_number":32,"context_line":"        self.executor_server.hold_jobs_in_build \u003d True"},{"line_number":33,"context_line":"        A \u003d self.fake_gerrit.addFakeChange(\u0027org/project\u0027, \u0027master\u0027, \u0027A\u0027)"},{"line_number":34,"context_line":"        self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_8efe8d03","line":31,"range":{"start_line":31,"start_character":46,"end_line":31,"end_character":57},"updated":"2019-05-17 17:53:09.000000000","message":"nit: that looks a little weird. Did you intend to set this to a constant value? If yes, I\u0027d expect python_path to be a boolean.","commit_id":"eb584342e0023512d2873981a9f8c1e40c4bd2e1"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"dbf5c6c1066396caf2dc660b874bf1f1b4c8eb51","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    def setUp(self, python_path\u003dNone):"},{"line_number":29,"context_line":"        super(TestInventoryBase, self).setUp()"},{"line_number":30,"context_line":"        if python_path:"},{"line_number":31,"context_line":"            self.fake_nodepool.python_path \u003d \u0027fake-python\u0027"},{"line_number":32,"context_line":"        self.executor_server.hold_jobs_in_build \u003d True"},{"line_number":33,"context_line":"        A \u003d self.fake_gerrit.addFakeChange(\u0027org/project\u0027, \u0027master\u0027, \u0027A\u0027)"},{"line_number":34,"context_line":"        self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_aeed9126","line":31,"range":{"start_line":31,"start_character":46,"end_line":31,"end_character":57},"in_reply_to":"bfb3d3c7_8efe8d03","updated":"2019-05-17 18:02:00.000000000","message":"Yah, was my attempt to modify the python-path of all nodes in nodepool, since we don\u0027t have a nodepool.yaml file.  We used the idea with zuul-executor zones too.","commit_id":"eb584342e0023512d2873981a9f8c1e40c4bd2e1"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"3fe9c3e3fbf253519e334577edee6428789aab9c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"class TestInventoryPythonPath(TestInventoryBase):"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def setUp(self):"},{"line_number":52,"context_line":"        super(TestInventoryPythonPath, self).setUp(python_path\u003d\u0027fake-python\u0027)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def test_single_inventory(self):"},{"line_number":55,"context_line":"        inventory \u003d self._get_build_inventory(\u0027single-inventory\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9fb8cfa7_57662f2c","line":52,"in_reply_to":"","updated":"2019-06-13 04:35:56.000000000","message":"In python3 this can just be super().setup...","commit_id":"5afac2cc19c7147673094afdba8e20484867160b"}]}