)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"c88f717f82d5dd42ef5e40d707f3c5e8e3d71d7b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mikolaj.Rzadca \u003cmikolaj.rzadca@mobica.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-05-22 16:30:37 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"collectd enable mcelog"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Depends-On: https://review.opendev.org/728125"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"ff570b3c_fc0b3328","line":7,"updated":"2020-05-22 19:20:56.000000000","message":"Could you please take the time to write a multi-line commit message.\n\nFor example:\n\n\"\"\"\"\ncollectd: add support for mcelog service\n\nAdding a new parameter, CollectdEnableMcelog (False by default) is set\nto True, a new host prep task will include the new\n\"tripleo_provision_mcelog\" role to configure the mcelog service on the\nhost where the collectd container is running.\n\"\"\"\"","commit_id":"147827fe07e40390480224b9c487fd0fbec40962"},{"author":{"_account_id":31988,"name":"Mikołaj Rządca","email":"mikolaj.rzadca@mobica.com","username":"hermes"},"change_message_id":"2a0ff5968c5d8c005c6e810145264fde3ff9516a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mikolaj.Rzadca \u003cmikolaj.rzadca@mobica.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-05-22 16:30:37 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"collectd enable mcelog"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Depends-On: https://review.opendev.org/728125"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"ff570b3c_ffa837dc","line":7,"in_reply_to":"ff570b3c_fc0b3328","updated":"2020-05-25 08:51:57.000000000","message":"sure","commit_id":"147827fe07e40390480224b9c487fd0fbec40962"}],"deployment/metrics/collectd-container-puppet.yaml":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"98ab3740b39f262b8e0ed6ffa2cdb4c7a3e78973","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    description: Service name or port number on which the qdrouterd will accept"},{"line_number":50,"context_line":"                 connections. This argument must be string, even if the numeric"},{"line_number":51,"context_line":"                 form is used."},{"line_number":52,"context_line":"    type: string"},{"line_number":53,"context_line":"  MetricsQdrUsername:"},{"line_number":54,"context_line":"    default: \u0027guest\u0027"},{"line_number":55,"context_line":"    description: Username which should be used to authenticate to the deployed"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ff570b3c_fbed67b7","line":52,"range":{"start_line":52,"start_character":10,"end_line":52,"end_character":16},"updated":"2020-05-18 14:27:30.000000000","message":"What\u0027s the reason for converting this to a string? This likely has larger impacts elsewhere","commit_id":"81adfc7624b6edf0bb24e0c9909b78ff5bfa6eae"},{"author":{"_account_id":31988,"name":"Mikołaj Rządca","email":"mikolaj.rzadca@mobica.com","username":"hermes"},"change_message_id":"c52c48cc0673ba038258e0986da4291f2d24044e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    description: Service name or port number on which the qdrouterd will accept"},{"line_number":50,"context_line":"                 connections. This argument must be string, even if the numeric"},{"line_number":51,"context_line":"                 form is used."},{"line_number":52,"context_line":"    type: string"},{"line_number":53,"context_line":"  MetricsQdrUsername:"},{"line_number":54,"context_line":"    default: \u0027guest\u0027"},{"line_number":55,"context_line":"    description: Username which should be used to authenticate to the deployed"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ff570b3c_f05101db","line":52,"range":{"start_line":52,"start_character":10,"end_line":52,"end_character":16},"in_reply_to":"ff570b3c_fbed67b7","updated":"2020-05-18 15:27:01.000000000","message":"sorry, that\u0027s not my change, I pulled the original file from GitHub instead of Opendev, maybe that\u0027s the cause of this","commit_id":"81adfc7624b6edf0bb24e0c9909b78ff5bfa6eae"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"98ab3740b39f262b8e0ed6ffa2cdb4c7a3e78973","unresolved":false,"context_lines":[{"line_number":705,"context_line":"        - name: set enable_mcelog fact"},{"line_number":706,"context_line":"          set_fact:"},{"line_number":707,"context_line":"            enable_mcelog: {get_param: CollectdEnableMcelog}"},{"line_number":708,"context_line":"        - name: set enable_snmpd fact"},{"line_number":709,"context_line":"          set_fact:"},{"line_number":710,"context_line":"            enable_snmpd: {get_param: CollectdEnableSnmpd}"},{"line_number":711,"context_line":"        - name: Configure rsyslog for container healthchecks"},{"line_number":712,"context_line":"          when:"},{"line_number":713,"context_line":"            - step|int \u003d\u003d 1"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ff570b3c_7b02f770","line":710,"range":{"start_line":708,"start_character":0,"end_line":710,"end_character":58},"updated":"2020-05-18 14:27:30.000000000","message":"As mentioned in the depends-on, we already have a tripleo service to configure snmp on a host. We should use that instead of this.","commit_id":"81adfc7624b6edf0bb24e0c9909b78ff5bfa6eae"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"726564c3b80faffe52cacd0bc7e3bfdf7dfa6f74","unresolved":false,"context_lines":[{"line_number":407,"context_line":"  sensubility_needs_sudo:"},{"line_number":408,"context_line":"    not:"},{"line_number":409,"context_line":"      equals: [{get_param: CollectdSensubilityExecSudoRule}, \u0027\u0027]"},{"line_number":410,"context_line":"  enable_mcelog:"},{"line_number":411,"context_line":"    equals: [{get_param: CollectdEnableMcelog}, true]"},{"line_number":412,"context_line":""},{"line_number":413,"context_line":"resources:"},{"line_number":414,"context_line":"  ContainersCommon:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ff570b3c_3de77b49","line":411,"range":{"start_line":410,"start_character":0,"end_line":411,"end_character":53},"updated":"2020-05-21 21:02:15.000000000","message":"You don\u0027t need this","commit_id":"e5cdc3280b3188aaca7da849a6d04ddebf0e35d9"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"726564c3b80faffe52cacd0bc7e3bfdf7dfa6f74","unresolved":false,"context_lines":[{"line_number":695,"context_line":"        - name: set enable_sensubility fact"},{"line_number":696,"context_line":"          set_fact:"},{"line_number":697,"context_line":"            enable_sensubility: {get_param: CollectdEnableSensubility}"},{"line_number":698,"context_line":"        - name: set enable_mcelog fact"},{"line_number":699,"context_line":"          set_fact:"},{"line_number":700,"context_line":"            enable_mcelog: {get_param: CollectdEnableMcelog}"},{"line_number":701,"context_line":"        - name: Configure rsyslog for container healthchecks"},{"line_number":702,"context_line":"          when:"},{"line_number":703,"context_line":"            - step|int \u003d\u003d 1"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ff570b3c_1d871764","line":700,"range":{"start_line":698,"start_character":0,"end_line":700,"end_character":60},"updated":"2020-05-21 21:02:15.000000000","message":"this needs to be either in the host_prep_tasks or dropped.","commit_id":"e5cdc3280b3188aaca7da849a6d04ddebf0e35d9"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"726564c3b80faffe52cacd0bc7e3bfdf7dfa6f74","unresolved":false,"context_lines":[{"line_number":744,"context_line":"        - name: import provision_mcelog"},{"line_number":745,"context_line":"          import_role:"},{"line_number":746,"context_line":"            name: tripleo_provision_mcelog"},{"line_number":747,"context_line":"          when: enable_mcelog \u003d\u003d \u0027true\u0027"},{"line_number":748,"context_line":"      fast_forward_upgrade_tasks:"},{"line_number":749,"context_line":"        - when:"},{"line_number":750,"context_line":"            - step|int \u003d\u003d 0"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ff570b3c_5d818f4a","line":747,"range":{"start_line":747,"start_character":16,"end_line":747,"end_character":39},"updated":"2020-05-21 21:02:15.000000000","message":"You can either use set fact right before this or just switch this to {get_param: CollectdEnableMcelog}","commit_id":"e5cdc3280b3188aaca7da849a6d04ddebf0e35d9"},{"author":{"_account_id":31988,"name":"Mikołaj Rządca","email":"mikolaj.rzadca@mobica.com","username":"hermes"},"change_message_id":"8c8ba3ff3ce44eac5dfa4e8eb03c3749d051b608","unresolved":false,"context_lines":[{"line_number":739,"context_line":"        - name: import provision_mcelog"},{"line_number":740,"context_line":"          import_role:"},{"line_number":741,"context_line":"            name: tripleo_provision_mcelog"},{"line_number":742,"context_line":"          when: {get_param: CollectdEnableMcelog} \u003d\u003d \u0027true\u0027"},{"line_number":743,"context_line":"      fast_forward_upgrade_tasks:"},{"line_number":744,"context_line":"        - when:"},{"line_number":745,"context_line":"            - step|int \u003d\u003d 0"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"ff570b3c_61fdc507","line":742,"range":{"start_line":742,"start_character":10,"end_line":742,"end_character":59},"updated":"2020-05-22 08:59:59.000000000","message":"like that?","commit_id":"da807e007e60b12ff1346fa3819b762aa2547208"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"08ee9f93185878542b79e4c3791a941ddf75df0c","unresolved":false,"context_lines":[{"line_number":739,"context_line":"        - name: import provision_mcelog"},{"line_number":740,"context_line":"          import_role:"},{"line_number":741,"context_line":"            name: tripleo_provision_mcelog"},{"line_number":742,"context_line":"          when: {get_param: CollectdEnableMcelog} \u003d\u003d \u0027true\u0027"},{"line_number":743,"context_line":"      fast_forward_upgrade_tasks:"},{"line_number":744,"context_line":"        - when:"},{"line_number":745,"context_line":"            - step|int \u003d\u003d 0"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"ff570b3c_b5e557df","line":742,"range":{"start_line":742,"start_character":10,"end_line":742,"end_character":59},"in_reply_to":"ff570b3c_61fdc507","updated":"2020-05-22 13:28:04.000000000","message":"Nope, just \"when: {get_param: CollectdEnableMcelog}\"","commit_id":"da807e007e60b12ff1346fa3819b762aa2547208"}]}