)]}'
{".zuul.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"afbc982f36e63927d42677ef075fbf13e38bf371","unresolved":false,"context_lines":[{"line_number":1031,"context_line":"      - ^playbooks/host_vars/review-dev\\d+.opendev.org.yaml"},{"line_number":1032,"context_line":"      - playbooks/zuul/templates/group_vars/review.yaml.j2"},{"line_number":1033,"context_line":"      - playbooks/roles/gerrit/"},{"line_number":1034,"context_line":"      - testinfra/test_gerrit.py"},{"line_number":1035,"context_line":""},{"line_number":1036,"context_line":"- job:"},{"line_number":1037,"context_line":"    name: infra-prod-playbook"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_1346a851","line":1034,"updated":"2019-10-29 22:34:08.000000000","message":"Should playbooks/service-review-dev.yaml be listed here too?","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c0411ebd2fc1e20b1420a2e71a35ef3a469a9924","unresolved":false,"context_lines":[{"line_number":1031,"context_line":"      - ^playbooks/host_vars/review-dev\\d+.opendev.org.yaml"},{"line_number":1032,"context_line":"      - playbooks/zuul/templates/group_vars/review.yaml.j2"},{"line_number":1033,"context_line":"      - playbooks/roles/gerrit/"},{"line_number":1034,"context_line":"      - testinfra/test_gerrit.py"},{"line_number":1035,"context_line":""},{"line_number":1036,"context_line":"- job:"},{"line_number":1037,"context_line":"    name: infra-prod-playbook"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_f332eca6","line":1034,"in_reply_to":"3fa7e38b_1346a851","updated":"2019-10-29 22:42:12.000000000","message":"Yeah.","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"}],"playbooks/roles/gerrit/files/gerrit-podman/docker-compose.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"afbc982f36e63927d42677ef075fbf13e38bf371","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"services:"},{"line_number":4,"context_line":"  gerrit:"},{"line_number":5,"context_line":"    image: docker.io/opendevorg/gerrit:2.13"},{"line_number":6,"context_line":"    network_mode: host"},{"line_number":7,"context_line":"    user: gerrit"},{"line_number":8,"context_line":"    volumes:"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_334de43a","line":5,"range":{"start_line":5,"start_character":11,"end_line":5,"end_character":43},"updated":"2019-10-29 22:34:08.000000000","message":"I don\u0027t think we want this to be hardcoded in the role so that we can run -dev and prod independent of each other to test the upgrade process.","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c0411ebd2fc1e20b1420a2e71a35ef3a469a9924","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"services:"},{"line_number":4,"context_line":"  gerrit:"},{"line_number":5,"context_line":"    image: docker.io/opendevorg/gerrit:2.13"},{"line_number":6,"context_line":"    network_mode: host"},{"line_number":7,"context_line":"    user: gerrit"},{"line_number":8,"context_line":"    volumes:"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_5321804c","line":5,"range":{"start_line":5,"start_character":11,"end_line":5,"end_character":43},"in_reply_to":"3fa7e38b_334de43a","updated":"2019-10-29 22:42:12.000000000","message":"That\u0027s an excellent point.","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"}],"playbooks/roles/gerrit/files/web_server.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"95ab4e207965a70aa5fe4a6c7004b809efe63498","unresolved":false,"context_lines":[{"line_number":19,"context_line":"as if it was a working remote server. It also proxies all the live"},{"line_number":20,"context_line":"date/*.json files into the local test server, so that the Ajax async"},{"line_number":21,"context_line":"loading works without hitting Cross Site Scripting violations."},{"line_number":22,"context_line":"\"\"\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import argparse"},{"line_number":25,"context_line":"import BaseHTTPServer"}],"source_content_type":"text/x-python","patch_set":48,"id":"3fa7e38b_85bbce73","line":22,"updated":"2019-10-18 14:27:33.000000000","message":"I still don\u0027t understand the purpose of this program or why it\u0027s here.","commit_id":"ca7046d52e8872913e014cde9de440a16c706519"}],"playbooks/roles/gerrit/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"afbc982f36e63927d42677ef075fbf13e38bf371","unresolved":false,"context_lines":[{"line_number":12,"context_line":"- name: Synchronize podman-compose directory"},{"line_number":13,"context_line":"  synchronize:"},{"line_number":14,"context_line":"    src: gerrit-podman/"},{"line_number":15,"context_line":"    dest: /etc/gerrit-podman/"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Create Gerrit Group"},{"line_number":18,"context_line":"  group:"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_933cd892","line":15,"updated":"2019-10-29 22:34:08.000000000","message":"Maybe we should start storing this container deployment data in a dir that isn\u0027t tool specific as we\u0027ve already changed once (or are planning to).","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c0411ebd2fc1e20b1420a2e71a35ef3a469a9924","unresolved":false,"context_lines":[{"line_number":12,"context_line":"- name: Synchronize podman-compose directory"},{"line_number":13,"context_line":"  synchronize:"},{"line_number":14,"context_line":"    src: gerrit-podman/"},{"line_number":15,"context_line":"    dest: /etc/gerrit-podman/"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Create Gerrit Group"},{"line_number":18,"context_line":"  group:"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_d3d25034","line":15,"in_reply_to":"3fa7e38b_933cd892","updated":"2019-10-29 22:42:12.000000000","message":"++","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"122198b1393c3fc9ea9886718406007dc037fd20","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  user:"},{"line_number":25,"context_line":"    name: \"{{ gerrit_user_name }}\""},{"line_number":26,"context_line":"    uid: \"{{ gerrit_id }}\""},{"line_number":27,"context_line":"    comment: Gerit User"},{"line_number":28,"context_line":"    shell: /bin/bash"},{"line_number":29,"context_line":"    home: \"{{ gerrit_home_dir }}\""},{"line_number":30,"context_line":"    group: \"{{ gerrit_user_name }}\""}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_a5d2e332","line":27,"range":{"start_line":27,"start_character":13,"end_line":27,"end_character":18},"updated":"2019-10-30 13:50:32.000000000","message":"nit: s/Gerit/Gerrit/","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"543bef00a79d22ffbba3654aedb61dff7594a03b","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  user:"},{"line_number":25,"context_line":"    name: \"{{ gerrit_user_name }}\""},{"line_number":26,"context_line":"    uid: \"{{ gerrit_id }}\""},{"line_number":27,"context_line":"    comment: Gerit User"},{"line_number":28,"context_line":"    shell: /bin/bash"},{"line_number":29,"context_line":"    home: \"{{ gerrit_home_dir }}\""},{"line_number":30,"context_line":"    group: \"{{ gerrit_user_name }}\""}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_56bcfcb9","line":27,"range":{"start_line":27,"start_character":13,"end_line":27,"end_character":18},"in_reply_to":"3fa7e38b_a5d2e332","updated":"2019-10-31 02:02:58.000000000","message":"hah","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"afbc982f36e63927d42677ef075fbf13e38bf371","unresolved":false,"context_lines":[{"line_number":80,"context_line":"    group: \"{{ gerrit_user_name }}\""},{"line_number":81,"context_line":"    mode: 0644"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"- name: Write Welcome SSH private key"},{"line_number":84,"context_line":"  copy:"},{"line_number":85,"context_line":"    content: \"{{ welcome_message_gerrit_ssh_private_key }}\""},{"line_number":86,"context_line":"    dest: \"{{ gerrit_site_dir }}/etc/ssh_welcome_rsa_key\""}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_33b16445","line":83,"updated":"2019-10-29 22:34:08.000000000","message":"What is a welcome ssh key?","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"3e9bc63003853b83e35c0d96e4f3853d1f25d508","unresolved":false,"context_lines":[{"line_number":80,"context_line":"    group: \"{{ gerrit_user_name }}\""},{"line_number":81,"context_line":"    mode: 0644"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"- name: Write Welcome SSH private key"},{"line_number":84,"context_line":"  copy:"},{"line_number":85,"context_line":"    content: \"{{ welcome_message_gerrit_ssh_private_key }}\""},{"line_number":86,"context_line":"    dest: \"{{ gerrit_site_dir }}/etc/ssh_welcome_rsa_key\""}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_13ec68f2","line":83,"updated":"2019-10-29 22:40:13.000000000","message":"first time contributor message.","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c0411ebd2fc1e20b1420a2e71a35ef3a469a9924","unresolved":false,"context_lines":[{"line_number":80,"context_line":"    group: \"{{ gerrit_user_name }}\""},{"line_number":81,"context_line":"    mode: 0644"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"- name: Write Welcome SSH private key"},{"line_number":84,"context_line":"  copy:"},{"line_number":85,"context_line":"    content: \"{{ welcome_message_gerrit_ssh_private_key }}\""},{"line_number":86,"context_line":"    dest: \"{{ gerrit_site_dir }}/etc/ssh_welcome_rsa_key\""}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_93d85852","line":83,"in_reply_to":"3fa7e38b_33b16445","updated":"2019-10-29 22:42:12.000000000","message":"It\u0027s the key we use to send the new user message.\n\nI don\u0027t really understand other than that.","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"afbc982f36e63927d42677ef075fbf13e38bf371","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    executable: pip3"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"# TODO(mordred) Make this suck less, like if we could do an init container"},{"line_number":151,"context_line":"# or something just generally less gross."},{"line_number":152,"context_line":"- name: Run gerrit init"},{"line_number":153,"context_line":"  when: gerrit_run_init | bool"},{"line_number":154,"context_line":"  command: \u003e"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_73703c73","line":151,"updated":"2019-10-29 22:34:08.000000000","message":"We should only need to run this if there was no gerrit there before or if the gerrit version updates. Maybe we can accomplish that via an init step of the container that checks for a local string and runs init on startup if current gerrit version !\u003d previous version or previous version does not exist?","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c0411ebd2fc1e20b1420a2e71a35ef3a469a9924","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    executable: pip3"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"# TODO(mordred) Make this suck less, like if we could do an init container"},{"line_number":151,"context_line":"# or something just generally less gross."},{"line_number":152,"context_line":"- name: Run gerrit init"},{"line_number":153,"context_line":"  when: gerrit_run_init | bool"},{"line_number":154,"context_line":"  command: \u003e"}],"source_content_type":"text/x-yaml","patch_set":69,"id":"3fa7e38b_53cf407e","line":151,"in_reply_to":"3fa7e38b_73703c73","updated":"2019-10-29 22:42:12.000000000","message":"Yeah - honestly, this is the wonkiest part of the whole thing. I\u0027m not crazy about auto-running, because I feel like it might make our upgrades harder - which makes me think maybe we need an upgrade playbook - and just need to have the job run upgrade then start or something? I dunno.","commit_id":"4f9720e76ed734b0e87df1242c8c09639d900425"}],"playbooks/roles/install-podman/tasks/main.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"0417a5cbac1c2963c31728a18d51c1c06dc114c4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- name: Add PPA GPG key"},{"line_number":2,"context_line":"  become: yes"},{"line_number":3,"context_line":"  apt_key:"},{"line_number":4,"context_line":"    data: \"{{ projectatomic_gpg_key }}\""}],"source_content_type":"text/x-yaml","patch_set":67,"id":"3fa7e38b_e6a010f6","line":1,"updated":"2019-10-28 01:10:49.000000000","message":"doesn\u0027t \n\n - apt_repository:\n     repo: projectatomic/ppa \n\nbasically do all this without having to use the template etc?","commit_id":"cb49ec570e9722c5d787dccd8a0c7da248bbd109"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"9335590ccf380c3f938134070f175f840039bb62","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- name: Add PPA GPG key"},{"line_number":2,"context_line":"  become: yes"},{"line_number":3,"context_line":"  apt_key:"},{"line_number":4,"context_line":"    data: \"{{ projectatomic_gpg_key }}\""}],"source_content_type":"text/x-yaml","patch_set":67,"id":"3fa7e38b_c647141d","line":1,"in_reply_to":"3fa7e38b_e6a010f6","updated":"2019-10-28 01:37:17.000000000","message":"I think so - but I think we\u0027ve been going for embedding gpg keys in the repos rather than downloading them?","commit_id":"cb49ec570e9722c5d787dccd8a0c7da248bbd109"}],"playbooks/service-review-container.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a119a6444ada35aa7a8751015d0208ffd388b3e3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: \"review-container01.opendev.org:!disabled\""},{"line_number":2,"context_line":"  name: \"Base: configure review.opendev.org\""},{"line_number":3,"context_line":"  roles:"},{"line_number":4,"context_line":"    - install-docker"},{"line_number":5,"context_line":"    - gerrit"}],"source_content_type":"text/x-yaml","patch_set":28,"id":"7faddb67_36ff4803","line":4,"updated":"2019-08-29 20:28:15.000000000","message":"The need to install docker-compose in the gerrit role makes me wonder if that should be a role too. Since we need to install it in quite a few places now.","commit_id":"ea31c6f9dcde2ac31a30c2add9502c6632c8e862"}]}