)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"35518bd233dc14b06613ddb07c7d458d110f2870","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Ian Wienand \u003ciwienand@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-07-01 14:03:18 +1000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"haproxy: redirectly logs to separate file"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"haproxy only logs to /dev/log; this means all our access logs get"},{"line_number":10,"context_line":"mixed into syslog.  This makes it impossible to pick out anything in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"125900c7_28353224","line":7,"range":{"start_line":7,"start_character":9,"end_line":7,"end_character":19},"updated":"2022-07-07 09:08:43.000000000","message":"Is that a word really?","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"6a90b60f5d3228997df0ba84bc7caa056bf160ca","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ian Wienand \u003ciwienand@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-07-01 14:03:18 +1000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"haproxy: redirectly logs to separate file"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"haproxy only logs to /dev/log; this means all our access logs get"},{"line_number":10,"context_line":"mixed into syslog.  This makes it impossible to pick out anything in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"36283fd3_3cd6285f","line":7,"range":{"start_line":7,"start_character":9,"end_line":7,"end_character":19},"in_reply_to":"102d1149_bf6459e4","updated":"2022-07-07 11:30:08.000000000","message":"Done","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"cb30f79e62fbae5f1c718301075f71e1493c30fe","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Ian Wienand \u003ciwienand@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-07-01 14:03:18 +1000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"haproxy: redirectly logs to separate file"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"haproxy only logs to /dev/log; this means all our access logs get"},{"line_number":10,"context_line":"mixed into syslog.  This makes it impossible to pick out anything in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"102d1149_bf6459e4","line":7,"range":{"start_line":7,"start_character":9,"end_line":7,"end_character":19},"in_reply_to":"125900c7_28353224","updated":"2022-07-07 11:28:04.000000000","message":"no it\u0027s a typo 😊","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"182c1f9091f64c6db90ae02e40356a57818a4c63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bb866d99_e610935f","updated":"2022-06-10 08:02:20.000000000","message":"passing but not quite working","commit_id":"96e01a24ce00d065d4dfe1141da6d0d178473b8f"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"35518bd233dc14b06613ddb07c7d458d110f2870","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e0f49c46_05fb8ced","updated":"2022-07-07 09:08:43.000000000","message":"I really like the idea of setting this up without the container actually knowing about it. Just some nits inline.","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"fc293bad7f4045d92721e3be3e3d24db54b06da0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"74ffdb8d_3c932dba","updated":"2022-07-19 23:18:18.000000000","message":"I will watch this closely as it merged to make sure it stays up","commit_id":"f97b9b8b8bb4dfad24ce4f2949f023ae900646e9"}],"playbooks/roles/haproxy/tasks/main.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"35518bd233dc14b06613ddb07c7d458d110f2870","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    mode: 0644"},{"line_number":29,"context_line":"  register: _rsyslog_added"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"- name: Resart rsyslog"},{"line_number":32,"context_line":"  service:"},{"line_number":33,"context_line":"    name: rsyslog"},{"line_number":34,"context_line":"    state: restarted"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"f3ca03f7_4130aaa7","line":31,"range":{"start_line":31,"start_character":8,"end_line":31,"end_character":14},"updated":"2022-07-07 09:08:43.000000000","message":"Nit: Restart","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"cb30f79e62fbae5f1c718301075f71e1493c30fe","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    mode: 0644"},{"line_number":29,"context_line":"  register: _rsyslog_added"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"- name: Resart rsyslog"},{"line_number":32,"context_line":"  service:"},{"line_number":33,"context_line":"    name: rsyslog"},{"line_number":34,"context_line":"    state: restarted"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"2dbb4601_358872e0","line":31,"range":{"start_line":31,"start_character":8,"end_line":31,"end_character":14},"in_reply_to":"f3ca03f7_4130aaa7","updated":"2022-07-07 11:28:04.000000000","message":"Done","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"35518bd233dc14b06613ddb07c7d458d110f2870","unresolved":true,"context_lines":[{"line_number":32,"context_line":"  service:"},{"line_number":33,"context_line":"    name: rsyslog"},{"line_number":34,"context_line":"    state: restarted"},{"line_number":35,"context_line":"  when: _rsyslog_added.changed"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"- name: Add haproxy log rotation"},{"line_number":38,"context_line":"  include_role:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"68964347_a231d1a6","line":35,"updated":"2022-07-07 09:08:43.000000000","message":"Shouldn\u0027t this be a handler?","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"cb30f79e62fbae5f1c718301075f71e1493c30fe","unresolved":true,"context_lines":[{"line_number":32,"context_line":"  service:"},{"line_number":33,"context_line":"    name: rsyslog"},{"line_number":34,"context_line":"    state: restarted"},{"line_number":35,"context_line":"  when: _rsyslog_added.changed"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"- name: Add haproxy log rotation"},{"line_number":38,"context_line":"  include_role:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bb272c36_fab6d3ac","line":35,"in_reply_to":"68964347_a231d1a6","updated":"2022-07-07 11:28:04.000000000","message":"AFAIK both achieve pretty much the same thing ... although the handler can be used generically, whereas this is only called once.  I can rework it if you feel strongly","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f638cabbb6b294379021a0235a676ff621c74c7d","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  service:"},{"line_number":33,"context_line":"    name: rsyslog"},{"line_number":34,"context_line":"    state: restarted"},{"line_number":35,"context_line":"  when: _rsyslog_added.changed"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"- name: Add haproxy log rotation"},{"line_number":38,"context_line":"  include_role:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"828b6d5c_2830703f","line":35,"in_reply_to":"bb272c36_fab6d3ac","updated":"2022-07-07 21:03:17.000000000","message":"No, it can stay this way, I was just thinking about consistency with the other reloads.","commit_id":"40b47766a89546ae5c8b996e61fe7b35ddf82ff9"}]}