)]}'
{"gerrit/acls/openstack-infra/puppet-system.config":[{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"93d329ce079a16ddaea450b7ac8ae2907bed4301","unresolved":false,"context_lines":[{"line_number":8,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":9,"context_line":"label-Verified \u003d -2..+2 group infra-core"},{"line_number":10,"context_line":"label-Workflow \u003d -1..+1 group infra-core"},{"line_number":11,"context_line":"submit \u003d group infra-core"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[access \"refs/tags/*\"]"},{"line_number":14,"context_line":"pushSignedTag \u003d group infra-core"}],"source_content_type":"text/x-ini","patch_set":5,"id":"baa201ad_704dbec3","line":11,"updated":"2014-10-07 22:53:31.000000000","message":"Do we really want to grant submit?","commit_id":"bcc1139fa7367463230c44cc32996b85cece59d9"},{"author":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"change_message_id":"93317162d30f691f586e53d541278e87ba6c2c50","unresolved":false,"context_lines":[{"line_number":8,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":9,"context_line":"label-Verified \u003d -2..+2 group infra-core"},{"line_number":10,"context_line":"label-Workflow \u003d -1..+1 group infra-core"},{"line_number":11,"context_line":"submit \u003d group infra-core"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[access \"refs/tags/*\"]"},{"line_number":14,"context_line":"pushSignedTag \u003d group infra-core"}],"source_content_type":"text/x-ini","patch_set":5,"id":"baa201ad_be6457b9","line":11,"in_reply_to":"baa201ad_704dbec3","updated":"2014-10-08 00:59:42.000000000","message":"I just copied it from a different acl file. I can remove it if you like.","commit_id":"bcc1139fa7367463230c44cc32996b85cece59d9"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f4139879aabec6138a1ed61fbaab040921c5041f","unresolved":false,"context_lines":[{"line_number":3,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":4,"context_line":"label-Workflow \u003d -1..+1 group infra-core"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[access \"refs/meta/config\"]"},{"line_number":7,"context_line":"abandon \u003d group infra-core"},{"line_number":8,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":9,"context_line":"label-Verified \u003d -2..+2 group infra-core"}],"source_content_type":"text/x-ini","patch_set":6,"id":"baa201ad_687fbc17","line":6,"updated":"2014-10-16 06:28:30.000000000","message":"Do we really need this?","commit_id":"e3cfbe018fdc00c3b22263be40c2112ee4a8c7cd"},{"author":{"_account_id":6609,"name":"Elizabeth K. Joseph","email":"lyz@princessleia.com","username":"lyz"},"change_message_id":"23d55699e93ebd28611bb7ed38fb2a3e15064f9d","unresolved":false,"context_lines":[{"line_number":3,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":4,"context_line":"label-Workflow \u003d -1..+1 group infra-core"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[access \"refs/meta/config\"]"},{"line_number":7,"context_line":"abandon \u003d group infra-core"},{"line_number":8,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":9,"context_line":"label-Verified \u003d -2..+2 group infra-core"}],"source_content_type":"text/x-ini","patch_set":6,"id":"baa201ad_35b32f0b","line":6,"in_reply_to":"baa201ad_220a178a","updated":"2014-10-16 23:32:34.000000000","message":"puppet-apparmor.config, puppet-dashboard.config and puppet-vcsrepo.config share this section, so I\u0027d like to hear from someone who knows why they exist","commit_id":"e3cfbe018fdc00c3b22263be40c2112ee4a8c7cd"},{"author":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"change_message_id":"7205cc59224fade025b28363d629d7ba91837b3f","unresolved":false,"context_lines":[{"line_number":3,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":4,"context_line":"label-Workflow \u003d -1..+1 group infra-core"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[access \"refs/meta/config\"]"},{"line_number":7,"context_line":"abandon \u003d group infra-core"},{"line_number":8,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":9,"context_line":"label-Verified \u003d -2..+2 group infra-core"}],"source_content_type":"text/x-ini","patch_set":6,"id":"9aa7fdbe_033bf53a","line":6,"in_reply_to":"baa201ad_35b32f0b","updated":"2014-10-20 22:02:56.000000000","message":"We found whre this comes from:\n\n14:46 \u003c fungi\u003e nibalizer: anteaya: mordred: the trail lands squarely in a history lesson... \n               https://review.openstack.org/#/c/15352/24/modules/openstack_project/files/gerrit/acls/openstack-ci-puppet.config\n14:46 \u003c fungi\u003e so long ago i barely remember reviewing that change\n14:47 \u003c@mordred\u003e fungi: wow. that was a long time ago\n14:48 \u003c fungi\u003e mordred: so basically the tracks go cold at the point where we imported the acls from what was previously being hand-managed in gerrit\n14:49 \u003c@mordred\u003e yeah -I think it was there beacuse we were hand managing it\n14:49 \u003c@mordred\u003e we\u0027d need that acl to hand manage the acl from git\n14:49 \u003c@mordred\u003e as one of us\n14:49 \u003c nibalizer\u003e if you were hand managing it it makes sense to have that acl\n14:56 \u003c anteaya\u003e mordred fungi so since we aren\u0027t hand managing anymore\n14:56 \u003c anteaya\u003e do we need that section?\n14:56 \u003c@mordred\u003e no","commit_id":"e3cfbe018fdc00c3b22263be40c2112ee4a8c7cd"},{"author":{"_account_id":6316,"name":"Anita Kuno","email":"anteaya@anteaya.info","username":"anteaya"},"change_message_id":"2205828d9b3080cb7b2640198f4bf1bc159e670c","unresolved":false,"context_lines":[{"line_number":3,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":4,"context_line":"label-Workflow \u003d -1..+1 group infra-core"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[access \"refs/meta/config\"]"},{"line_number":7,"context_line":"abandon \u003d group infra-core"},{"line_number":8,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":9,"context_line":"label-Verified \u003d -2..+2 group infra-core"}],"source_content_type":"text/x-ini","patch_set":6,"id":"baa201ad_220a178a","line":6,"in_reply_to":"baa201ad_629b3f66","updated":"2014-10-16 22:37:35.000000000","message":"Yeah, so let\u0027s lose this entire stanza, lines 6 - 11, you can always add it back if you really do need it.","commit_id":"e3cfbe018fdc00c3b22263be40c2112ee4a8c7cd"},{"author":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"change_message_id":"06e8e79b015b9f2583d80db18bcd6eb550f7effa","unresolved":false,"context_lines":[{"line_number":3,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":4,"context_line":"label-Workflow \u003d -1..+1 group infra-core"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[access \"refs/meta/config\"]"},{"line_number":7,"context_line":"abandon \u003d group infra-core"},{"line_number":8,"context_line":"label-Code-Review \u003d -2..+2 group infra-core"},{"line_number":9,"context_line":"label-Verified \u003d -2..+2 group infra-core"}],"source_content_type":"text/x-ini","patch_set":6,"id":"baa201ad_629b3f66","line":6,"in_reply_to":"baa201ad_687fbc17","updated":"2014-10-16 22:32:51.000000000","message":"I don\u0027t actually know what this does, so probably not?","commit_id":"e3cfbe018fdc00c3b22263be40c2112ee4a8c7cd"}],"gerrit/projects.yaml":[{"author":{"_account_id":6609,"name":"Elizabeth K. Joseph","email":"lyz@princessleia.com","username":"lyz"},"change_message_id":"9e1ed2b797a0878c6f57c899458ab5d8e65b9f6c","unresolved":false,"context_lines":[{"line_number":186,"context_line":"  groups:"},{"line_number":187,"context_line":"    - openstack-ci"},{"line_number":188,"context_line":"- project: openstack-infra/puppet-pip"},{"line_number":189,"context_line":"  upstream: git://github.com/nbializer/puppet-pip"},{"line_number":190,"context_line":"  description: Puppet module to add pip functionality"},{"line_number":191,"context_line":"  groups:"},{"line_number":192,"context_line":"    - openstack-ci"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"baa201ad_397b4c8d","line":189,"updated":"2014-10-06 16:37:51.000000000","message":"You spelled nibalizer wrong :D\n\nBut even spelled properly, this repo doesn\u0027t exist","commit_id":"cffd57d8e571f100129ef57caea27cd8efe77fd6"},{"author":{"_account_id":6316,"name":"Anita Kuno","email":"anteaya@anteaya.info","username":"anteaya"},"change_message_id":"2205828d9b3080cb7b2640198f4bf1bc159e670c","unresolved":false,"context_lines":[{"line_number":186,"context_line":"  groups:"},{"line_number":187,"context_line":"    - openstack-ci"},{"line_number":188,"context_line":"- project: openstack-infra/puppet-pip"},{"line_number":189,"context_line":"  upstream: git://github.com/nibalizer/puppet-module-openstack-pip"},{"line_number":190,"context_line":"  acl-config: /home/gerrit2/acls/openstack-infra/puppet-system.config"},{"line_number":191,"context_line":"  description: Puppet module to add pip functionality"},{"line_number":192,"context_line":"  groups:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"baa201ad_c29dcb67","line":189,"updated":"2014-10-16 22:37:35.000000000","message":"If we do a respin, having the .git file extension never hurts.","commit_id":"e3cfbe018fdc00c3b22263be40c2112ee4a8c7cd"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"46d4ab99346becb72d316f86c200f293613e37f4","unresolved":false,"context_lines":[{"line_number":1937,"context_line":"      - puppet-check-jobs"},{"line_number":1938,"context_line":""},{"line_number":1939,"context_line":"- project:"},{"line_number":1940,"context_line":"    name: puppet-pip"},{"line_number":1941,"context_line":"    node: \u0027bare-precise || bare-trusty\u0027"},{"line_number":1942,"context_line":"    tarball-site: tarballs.openstack.org"},{"line_number":1943,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"baa201ad_120cbaa4","line":1940,"updated":"2014-10-04 04:32:01.000000000","message":"This is not in alphabetical order, please add further up.","commit_id":"5b4355e3cd65946c0e30fef09552b4daae08061b"}],"zuul/layout.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"46d4ab99346becb72d316f86c200f293613e37f4","unresolved":false,"context_lines":[{"line_number":3307,"context_line":"      - gate-infra-puppet-apply-precise"},{"line_number":3308,"context_line":"      - gate-infra-puppet-apply-centos6"},{"line_number":3309,"context_line":""},{"line_number":3310,"context_line":"  - name: openstack-infra/puppet-pip"},{"line_number":3311,"context_line":"    template:"},{"line_number":3312,"context_line":"      - name: merge-check"},{"line_number":3313,"context_line":"    check:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"baa201ad_32fd3e6f","line":3310,"updated":"2014-10-04 04:32:01.000000000","message":"This is not in alphabetical order.","commit_id":"5b4355e3cd65946c0e30fef09552b4daae08061b"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"46d4ab99346becb72d316f86c200f293613e37f4","unresolved":false,"context_lines":[{"line_number":3311,"context_line":"    template:"},{"line_number":3312,"context_line":"      - name: merge-check"},{"line_number":3313,"context_line":"    check:"},{"line_number":3314,"context_line":"      - gate-puppet-storyboard-puppet-lint"},{"line_number":3315,"context_line":"      - gate-puppet-storyboard-puppet-syntax"},{"line_number":3316,"context_line":"      - gate-infra-puppet-apply-precise"},{"line_number":3317,"context_line":"      - gate-infra-puppet-apply-centos6"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"baa201ad_5202c26f","line":3314,"updated":"2014-10-04 04:32:01.000000000","message":"this looks like a pasto, shouldn\u0027t this be \"gate-puppet-pip-puppet-lint\" etc.?","commit_id":"5b4355e3cd65946c0e30fef09552b4daae08061b"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"8b9f6e1e878d78345ba176e0b5cc1871f938fa87","unresolved":false,"context_lines":[{"line_number":3297,"context_line":"    template:"},{"line_number":3298,"context_line":"      - name: merge-check"},{"line_number":3299,"context_line":"    check:"},{"line_number":3300,"context_line":"      - gate-puppet-pip-puppet-lint"},{"line_number":3301,"context_line":"      - gate-puppet-pip-puppet-syntax"},{"line_number":3302,"context_line":"      - gate-infra-puppet-apply-precise"},{"line_number":3303,"context_line":"      - gate-infra-puppet-apply-centos6"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"baa201ad_753949a3","line":3300,"updated":"2014-10-07 06:19:01.000000000","message":"You can use the template puppet-check-jobs here.","commit_id":"bcc1139fa7367463230c44cc32996b85cece59d9"}]}
