)]}'
{"modules/gerritbot/files/gerritbot_channel_config.yaml":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"7321af3f3c79148018d351f9497836785c0b9b0c","unresolved":false,"context_lines":[{"line_number":467,"context_line":"      - master"},{"line_number":468,"context_line":"      - grizzly"},{"line_number":469,"context_line":""},{"line_number":470,"context_line":"edeploy:"},{"line_number":471,"context_line":"    events:"},{"line_number":472,"context_line":"      - patchset-created"},{"line_number":473,"context_line":"      - change-merged"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"AAAAUH%2F%2FwQs%3D","line":470,"updated":"2014-01-08 21:20:43.000000000","message":"Please see the last line of http://ci.openstack.org/irc.html#gerritbot about adding access for our management bot account. This will allow the infrastructure team to more easily troubleshoot problems with gerritbot and any other infra services bots you may make use of in the future.","commit_id":"561ce28ba57cc792b74f7dd8dbc2977748cbea2e"}],"modules/openstack_project/files/gerrit/acls/stackforge/edeploy.config":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"7321af3f3c79148018d351f9497836785c0b9b0c","unresolved":false,"context_lines":[{"line_number":3,"context_line":"        label-Approved \u003d +0..+1 group edeploy-core"},{"line_number":4,"context_line":"        workInProgress \u003d group edeploy-core"},{"line_number":5,"context_line":"[access \"refs/tags/*\"]"},{"line_number":6,"context_line":"        create \u003d group edeploy-core"},{"line_number":7,"context_line":"        pushTag \u003d group edeploy-core"},{"line_number":8,"context_line":"[receive]"},{"line_number":9,"context_line":"        requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"AAAAUH%2F%2FwMA%3D","line":6,"updated":"2014-01-08 21:20:43.000000000","message":"Signing and pushing release tags is somewhat tricky and can\u0027t really be undone/redone if you make a mistake (you basically have a broken release and need to burn that release number and make the next tag higher when you redo it). As such, you should consider having a separate group for these permissions if you ever plan to have more than one or two people in your core group, and anyone with permission to do this also needs to be intimately familiar with https://wiki.openstack.org/wiki/GerritJenkinsGit#Tagging_a_Release since the Infrastructure team lacks sufficient manpower to provide any assistance on these matters.","commit_id":"561ce28ba57cc792b74f7dd8dbc2977748cbea2e"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"104439312876021f7e667475e50d559a6488ee4e","unresolved":false,"context_lines":[{"line_number":3,"context_line":"        label-Approved \u003d +0..+1 group edeploy-core"},{"line_number":4,"context_line":"        workInProgress \u003d group edeploy-core"},{"line_number":5,"context_line":"[access \"refs/tags/*\"]"},{"line_number":6,"context_line":"        create \u003d group edeploy-core"},{"line_number":7,"context_line":"        pushTag \u003d group edeploy-core"},{"line_number":8,"context_line":"[receive]"},{"line_number":9,"context_line":"        requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":2,"id":"AAAAUH%2F%2Fc4s%3D","line":6,"updated":"2014-01-13 18:52:24.000000000","message":"Signing and pushing release tags is somewhat tricky and can\u0027t really be undone/redone if you make a mistake (you basically have a broken release and need to burn that release number and make the next tag higher when you redo it). Anyone with permission to do this needs to be intimately familiar with https://wiki.openstack.org/wiki/GerritJenkinsGit#Tagging_a_Release since the Infrastructure team lacks sufficient manpower to provide any assistance on these matters.\n\nI\u0027ll see about patching the stackforge howto to make this a little clearer. I don\u0027t think we should be recommending to new projects to give all their core reviewers the ability to push release tags, since many of them end up growing large core teams. The ACL example tries to be too comprehensive without getting into the complexity of describing multiple release models. Your options are...\n\n1. You don\u0027t ever expect your core review group to ever grow past two or three people and you\u0027ll trust them all to also handle release management duties: then this is fine as-is.\n\n2. You don\u0027t plan to make releases of this project, at least not any time in the forseeable future, and will revisit things later when that changes: remove the access section for refs/tags/* so that Gerrit won\u0027t allow you to push tags to it for now.\n\n3. You plan to make releases of this project and you expect your core review team to grow past two or three people or won\u0027t want them all to have release permissions: replace the php-client-core group in the access section for refs/tags/* with one named php-client-drivers or php-client-ptl and that will allow you to have separate groups for your core reviewers and release managers.","commit_id":"d7dd95b7374b14264cba0a6d0ef563cb52055cd2"}],"modules/openstack_project/templates/review.projects.yaml.erb":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"7321af3f3c79148018d351f9497836785c0b9b0c","unresolved":false,"context_lines":[{"line_number":610,"context_line":"- project: stackforge/entropy"},{"line_number":611,"context_line":"  description: A framework for Audit/Repair scripts for openstack"},{"line_number":612,"context_line":"- project: stackforge/edeploy"},{"line_number":613,"context_line":"  description: \"Linux systems provisionning and updating made easy\""},{"line_number":614,"context_line":"  upstream: git://github.com/enovance/edeploy.git"},{"line_number":615,"context_line":"  acl-config: /home/gerrit2/acls/openstack/edeploy.config"}],"source_content_type":"application/x-erb","patch_set":1,"id":"AAAAUH%2F%2FwOs%3D","line":613,"updated":"2014-01-08 21:20:43.000000000","message":"Minor typo here? I think it probably should be \"provisioning\".","commit_id":"561ce28ba57cc792b74f7dd8dbc2977748cbea2e"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"7321af3f3c79148018d351f9497836785c0b9b0c","unresolved":false,"context_lines":[{"line_number":611,"context_line":"  description: A framework for Audit/Repair scripts for openstack"},{"line_number":612,"context_line":"- project: stackforge/edeploy"},{"line_number":613,"context_line":"  description: \"Linux systems provisionning and updating made easy\""},{"line_number":614,"context_line":"  upstream: git://github.com/enovance/edeploy.git"},{"line_number":615,"context_line":"  acl-config: /home/gerrit2/acls/openstack/edeploy.config"}],"source_content_type":"application/x-erb","patch_set":1,"id":"AAAAUH%2F%2FwMo%3D","line":614,"updated":"2014-01-08 21:20:43.000000000","message":"You have a lot of branches besides \"master\" in this repository on github:\n\n    debian/unstable\n    dell\n    hp\n    packaging\n    staging\n\nIf any of these aren\u0027t going to be used for code review and don\u0027t need to be published globally, please remove them or change this URL to refer to a cleaned-up fork where they\u0027re omitted. It\u0027s much easier to not import them in the first place than to have to ask for the Infra team admins to delete them once you\u0027re using Gerrit. Same question for the unsigned/lightweight 0.1 and 0.2 tags.","commit_id":"561ce28ba57cc792b74f7dd8dbc2977748cbea2e"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"7321af3f3c79148018d351f9497836785c0b9b0c","unresolved":false,"context_lines":[{"line_number":612,"context_line":"- project: stackforge/edeploy"},{"line_number":613,"context_line":"  description: \"Linux systems provisionning and updating made easy\""},{"line_number":614,"context_line":"  upstream: git://github.com/enovance/edeploy.git"},{"line_number":615,"context_line":"  acl-config: /home/gerrit2/acls/openstack/edeploy.config"}],"source_content_type":"application/x-erb","patch_set":1,"id":"AAAAUH%2F%2FwOI%3D","line":615,"updated":"2014-01-08 21:20:43.000000000","message":"This would actually be stackforge/edeploy.config not openstack, but the line isn\u0027t necessary anyway (the manage-projects script defaults to assuming your ACL file is named the same as your org/project with a .config extension).","commit_id":"561ce28ba57cc792b74f7dd8dbc2977748cbea2e"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"104439312876021f7e667475e50d559a6488ee4e","unresolved":false,"context_lines":[{"line_number":371,"context_line":"- project: stackforge/designate"},{"line_number":372,"context_line":"- project: stackforge/edeploy"},{"line_number":373,"context_line":"  description: \"Linux systems provisioning and updating made easy\""},{"line_number":374,"context_line":"  upstream: git://github.com/enovance/edeploy.git"},{"line_number":375,"context_line":"- project: stackforge/entropy"},{"line_number":376,"context_line":"  description: A framework for Audit/Repair scripts for openstack"},{"line_number":377,"context_line":"- project: stackforge/fuel-astute"}],"source_content_type":"application/x-erb","patch_set":2,"id":"AAAAUH%2F%2Fc5E%3D","line":374,"updated":"2014-01-13 18:52:24.000000000","message":"You have a few branches besides \"master\" in this repository on github:\n\n    debian/unstable\n    dell\n    hp\n    packaging\n\nIf any of these aren\u0027t going to be used for code review and don\u0027t need to be published globally, please remove them or change this URL to refer to a cleaned-up fork where they\u0027re omitted. It\u0027s much easier to not import them in the first place than to have to ask for the Infra team admins to delete them once you\u0027re using Gerrit. Same question for the unsigned/lightweight 0.1 and 0.2 tags.","commit_id":"d7dd95b7374b14264cba0a6d0ef563cb52055cd2"}]}