)]}'
{"zuul/driver/gerrit/gerritconnection.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"f7b4292a99665b5b91f4b3e6c7c96726a87c0a44","unresolved":false,"context_lines":[{"line_number":1019,"context_line":"            if labels:"},{"line_number":1020,"context_line":"                data[\u0027labels\u0027] \u003d labels"},{"line_number":1021,"context_line":"            if file_comments:"},{"line_number":1022,"context_line":"                if self.version \u003e (2, 15, 0):"},{"line_number":1023,"context_line":"                    file_comments \u003d copy.deepcopy(file_comments)"},{"line_number":1024,"context_line":"                    url \u003d item.formatStatusUrl()"},{"line_number":1025,"context_line":"                    for comments in itertools.chain(file_comments.values()):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_c03691b5","line":1022,"updated":"2019-09-17 16:56:57.000000000","message":"Actually, all I really know is they are in the latest 2.15.  I assume they are in 2.15.0, but to be safe, why don\u0027t I set it to \u003e\u003d 2.15.16 for now?","commit_id":"84f3a06f49c622d6de09056bc92215eb06ee1946"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"b63bc55d222033b547b8b549a0d4d8e42be9066f","unresolved":false,"context_lines":[{"line_number":1019,"context_line":"            if labels:"},{"line_number":1020,"context_line":"                data[\u0027labels\u0027] \u003d labels"},{"line_number":1021,"context_line":"            if file_comments:"},{"line_number":1022,"context_line":"                if self.version \u003e (2, 15, 0):"},{"line_number":1023,"context_line":"                    file_comments \u003d copy.deepcopy(file_comments)"},{"line_number":1024,"context_line":"                    url \u003d item.formatStatusUrl()"},{"line_number":1025,"context_line":"                    for comments in itertools.chain(file_comments.values()):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_40970192","line":1022,"range":{"start_line":1022,"start_character":34,"end_line":1022,"end_character":44},"updated":"2019-09-17 16:54:25.000000000","message":"Do robot comments work in 2.15.1 but not 2.15.0? Or is this meant to be a \u003e\u003d change?","commit_id":"84f3a06f49c622d6de09056bc92215eb06ee1946"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"146a3fa83a654871a1c541b9eaf4e0ff2d522052","unresolved":false,"context_lines":[{"line_number":1024,"context_line":"                    url \u003d item.formatStatusUrl()"},{"line_number":1025,"context_line":"                    for comments in itertools.chain(file_comments.values()):"},{"line_number":1026,"context_line":"                        for comment in comments:"},{"line_number":1027,"context_line":"                            comment[\u0027robot_id\u0027] \u003d \u0027zuul\u0027"},{"line_number":1028,"context_line":"                            comment[\u0027robot_run_id\u0027] \u003d \\"},{"line_number":1029,"context_line":"                                item.current_build_set.uuid"},{"line_number":1030,"context_line":"                            if url:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_303d76bb","line":1027,"updated":"2019-09-17 14:31:23.000000000","message":"Possibly, but nothing is using these at the moment, so it\u0027s hard to tell what level of granularity would be useful.  I dug around a little bit and couldn\u0027t find any suggested uses for the field on the Gerrit side other than to test for its existence.  So my inclination was to keep it simple until we figured out it was a problem.\n\nWe could play it a little safer and add a robot_id option to the reporter that gets used here. That would let the admin set the robot_id for that pipeline and could also let them opt-in or -out of robot comments (though I don\u0027t see a disadvantage to using them).","commit_id":"84f3a06f49c622d6de09056bc92215eb06ee1946"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"d2f103f3f72cb509ce68f7301ee7d44a5c5288c2","unresolved":false,"context_lines":[{"line_number":1024,"context_line":"                    url \u003d item.formatStatusUrl()"},{"line_number":1025,"context_line":"                    for comments in itertools.chain(file_comments.values()):"},{"line_number":1026,"context_line":"                        for comment in comments:"},{"line_number":1027,"context_line":"                            comment[\u0027robot_id\u0027] \u003d \u0027zuul\u0027"},{"line_number":1028,"context_line":"                            comment[\u0027robot_run_id\u0027] \u003d \\"},{"line_number":1029,"context_line":"                                item.current_build_set.uuid"},{"line_number":1030,"context_line":"                            if url:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_f57cb721","line":1027,"updated":"2019-09-17 07:44:00.000000000","message":"What happens if there are more than one zuul reporting? Like - in opendev, for 3rd Party CI systems running zuul. Is there an issue with all of them using \"zuul\" as the robot id?","commit_id":"84f3a06f49c622d6de09056bc92215eb06ee1946"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"fbef8ad3f58320d7d6a10bc8da78414a191ed478","unresolved":false,"context_lines":[{"line_number":1024,"context_line":"                    url \u003d item.formatStatusUrl()"},{"line_number":1025,"context_line":"                    for comments in itertools.chain(file_comments.values()):"},{"line_number":1026,"context_line":"                        for comment in comments:"},{"line_number":1027,"context_line":"                            comment[\u0027robot_id\u0027] \u003d \u0027zuul\u0027"},{"line_number":1028,"context_line":"                            comment[\u0027robot_run_id\u0027] \u003d \\"},{"line_number":1029,"context_line":"                                item.current_build_set.uuid"},{"line_number":1030,"context_line":"                            if url:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_70a9aedd","line":1027,"in_reply_to":"3fa7e38b_303d76bb","updated":"2019-09-17 14:34:03.000000000","message":"That seems like a thing we could add in the future if we figure out that it\u0027s important to. :)","commit_id":"84f3a06f49c622d6de09056bc92215eb06ee1946"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2ca6be909fc6cdd1ef176df64aae3e29b074334c","unresolved":false,"context_lines":[{"line_number":1020,"context_line":"            if labels:"},{"line_number":1021,"context_line":"                data[\u0027labels\u0027] \u003d labels"},{"line_number":1022,"context_line":"            if file_comments:"},{"line_number":1023,"context_line":"                if self.version \u003e\u003d (2, 15, 0):"},{"line_number":1024,"context_line":"                    file_comments \u003d copy.deepcopy(file_comments)"},{"line_number":1025,"context_line":"                    url \u003d item.formatStatusUrl()"},{"line_number":1026,"context_line":"                    for comments in itertools.chain(file_comments.values()):"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_00aca98a","line":1023,"range":{"start_line":1023,"start_character":43,"end_line":1023,"end_character":44},"updated":"2019-09-17 17:04:21.000000000","message":"Based on your comment on the previous patchset I expect you mean for this 0 to become a 16.","commit_id":"d2e9321009e226e27fa71900717787fe15ed334d"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"893a550c5c7686884888eac4a724bc047c4369d7","unresolved":false,"context_lines":[{"line_number":1020,"context_line":"            if labels:"},{"line_number":1021,"context_line":"                data[\u0027labels\u0027] \u003d labels"},{"line_number":1022,"context_line":"            if file_comments:"},{"line_number":1023,"context_line":"                if self.version \u003e\u003d (2, 15, 0):"},{"line_number":1024,"context_line":"                    file_comments \u003d copy.deepcopy(file_comments)"},{"line_number":1025,"context_line":"                    url \u003d item.formatStatusUrl()"},{"line_number":1026,"context_line":"                    for comments in itertools.chain(file_comments.values()):"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_a0b6f5d8","line":1023,"range":{"start_line":1023,"start_character":43,"end_line":1023,"end_character":44},"in_reply_to":"3fa7e38b_00aca98a","updated":"2019-09-17 17:05:12.000000000","message":"Paladox clarified that this should go back to a 2.14 release even. So \u003e\u003d 2.15 should be safe.","commit_id":"d2e9321009e226e27fa71900717787fe15ed334d"}]}