)]}'
{"doc/source/gerrit.rst":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"unresolved":false,"context_lines":[{"line_number":690,"context_line":"should have a value that is a path to the project.config for that"},{"line_number":691,"context_line":"project."},{"line_number":692,"context_line":""},{"line_number":693,"context_line":"That is the high level view of how we are can configure projects using"},{"line_number":694,"context_line":"the pupppet repository. To create an actual change that does all of this"},{"line_number":695,"context_line":"for a single project you will want to do the following:"},{"line_number":696,"context_line":""}],"source_content_type":"text/x-rst","patch_set":17,"id":"AAAAMH%2F%2F9n4%3D","line":693,"updated":"2012-11-14 00:27:17.000000000","message":"s/are//","commit_id":"ab071f7405aa860ba654b89759d9b1c8067dc67b"}],"manifests/site.pp":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":28,"context_line":"    ssh_rsa_key_contents    \u003d\u003e hiera(\u0027gerrit_ssh_rsa_key_contents\u0027),"},{"line_number":29,"context_line":"    ssh_rsa_pubkey_contents \u003d\u003e hiera(\u0027gerrit_ssh_rsa_pubkey_contents\u0027),"},{"line_number":30,"context_line":"    ssh_rsa_project_key_contents    \u003d\u003e hiera(\u0027gerrit_ssh_rsa_key_contents\u0027),"},{"line_number":31,"context_line":"    ssh_rsa_project_pubkey_contents \u003d\u003e hiera(\u0027gerrit_ssh_rsa_pubkey_contents\u0027),"},{"line_number":32,"context_line":"    lp_sync_key             \u003d\u003e hiera(\u0027gerrit_lp_sync_key\u0027),"},{"line_number":33,"context_line":"    lp_sync_pubkey          \u003d\u003e hiera(\u0027gerrit_lp_sync_pubkey\u0027),"},{"line_number":34,"context_line":"    lp_sync_consumer_key    \u003d\u003e hiera(\u0027gerrit_lp_consumer_key\u0027),"}],"source_content_type":"text/x-puppet","patch_set":14,"id":"AAAAMH%2F%2F%2FY8%3D","line":31,"updated":"2012-11-12 18:13:54.000000000","message":"Do we need ssh_rsa_project_key_contents and ssh_rsa_project_pubkey_contents when we have ssh_rsa_key_contents and ssh_rsa_pubkey_contents that have the same values? I guess this gives us more flexibility, but is currently not used that way.\n\nAlso puppet lint will want you to align the rocket ships.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":28,"context_line":"    ssh_rsa_key_contents    \u003d\u003e hiera(\u0027gerrit_ssh_rsa_key_contents\u0027),"},{"line_number":29,"context_line":"    ssh_rsa_pubkey_contents \u003d\u003e hiera(\u0027gerrit_ssh_rsa_pubkey_contents\u0027),"},{"line_number":30,"context_line":"    ssh_rsa_project_key_contents    \u003d\u003e hiera(\u0027gerrit_ssh_rsa_key_contents\u0027),"},{"line_number":31,"context_line":"    ssh_rsa_project_pubkey_contents \u003d\u003e hiera(\u0027gerrit_ssh_rsa_pubkey_contents\u0027),"},{"line_number":32,"context_line":"    lp_sync_key             \u003d\u003e hiera(\u0027gerrit_lp_sync_key\u0027),"},{"line_number":33,"context_line":"    lp_sync_pubkey          \u003d\u003e hiera(\u0027gerrit_lp_sync_pubkey\u0027),"},{"line_number":34,"context_line":"    lp_sync_consumer_key    \u003d\u003e hiera(\u0027gerrit_lp_consumer_key\u0027),"}],"source_content_type":"text/x-puppet","patch_set":14,"id":"AAAAMH%2F%2F%2B4I%3D","line":31,"in_reply_to":"AAAAMH%2F%2F%2FY8%3D","updated":"2012-11-12 23:46:17.000000000","message":"The data getting sucked out of hiera is wrong here and should be what review2 is using.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":38,"context_line":"  }"},{"line_number":39,"context_line":"}"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"node \u0027review2.openstack.org\u0027 {"},{"line_number":42,"context_line":"  class { \u0027openstack_project::review\u0027:"},{"line_number":43,"context_line":"    github_oauth_token      \u003d\u003e hiera(\u0027gerrit_github_token\u0027),"},{"line_number":44,"context_line":"    github_project_username \u003d\u003e hiera(\u0027github_project_username\u0027),"}],"source_content_type":"text/x-puppet","patch_set":14,"id":"AAAAMH%2F%2F%2FY4%3D","line":41,"updated":"2012-11-12 18:13:54.000000000","message":"We can actually just remove review2 at this point I think. Rather than trying to keep it up to date.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":81,"context_line":"    ssh_rsa_key_contents    \u003d\u003e hiera(\u0027gerrit_dev_ssh_rsa_key_contents\u0027),"},{"line_number":82,"context_line":"    ssh_rsa_pubkey_contents \u003d\u003e hiera(\u0027gerrit_dev_ssh_rsa_pubkey_contents\u0027),"},{"line_number":83,"context_line":"    ssh_project_rsa_key_contents    \u003d\u003e hiera(\u0027gerrit_dev_project_ssh_rsa_key_contents\u0027),"},{"line_number":84,"context_line":"    ssh_project_rsa_pubkey_contents \u003d\u003e hiera(\u0027gerrit_dev_project_ssh_rsa_pubkey_contents\u0027),"},{"line_number":85,"context_line":"    lp_sync_key             \u003d\u003e hiera(\u0027gerrit_dev_lp_sync_key\u0027),"},{"line_number":86,"context_line":"    lp_sync_pubkey          \u003d\u003e hiera(\u0027gerrit_dev_lp_sync_pubkey\u0027),"},{"line_number":87,"context_line":"    lp_sync_consumer_key    \u003d\u003e hiera(\u0027gerrit_dev_lp_consumer_key\u0027),"}],"source_content_type":"text/x-puppet","patch_set":14,"id":"AAAAMH%2F%2F%2FY0%3D","line":84,"updated":"2012-11-12 18:13:54.000000000","message":"See above comment about necessity for extra parameters and aligning rocket ships.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"}],"modules/gerrit/files/scripts/make_local_repos.py":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"25e5d5a6e19302df4772b8b3aeafcfbe70467d37","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#! /usr/bin/env python"},{"line_number":2,"context_line":"# Copyright (C) 2011 OpenStack, LLC."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2Fw9c%3D","line":1,"updated":"2012-11-07 22:55:02.000000000","message":"I always forget to append the current year on these when I make changes. I wonder if that\u0027s maybe a good candidate for testing, eventually (makes for an interesting regular expression experiment)?","commit_id":"08b0fe0918132805b9fb91bb7f13d482ee5653ec"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"25e5d5a6e19302df4772b8b3aeafcfbe70467d37","unresolved":false,"context_lines":[{"line_number":60,"context_line":"GITHUB_SECURE_CONFIG \u003d os.environ.get(\u0027GITHUB_SECURE_CONFIG\u0027,"},{"line_number":61,"context_line":"                                      \u0027/etc/github/github.secure.config\u0027)"},{"line_number":62,"context_line":"GERRIT_CONFIG \u003d os.environ.get(\u0027GERRIT_CONFIG\u0027,"},{"line_number":63,"context_line":"                               \u0027/etc/gerrit/gerrit.secure.config\u0027)"},{"line_number":64,"context_line":" "},{"line_number":65,"context_line":"secure_config \u003d ConfigParser.ConfigParser()"},{"line_number":66,"context_line":"secure_config.read(GITHUB_SECURE_CONFIG)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2Fw9Y%3D","line":63,"updated":"2012-11-07 22:55:02.000000000","message":"I don\u0027t see this file on our gerrit servers... are we overriding in the calling environment, or do we ultimately want to symlink ~gerrit2/review_site/etc/secure.config here?","commit_id":"08b0fe0918132805b9fb91bb7f13d482ee5653ec"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"25e5d5a6e19302df4772b8b3aeafcfbe70467d37","unresolved":false,"context_lines":[{"line_number":106,"context_line":"                                           homepage\u003d\u0027http://openstack.org\u0027,"},{"line_number":107,"context_line":"                                           has_issues\u003dFalse,"},{"line_number":108,"context_line":"                                           has_downloads\u003dFalse,"},{"line_number":109,"context_line":"                                           team_id\u003d153703,"},{"line_number":110,"context_line":"                                           has_wiki \u003d False)"},{"line_number":111,"context_line":"    if description:"},{"line_number":112,"context_line":"        ghub.get_user().edit_repo(repo_name, description\u003ddescription)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2FwvE%3D","line":109,"updated":"2012-11-07 22:55:02.000000000","message":"Should this be extracted out to a config or something? (Maybe same with the homepage above?)","commit_id":"08b0fe0918132805b9fb91bb7f13d482ee5653ec"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"25e5d5a6e19302df4772b8b3aeafcfbe70467d37","unresolved":false,"context_lines":[{"line_number":113,"context_line":""},{"line_number":114,"context_line":"    if not os.path.exists(project_dir):"},{"line_number":115,"context_line":"        run_command(\"git --bare init %s\" % project_dir)"},{"line_number":116,"context_line":"        run_command(\"chown -R gerrit2:gerrit2 %s\" % project_dir)"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    if project not in project_list:"},{"line_number":119,"context_line":"        gerrit.createProject(project)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2FwvA%3D","line":116,"updated":"2012-11-07 22:55:02.000000000","message":"User/group might be more externalized config fodder.","commit_id":"08b0fe0918132805b9fb91bb7f13d482ee5653ec"}],"modules/gerrit/files/scripts/manage_projects.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"unresolved":false,"context_lines":[{"line_number":90,"context_line":"    project \u003d section[\u0027project\u0027]"},{"line_number":91,"context_line":"    options \u003d section[\u0027options\u0027]"},{"line_number":92,"context_line":"    description \u003d section.get(\u0027description\u0027, None)"},{"line_number":93,"context_line":"    homepage \u003d section.get(\u0027homepage\u0027, \u0027http://openstack.org\u0027)"},{"line_number":94,"context_line":"    upstream \u003d section.get(\u0027upstream\u0027, None)"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    project_git \u003d \"%s.git\" % project"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAALn%2F%2Fv10%3D","line":93,"updated":"2012-11-08 17:57:48.000000000","message":"how about a defaults section in the yaml where things like the default homepage can be set, as well as defaults for the has-* booleans.","commit_id":"7556133d432330df9958de3fbc2d36cd0f36ad01"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"unresolved":false,"context_lines":[{"line_number":112,"context_line":"                                           homepage\u003dhomepage,"},{"line_number":113,"context_line":"                                           has_issues\u003dhas_issues,"},{"line_number":114,"context_line":"                                           has_downloads\u003dhas_downloads,"},{"line_number":115,"context_line":"                                           team_id\u003d153703,"},{"line_number":116,"context_line":"                                           has_wiki\u003dhas_wiki)"},{"line_number":117,"context_line":"    if description:"},{"line_number":118,"context_line":"        ghub.get_user().edit_repo(repo_name,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAALn%2F%2Fv1s%3D","line":115,"updated":"2012-11-08 17:57:48.000000000","message":"maybe make the team id a per-project option that can also be set in defaults.","commit_id":"7556133d432330df9958de3fbc2d36cd0f36ad01"}],"modules/gerrit/manifests/init.pp":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"25e5d5a6e19302df4772b8b3aeafcfbe70467d37","unresolved":false,"context_lines":[{"line_number":148,"context_line":"  package { \u0027gerritlib\u0027:"},{"line_number":149,"context_line":"    ensure   \u003d\u003e latest,"},{"line_number":150,"context_line":"    provider \u003d\u003e \u0027pip\u0027,"},{"line_number":151,"context_line":"    require \u003d\u003e Class[pip],"},{"line_number":152,"context_line":"  }"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"  file { \u0027/var/log/gerrit\u0027:"}],"source_content_type":"text/x-puppet","patch_set":18,"id":"AAAAMH%2F%2F9nI%3D","line":151,"updated":"2012-11-14 00:34:31.000000000","message":"align all the rocketships?","commit_id":"ef3cdd1b37888e2ec0aac1f660b6cbdf3c616cde"}],"modules/github/templates/github-projects.secure.config.erb":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"[github]"},{"line_number":2,"context_line":"username \u003d \u003c%\u003d project_username %\u003e"},{"line_number":3,"context_line":"password \u003d \u003c%\u003d project_password %\u003e"}],"source_content_type":"application/x-erb","patch_set":14,"id":"AAAAMH%2F%2F%2FW8%3D","line":1,"updated":"2012-11-12 18:13:54.000000000","message":"Didn\u0027t you find that the token auth was needed to perform some of the github functions that you needed to do?","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"[github]"},{"line_number":2,"context_line":"username \u003d \u003c%\u003d project_username %\u003e"},{"line_number":3,"context_line":"password \u003d \u003c%\u003d project_password %\u003e"}],"source_content_type":"application/x-erb","patch_set":14,"id":"AAAAMH%2F%2F%2B4Y%3D","line":1,"in_reply_to":"AAAAMH%2F%2F%2FW8%3D","updated":"2012-11-12 23:46:17.000000000","message":"I misinterpreted this. The username + password auth is required to perform the functions.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"}],"modules/openstack_project/files/review.projects.yaml":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"25e5d5a6e19302df4772b8b3aeafcfbe70467d37","unresolved":false,"context_lines":[{"line_number":10,"context_line":"- project: openstack-ci/gerrit"},{"line_number":11,"context_line":"  options:"},{"line_number":12,"context_line":"  - close-pull"},{"line_number":13,"context_line":"  description: \"Foo\""},{"line_number":14,"context_line":"  remote: https://gerrit.googlesource.com/gerrit"},{"line_number":15,"context_line":"- project: openstack-ci/gerrit-verification-status-plugin"},{"line_number":16,"context_line":"  options:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"AAAALn%2F%2Fw%2Bo%3D","line":13,"updated":"2012-11-07 22:55:02.000000000","message":"Maybe \"Gerrit for OpenStack\" like is currently set at GitHub? We should probably come up with a consistent set of initial descriptions for all these projects anyway. Looks like we have initial candidates for most of these in GitHub already in fact, if we want to just splat them in here.","commit_id":"08b0fe0918132805b9fb91bb7f13d482ee5653ec"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"- project: heat-api/heat"},{"line_number":2,"context_line":"  options:"},{"line_number":3,"context_line":"  - has-wiki"},{"line_number":4,"context_line":"  description: \"Heat\" keeps the clouds up ;)"},{"line_number":5,"context_line":"- project: heat-api/python-heatclient"},{"line_number":6,"context_line":"- project: openstack-ci/devstack-gate"},{"line_number":7,"context_line":"- project: openstack-ci/gerrit"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"AAAALn%2F%2Fv1U%3D","line":4,"updated":"2012-11-08 17:57:48.000000000","message":"invalid yaml:\n\n\"\\\\\"Heat\\\\\" keeps the clouds up ;)\"\n\nor just:\n\nHeat keeps the clouds up ;)","commit_id":"7556133d432330df9958de3fbc2d36cd0f36ad01"}],"modules/openstack_project/manifests/gerrit.pp":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":228,"context_line":"    }"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    exec { \u0027manage_projects\u0027:"},{"line_number":231,"context_line":"      user        \u003d\u003e \u0027gerrit2\u0027,"},{"line_number":232,"context_line":"      command     \u003d\u003e \"/usr/local/gerrit/scripts/manage_projects.py \\"},{"line_number":233,"context_line":"        ${local_git_dir} ${ssh_host_key}\","},{"line_number":234,"context_line":"      subscribe   \u003d\u003e File[\u0027/home/gerrit2/projects.yaml\u0027],"}],"source_content_type":"text/x-puppet","patch_set":5,"id":"AAAALn%2F%2Fv2A%3D","line":231,"updated":"2012-11-08 17:52:12.000000000","message":"Its worth noting that using puppet to exec something running other than root can be slightly problematic as almost no environment info is pulled in.\n\nIt does look like the manage_projects scripts is chowning things to gerrit2:gerrit2 almost as if it assumes it will be run as root.","commit_id":"7556133d432330df9958de3fbc2d36cd0f36ad01"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":212,"context_line":""},{"line_number":213,"context_line":"  if ($projects_file !\u003d \u0027UNDEF\u0027) {"},{"line_number":214,"context_line":"    if ($replicate_local) {"},{"line_number":215,"context_line":"      file { $local_git_dir:"},{"line_number":216,"context_line":"        ensure \u003d\u003e directory,"},{"line_number":217,"context_line":"        owner  \u003d\u003e \u0027gerrit2\u0027,"},{"line_number":218,"context_line":"      }"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"AAAALn%2F%2FvXM%3D","line":215,"updated":"2012-11-09 00:10:17.000000000","message":"Each of these new resources should require \u003d\u003e Class[\u0027::gerrit\u0027] (as appropriate) to make sure they are only created after the base Gerrit stuff is done.","commit_id":"8f13871dff54d8470762eef30891e9758253244f"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":78,"context_line":"    ssh_rsa_key_contents     \u003d\u003e $ssh_rsa_key_contents,"},{"line_number":79,"context_line":"    ssh_rsa_pubkey_contents  \u003d\u003e $ssh_rsa_pubkey_contents,"},{"line_number":80,"context_line":"    ssh_project_rsa_key_contents     \u003d\u003e $ssh_project_rsa_key_contents,"},{"line_number":81,"context_line":"    ssh_project_rsa_pubkey_contents  \u003d\u003e $ssh_project_rsa_pubkey_contents,"},{"line_number":82,"context_line":"    email                    \u003d\u003e $email,"},{"line_number":83,"context_line":"    openidssourl             \u003d\u003e \"https://login.launchpad.net/+openid\","},{"line_number":84,"context_line":"    database_poollimit       \u003d\u003e $database_poollimit,"}],"source_content_type":"text/x-puppet","patch_set":14,"id":"AAAAMH%2F%2F%2FW4%3D","line":81,"updated":"2012-11-12 18:13:54.000000000","message":"Rocketships need aligning.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"}],"modules/openstack_project/templates/review.projects.yaml.erb":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"- homepage: http://openstack.org"},{"line_number":2,"context_line":"  local-git-dir: \u003c%\u003d local_git_dir %\u003e"},{"line_number":3,"context_line":"  gerrit-user: openstack-project-creator"},{"line_number":4,"context_line":"  gerrit-key: \u003c%\u003d ssh_project_key %\u003e"},{"line_number":5,"context_line":"  github-config: /etc/github/github-projects.secure.config"}],"source_content_type":"application/x-erb","patch_set":11,"id":"AAAAL3%2F%2F%2F%2F8%3D","line":2,"updated":"2012-11-11 18:25:50.000000000","message":"I see some scoping issues with these variables.  From what I see $local_git_dir is setup in manifests/gerrit.pp.  However we are adding this file in pypimirror/manifests/init.pp\n\nYou\u0027ll need to change it to\n\n\u003c%\u003d scope.lookupvar(\u0027openstack_project::gerrit::local_git_dir\u0027) %\u003e\n\nThen you\u0027ll need to have pypimirror require openstack_project::gerrit since we are accessing its\u0027 variables.","commit_id":"14fbcd3a5e0addf8cd9eb6f158ddcb7c5880a2d9"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"- homepage: http://openstack.org"},{"line_number":2,"context_line":"  local-git-dir: \u003c%\u003d local_git_dir %\u003e"},{"line_number":3,"context_line":"  gerrit-user: openstack-project-creator"},{"line_number":4,"context_line":"  gerrit-key: \u003c%\u003d ssh_project_key %\u003e"},{"line_number":5,"context_line":"  github-config: /etc/github/github-projects.secure.config"},{"line_number":6,"context_line":"  has-wiki: False"},{"line_number":7,"context_line":"  has-issues: False"}],"source_content_type":"application/x-erb","patch_set":11,"id":"AAAAL3%2F%2F%2F%2F4%3D","line":4,"updated":"2012-11-11 18:25:50.000000000","message":"Same comment","commit_id":"14fbcd3a5e0addf8cd9eb6f158ddcb7c5880a2d9"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":51,"context_line":"- project: openstack/openstack-chef"},{"line_number":52,"context_line":"- project: openstack/openstack-ci"},{"line_number":53,"context_line":"- project: openstack/openstack-ci-puppet"},{"line_number":54,"context_line":"  acl_config: /home/gerrit2/acls/openstack-ci-puppet.config"},{"line_number":55,"context_line":"- project: openstack/openstack-manuals"},{"line_number":56,"context_line":"- project: openstack/openstack-planet"},{"line_number":57,"context_line":"- project: openstack/openstack-puppet"}],"source_content_type":"application/x-erb","patch_set":14,"id":"AAAAMH%2F%2F%2FWc%3D","line":54,"updated":"2012-11-12 18:13:54.000000000","message":"Any concern with having this change potentially modify the ACLs for the puppet stuff? I guess you tested this with review-dev so shouldn\u0027t be an issue. Just pointing it out.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"unresolved":false,"context_lines":[{"line_number":51,"context_line":"- project: openstack/openstack-chef"},{"line_number":52,"context_line":"- project: openstack/openstack-ci"},{"line_number":53,"context_line":"- project: openstack/openstack-ci-puppet"},{"line_number":54,"context_line":"  acl_config: /home/gerrit2/acls/openstack-ci-puppet.config"},{"line_number":55,"context_line":"- project: openstack/openstack-manuals"},{"line_number":56,"context_line":"- project: openstack/openstack-planet"},{"line_number":57,"context_line":"- project: openstack/openstack-puppet"}],"source_content_type":"application/x-erb","patch_set":14,"id":"AAAAMH%2F%2F%2B4U%3D","line":54,"in_reply_to":"AAAAMH%2F%2F%2FWc%3D","updated":"2012-11-12 23:46:17.000000000","message":"Apparently not. We can unfsck ourselves manually if need be.","commit_id":"fb07ff235e9d3bad2b7573d70a7af13b63af69ee"}],"modules/pypimirror/manifests/init.pp":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"unresolved":false,"context_lines":[{"line_number":47,"context_line":"    owner   \u003d\u003e \u0027root\u0027,"},{"line_number":48,"context_line":"    group   \u003d\u003e \u0027root\u0027,"},{"line_number":49,"context_line":"    mode    \u003d\u003e \u00270444\u0027,"},{"line_number":50,"context_line":"    content \u003d\u003e template(\u0027openstack_project/review.projects.yaml.erb\u0027),"},{"line_number":51,"context_line":"    replace \u003d\u003e true,"},{"line_number":52,"context_line":"  }"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":11,"id":"AAAAL3%2F%2F%2F%2F0%3D","line":50,"updated":"2012-11-11 18:25:50.000000000","message":"Made a few comments in the template file, however I think this file could more to openstack_project::gerrit, since that is where the variables are coming from.","commit_id":"14fbcd3a5e0addf8cd9eb6f158ddcb7c5880a2d9"}]}