)]}'
{"gerrit/projects.yaml":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"731a5173cd797b7b261b72a4314ccf8b2e6ffe40","unresolved":false,"context_lines":[{"line_number":1255,"context_line":"  description: OpenStack Monasca Puppet Module"},{"line_number":1256,"context_line":"- project: openstack/puppet-murano"},{"line_number":1257,"context_line":"  description: OpenStack Murano Puppet Module"},{"line_number":1258,"context_line":"  acl-config: /home/gerrit2/acls/stackforge/puppet-murano.config"},{"line_number":1259,"context_line":"- project: openstack/puppet-neutron"},{"line_number":1260,"context_line":"  description: OpenStack Neutron Puppet Module"},{"line_number":1261,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-modules.config"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fa32b979_36d2c235","line":1258,"updated":"2015-06-19 14:46:36.000000000","message":"this should be moved under the acls/openstack folder","commit_id":"63815cbaec9002be6fd5a1d6aeb67cf695441ab4"},{"author":{"_account_id":6316,"name":"Anita Kuno","email":"anteaya@anteaya.info","username":"anteaya"},"change_message_id":"6866b443e9f1f92fda13113ed27e9cadf13658fe","unresolved":false,"context_lines":[{"line_number":1265,"context_line":"  description: OpenStack Monasca Puppet Module"},{"line_number":1266,"context_line":"- project: openstack/puppet-murano"},{"line_number":1267,"context_line":"  description: OpenStack Murano Puppet Module"},{"line_number":1268,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-murano.config"},{"line_number":1269,"context_line":"- project: openstack/puppet-neutron"},{"line_number":1270,"context_line":"  description: OpenStack Neutron Puppet Module"},{"line_number":1271,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-modules.config"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ba3cc151_5cc1c74c","line":1268,"updated":"2015-07-09 18:03:12.000000000","message":"This line isn\u0027t required. It doesn\u0027t hurt but isn\u0027t necessary as the project name and the name of the acl file match.","commit_id":"ae30f35c7e45520560e869d4a06a30c9bf26d8b1"},{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"96eb624e871ef1ccf3e35128d2fc083658a5a1bb","unresolved":false,"context_lines":[{"line_number":1265,"context_line":"  description: OpenStack Monasca Puppet Module"},{"line_number":1266,"context_line":"- project: openstack/puppet-murano"},{"line_number":1267,"context_line":"  description: OpenStack Murano Puppet Module"},{"line_number":1268,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-murano.config"},{"line_number":1269,"context_line":"- project: openstack/puppet-neutron"},{"line_number":1270,"context_line":"  description: OpenStack Neutron Puppet Module"},{"line_number":1271,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-modules.config"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ba3cc151_1cac7134","line":1268,"in_reply_to":"ba3cc151_5cc1c74c","updated":"2015-07-10 14:56:55.000000000","message":"Done","commit_id":"ae30f35c7e45520560e869d4a06a30c9bf26d8b1"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"753d0352819290f9467036254c4e0c1355910a6c","unresolved":false,"context_lines":[{"line_number":1300,"context_line":"- project: openstack/puppet-monasca"},{"line_number":1301,"context_line":"  description: OpenStack Monasca Puppet Module"},{"line_number":1302,"context_line":"- project: openstack/puppet-murano"},{"line_number":1303,"context_line":"  description: OpenStack Murano Puppet Module"},{"line_number":1304,"context_line":"- project: openstack/puppet-neutron"},{"line_number":1305,"context_line":"  description: OpenStack Neutron Puppet Module"},{"line_number":1306,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-modules.config"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ba3cc151_e567b0b5","line":1303,"updated":"2015-07-10 15:28:00.000000000","message":"I would like to see puppet-modules ACL, because it will be own by Puppet OpenStack group OR a comment that says currently we use puppet-murano ACL, but you add Puppet OpenStack members into the core permissions for this module.","commit_id":"2503ff61ffad0383f4e3b5293caa08474b244688"},{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"8f686642c00e35d03d63788eea60486812a12863","unresolved":false,"context_lines":[{"line_number":1300,"context_line":"- project: openstack/puppet-monasca"},{"line_number":1301,"context_line":"  description: OpenStack Monasca Puppet Module"},{"line_number":1302,"context_line":"- project: openstack/puppet-murano"},{"line_number":1303,"context_line":"  description: OpenStack Murano Puppet Module"},{"line_number":1304,"context_line":"- project: openstack/puppet-neutron"},{"line_number":1305,"context_line":"  description: OpenStack Neutron Puppet Module"},{"line_number":1306,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-modules.config"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ba3cc151_b085645a","line":1303,"in_reply_to":"ba3cc151_e567b0b5","updated":"2015-07-10 15:50:50.000000000","message":"Done.\n\nThe line was removed in response to comment for 3d patch-set, btw.","commit_id":"2503ff61ffad0383f4e3b5293caa08474b244688"},{"author":{"_account_id":6316,"name":"Anita Kuno","email":"anteaya@anteaya.info","username":"anteaya"},"change_message_id":"10e986808106eac8a877f5ab08c8be2891cb5979","unresolved":false,"context_lines":[{"line_number":1300,"context_line":"- project: openstack/puppet-monasca"},{"line_number":1301,"context_line":"  description: OpenStack Monasca Puppet Module"},{"line_number":1302,"context_line":"- project: openstack/puppet-murano"},{"line_number":1303,"context_line":"  description: OpenStack Murano Puppet Module"},{"line_number":1304,"context_line":"- project: openstack/puppet-neutron"},{"line_number":1305,"context_line":"  description: OpenStack Neutron Puppet Module"},{"line_number":1306,"context_line":"  acl-config: /home/gerrit2/acls/openstack/puppet-modules.config"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ba3cc151_6b5b71f7","line":1303,"in_reply_to":"ba3cc151_e567b0b5","updated":"2015-07-10 15:58:01.000000000","message":"I didn\u0027t know you wanted to see acl paths. Sorry Emilien. I asked to have it removed because it can cause confusion if the path has a typo in it. That is fine if you want to see them for puppet modules. Maybe let other reviewers know what you want to see here as well.","commit_id":"2503ff61ffad0383f4e3b5293caa08474b244688"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":6609,"name":"Elizabeth K. Joseph","email":"lyz@princessleia.com","username":"lyz"},"change_message_id":"2da0fbd6f4a1427677599cf8f170aa8c8b72ed1e","unresolved":false,"context_lines":[{"line_number":2342,"context_line":"    jobs:"},{"line_number":2343,"context_line":"      - puppet-check-jobs"},{"line_number":2344,"context_line":"      - puppet-module-unit"},{"line_number":2345,"context_line":"      - puppet-beaker-jobs"},{"line_number":2346,"context_line":"      - gate-{name}-puppet-syntax-future"},{"line_number":2347,"context_line":""},{"line_number":2348,"context_line":"- project:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ba3cc151_80e4bd0c","line":2345,"updated":"2015-07-09 20:36:11.000000000","message":"This job being added along with the rename. We don\u0027t really like seeing this done since it complicates a simple rename, but if this is intentional of some reason, needs to at least be noted in the commit message.","commit_id":"ae30f35c7e45520560e869d4a06a30c9bf26d8b1"},{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"96eb624e871ef1ccf3e35128d2fc083658a5a1bb","unresolved":false,"context_lines":[{"line_number":2342,"context_line":"    jobs:"},{"line_number":2343,"context_line":"      - puppet-check-jobs"},{"line_number":2344,"context_line":"      - puppet-module-unit"},{"line_number":2345,"context_line":"      - puppet-beaker-jobs"},{"line_number":2346,"context_line":"      - gate-{name}-puppet-syntax-future"},{"line_number":2347,"context_line":""},{"line_number":2348,"context_line":"- project:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ba3cc151_7c5edd7a","line":2345,"in_reply_to":"ba3cc151_80e4bd0c","updated":"2015-07-10 14:56:55.000000000","message":"Will add a separate dependant commit with it. Feels like a right thing to do.","commit_id":"ae30f35c7e45520560e869d4a06a30c9bf26d8b1"}]}