)]}'
{"zuul/model.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"73e1b680fe2153ce21bfcd39a08159bc0efdd710","unresolved":false,"context_lines":[{"line_number":726,"context_line":"        self.canceled \u003d False"},{"line_number":727,"context_line":"        self.meta \u003d {}"},{"line_number":728,"context_line":"        if event is not None:"},{"line_number":729,"context_line":"            self.meta[\u0027zuul_event_id\u0027] \u003d event.zuul_event_id"},{"line_number":730,"context_line":""},{"line_number":731,"context_line":"    def _getPausedParent(self):"},{"line_number":732,"context_line":"        if self.build_set:"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfbec78f_e739bef1","line":729,"updated":"2019-05-15 08:37:07.000000000","message":"This is under the comment \"not stored in ZK\" but it is -- maybe move it above line 723?","commit_id":"3584b6dc1445904184317c5b7debfa3c5e3a0bbd"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"6fa662892ca643f205039f50d2bcf3941778193c","unresolved":false,"context_lines":[{"line_number":726,"context_line":"        self.canceled \u003d False"},{"line_number":727,"context_line":"        self.meta \u003d {}"},{"line_number":728,"context_line":"        if event is not None:"},{"line_number":729,"context_line":"            self.meta[\u0027zuul_event_id\u0027] \u003d event.zuul_event_id"},{"line_number":730,"context_line":""},{"line_number":731,"context_line":"    def _getPausedParent(self):"},{"line_number":732,"context_line":"        if self.build_set:"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfbec78f_1b9b0b66","line":729,"in_reply_to":"dfbec78f_aa9aabb5","updated":"2019-05-15 12:20:36.000000000","message":"Done","commit_id":"3584b6dc1445904184317c5b7debfa3c5e3a0bbd"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"12509584ff5e033a7ddf8eb77f8d77f0ca04610c","unresolved":false,"context_lines":[{"line_number":726,"context_line":"        self.canceled \u003d False"},{"line_number":727,"context_line":"        self.meta \u003d {}"},{"line_number":728,"context_line":"        if event is not None:"},{"line_number":729,"context_line":"            self.meta[\u0027zuul_event_id\u0027] \u003d event.zuul_event_id"},{"line_number":730,"context_line":""},{"line_number":731,"context_line":"    def _getPausedParent(self):"},{"line_number":732,"context_line":"        if self.build_set:"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfbec78f_aa9aabb5","line":729,"in_reply_to":"dfbec78f_e739bef1","updated":"2019-05-15 08:45:27.000000000","message":"oops, will fix, thanks","commit_id":"3584b6dc1445904184317c5b7debfa3c5e3a0bbd"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"3bd420f3d43ff339c2b60da1e5a97554abf212f9","unresolved":false,"context_lines":[{"line_number":720,"context_line":"        self.provider \u003d self._getPausedParentProvider()"},{"line_number":721,"context_line":"        self.id \u003d None"},{"line_number":722,"context_line":"        self._zk_data \u003d {}  # Data that we read back from ZK"},{"line_number":723,"context_line":"        self.meta \u003d {}"},{"line_number":724,"context_line":"        if event is not None:"},{"line_number":725,"context_line":"            self.meta[\u0027zuul_event_id\u0027] \u003d event.zuul_event_id"},{"line_number":726,"context_line":"        # Zuul internal flags (not stored in ZK so they are not"},{"line_number":727,"context_line":"        # overwritten)."},{"line_number":728,"context_line":"        self.failed \u003d False"}],"source_content_type":"text/x-python","patch_set":8,"id":"bfb3d3c7_219bdd72","line":725,"range":{"start_line":723,"start_character":0,"end_line":725,"end_character":60},"updated":"2019-05-20 14:20:37.000000000","message":"I don\u0027t see the need for a meta-level abstraction here. It\u0027s either part of the request or isn\u0027t. A top-level request attribute seems appropriate to me. But other than just requiring slightly more code, I don\u0027t have a problem with this technically.\n\nAlso, I prefer a non-zuul specific name. \u0027event_id\u0027 seems better.","commit_id":"1f4bb49825751d1497225bd2c304337a40f72663"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"82d34d46e67faca34b9ac7900e9e04703e70f66b","unresolved":false,"context_lines":[{"line_number":720,"context_line":"        self.provider \u003d self._getPausedParentProvider()"},{"line_number":721,"context_line":"        self.id \u003d None"},{"line_number":722,"context_line":"        self._zk_data \u003d {}  # Data that we read back from ZK"},{"line_number":723,"context_line":"        self.meta \u003d {}"},{"line_number":724,"context_line":"        if event is not None:"},{"line_number":725,"context_line":"            self.meta[\u0027zuul_event_id\u0027] \u003d event.zuul_event_id"},{"line_number":726,"context_line":"        # Zuul internal flags (not stored in ZK so they are not"},{"line_number":727,"context_line":"        # overwritten)."},{"line_number":728,"context_line":"        self.failed \u003d False"}],"source_content_type":"text/x-python","patch_set":8,"id":"bfb3d3c7_a104ad7c","line":725,"range":{"start_line":723,"start_character":0,"end_line":725,"end_character":60},"in_reply_to":"bfb3d3c7_219bdd72","updated":"2019-05-20 14:26:49.000000000","message":"Done","commit_id":"1f4bb49825751d1497225bd2c304337a40f72663"}]}
