)]}'
{"playbooks/roles/lodgeit/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"219e56b0a3296d9a0d1464f6a1cf9ff784af5de3","unresolved":true,"context_lines":[{"line_number":43,"context_line":"    - proxy_http"},{"line_number":44,"context_line":"    - ssl"},{"line_number":45,"context_line":"    - headers"},{"line_number":46,"context_line":"    - proxy_wstunnel"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"- name: Copy apache config"},{"line_number":49,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"cf2213df_91c9a5e5","line":46,"updated":"2021-07-06 21:52:28.000000000","message":"Does lodgeit use websockets?","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8c7e76375242c0944f8d8b74fc089ed97c978b56","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    - proxy_http"},{"line_number":44,"context_line":"    - ssl"},{"line_number":45,"context_line":"    - headers"},{"line_number":46,"context_line":"    - proxy_wstunnel"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"- name: Copy apache config"},{"line_number":49,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"4068e413_4312789d","line":46,"in_reply_to":"cf2213df_91c9a5e5","updated":"2021-07-06 22:26:38.000000000","message":"no, that\u0027s a copy-paste, will remove","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"}],"playbooks/roles/lodgeit/templates/docker-compose.yaml.j2":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"219e56b0a3296d9a0d1464f6a1cf9ff784af5de3","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    depends_on:"},{"line_number":25,"context_line":"      - mariadb"},{"line_number":26,"context_line":"    network_mode: host"},{"line_number":27,"context_line":"    command: [\u0027/bin/bash\u0027, \u0027-c\u0027, \u0027echo \"*** Starting\"; sleep 30; /usr/local/bin/uwsgi\u0027]"},{"line_number":28,"context_line":"    logging:"},{"line_number":29,"context_line":"      driver: syslog"},{"line_number":30,"context_line":"      options:"}],"source_content_type":"text/x-jinja2","patch_set":22,"id":"9de18744_3d6857e8","line":27,"range":{"start_line":27,"start_character":55,"end_line":27,"end_character":63},"updated":"2021-07-06 21:52:28.000000000","message":"I guess this ensures the db is up first?","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8c7e76375242c0944f8d8b74fc089ed97c978b56","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    depends_on:"},{"line_number":25,"context_line":"      - mariadb"},{"line_number":26,"context_line":"    network_mode: host"},{"line_number":27,"context_line":"    command: [\u0027/bin/bash\u0027, \u0027-c\u0027, \u0027echo \"*** Starting\"; sleep 30; /usr/local/bin/uwsgi\u0027]"},{"line_number":28,"context_line":"    logging:"},{"line_number":29,"context_line":"      driver: syslog"},{"line_number":30,"context_line":"      options:"}],"source_content_type":"text/x-jinja2","patch_set":22,"id":"b8838beb_f5ea7228","line":27,"range":{"start_line":27,"start_character":55,"end_line":27,"end_character":63},"in_reply_to":"9de18744_3d6857e8","updated":"2021-07-06 22:26:38.000000000","message":"that\u0027s right.  i\u0027ve considered putting the wait-for-it.sh script into the uwsgi container, might be something worth doing if we have more instances of this sort of thing.","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"}],"testinfra/test_paste.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"219e56b0a3296d9a0d1464f6a1cf9ff784af5de3","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    assert graphite_http.is_listening"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    graphite_https \u003d host.socket(\"tcp://127.0.0.1:443\")"},{"line_number":28,"context_line":"    assert graphite_https.is_listening"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"def test_paste(host):"},{"line_number":31,"context_line":"    cmd \u003d host.run(\u0027curl --insecure \u0027"}],"source_content_type":"text/x-python","patch_set":22,"id":"78b7d472_76f61475","line":28,"updated":"2021-07-06 21:52:28.000000000","message":"The variable names in this function indicate copy pasta that wasn\u0027t fully updated :) Can we s/graphite/paste/ ?","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8c7e76375242c0944f8d8b74fc089ed97c978b56","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    assert graphite_http.is_listening"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    graphite_https \u003d host.socket(\"tcp://127.0.0.1:443\")"},{"line_number":28,"context_line":"    assert graphite_https.is_listening"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"def test_paste(host):"},{"line_number":31,"context_line":"    cmd \u003d host.run(\u0027curl --insecure \u0027"}],"source_content_type":"text/x-python","patch_set":22,"id":"8e102f2b_f45990bf","line":28,"in_reply_to":"78b7d472_76f61475","updated":"2021-07-06 22:26:38.000000000","message":"Done","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"}],"zuul.d/project.yaml":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"a2115b8b1bbfc883eacbac9dabd30edd246f4a12","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        - system-config-run-paste:"},{"line_number":41,"context_line":"            dependencies:"},{"line_number":42,"context_line":"              - name: opendev-buildset-registry"},{"line_number":43,"context_line":"              - name: lodgeit-build-opendev-image"},{"line_number":44,"context_line":"                soft: true"},{"line_number":45,"context_line":"        - system-config-run-static"},{"line_number":46,"context_line":"        - system-config-run-docker-registry"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"ad9e27aa_697eefaf","line":43,"updated":"2021-06-30 00:12:58.000000000","message":"Remove both references of lodgeit-build-opendev-image since that\u0027s built in a different repo.","commit_id":"af5f5b24abcbf816b9558bd39e4f5e47dbfb5153"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"219e56b0a3296d9a0d1464f6a1cf9ff784af5de3","unresolved":true,"context_lines":[{"line_number":38,"context_line":"        - system-config-run-mirror-update"},{"line_number":39,"context_line":"        - system-config-run-paste:"},{"line_number":40,"context_line":"            dependencies:"},{"line_number":41,"context_line":"              - name: opendev-buildset-registry"},{"line_number":42,"context_line":"        - system-config-run-static"},{"line_number":43,"context_line":"        - system-config-run-docker-registry"},{"line_number":44,"context_line":"        - system-config-run-etherpad:"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"23c97ebc_63a4b747","line":41,"updated":"2021-07-06 21:52:28.000000000","message":"Why does this one have a dependency on the buildset-registry but the gate job does not? I think we want it in both places?","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8c7e76375242c0944f8d8b74fc089ed97c978b56","unresolved":true,"context_lines":[{"line_number":38,"context_line":"        - system-config-run-mirror-update"},{"line_number":39,"context_line":"        - system-config-run-paste:"},{"line_number":40,"context_line":"            dependencies:"},{"line_number":41,"context_line":"              - name: opendev-buildset-registry"},{"line_number":42,"context_line":"        - system-config-run-static"},{"line_number":43,"context_line":"        - system-config-run-docker-registry"},{"line_number":44,"context_line":"        - system-config-run-etherpad:"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"e4152e5b_f61f1976","line":41,"in_reply_to":"23c97ebc_63a4b747","updated":"2021-07-06 22:26:38.000000000","message":"this was a choice because at gate time we want to be pulling the released lodgeit image.  if it depends-on: a lodgeit change, that should have merged (and promoted) before we get to gate","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9d416447c1e9558d7a810f60804a7e8f326a0a71","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        - system-config-run-mirror-update"},{"line_number":39,"context_line":"        - system-config-run-paste:"},{"line_number":40,"context_line":"            dependencies:"},{"line_number":41,"context_line":"              - name: opendev-buildset-registry"},{"line_number":42,"context_line":"        - system-config-run-static"},{"line_number":43,"context_line":"        - system-config-run-docker-registry"},{"line_number":44,"context_line":"        - system-config-run-etherpad:"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"733414e5_289a6e6a","line":41,"in_reply_to":"e4152e5b_f61f1976","updated":"2021-07-06 22:29:13.000000000","message":"looks like consensus is to use intermeidate registry in gate, so i\u0027ve updated that","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"}],"zuul.d/system-config-run.yaml":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"a2115b8b1bbfc883eacbac9dabd30edd246f4a12","unresolved":false,"context_lines":[{"line_number":695,"context_line":"    timeout: 3600"},{"line_number":696,"context_line":"    required-projects:"},{"line_number":697,"context_line":"      - opendev/system-config"},{"line_number":698,"context_line":"      - opendev/lodgeit"},{"line_number":699,"context_line":"    requires:"},{"line_number":700,"context_line":"      - lodgeit-container-image"},{"line_number":701,"context_line":"    nodeset:"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"923bbf54_bb4dcf70","line":698,"updated":"2021-06-30 00:12:58.000000000","message":"Also; do we actually use the lodgeit repo to run it?  I think we could drop required-projects here.","commit_id":"af5f5b24abcbf816b9558bd39e4f5e47dbfb5153"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1d7e2ecf9aa79c5be27c6ea411de74743838148d","unresolved":false,"context_lines":[{"line_number":695,"context_line":"    timeout: 3600"},{"line_number":696,"context_line":"    required-projects:"},{"line_number":697,"context_line":"      - opendev/system-config"},{"line_number":698,"context_line":"      - opendev/lodgeit"},{"line_number":699,"context_line":"    requires:"},{"line_number":700,"context_line":"      - lodgeit-container-image"},{"line_number":701,"context_line":"    nodeset:"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"cd07ef5a_204e99b5","line":698,"in_reply_to":"923bbf54_bb4dcf70","updated":"2021-06-30 00:16:27.000000000","message":"no we only run out of the container, so we don\u0027t need it on disk","commit_id":"af5f5b24abcbf816b9558bd39e4f5e47dbfb5153"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"a2115b8b1bbfc883eacbac9dabd30edd246f4a12","unresolved":false,"context_lines":[{"line_number":708,"context_line":"      run_playbooks:"},{"line_number":709,"context_line":"        - playbooks/letsencrypt.yaml"},{"line_number":710,"context_line":"        - playbooks/service-paste.yaml"},{"line_number":711,"context_line":"      run_test_playbook: playbooks/test-grafana.yaml"},{"line_number":712,"context_line":"    files:"},{"line_number":713,"context_line":"      - playbooks/bridge.yaml"},{"line_number":714,"context_line":"      - playbooks/letsencrypt.yaml"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"a0e8820d_847ae827","line":711,"updated":"2021-06-30 00:12:58.000000000","message":"grafana?","commit_id":"af5f5b24abcbf816b9558bd39e4f5e47dbfb5153"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1d7e2ecf9aa79c5be27c6ea411de74743838148d","unresolved":false,"context_lines":[{"line_number":708,"context_line":"      run_playbooks:"},{"line_number":709,"context_line":"        - playbooks/letsencrypt.yaml"},{"line_number":710,"context_line":"        - playbooks/service-paste.yaml"},{"line_number":711,"context_line":"      run_test_playbook: playbooks/test-grafana.yaml"},{"line_number":712,"context_line":"    files:"},{"line_number":713,"context_line":"      - playbooks/bridge.yaml"},{"line_number":714,"context_line":"      - playbooks/letsencrypt.yaml"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"e295ed7d_a27195da","line":711,"in_reply_to":"a0e8820d_847ae827","updated":"2021-06-30 00:16:27.000000000","message":"copy paste :)","commit_id":"af5f5b24abcbf816b9558bd39e4f5e47dbfb5153"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"219e56b0a3296d9a0d1464f6a1cf9ff784af5de3","unresolved":true,"context_lines":[{"line_number":715,"context_line":"      - playbooks/roles/lodgeit"},{"line_number":716,"context_line":"      - playbooks/roles/install-docker/"},{"line_number":717,"context_line":"      - playbooks/roles/pip3/"},{"line_number":718,"context_line":"      - testinfra/test_paste.py"},{"line_number":719,"context_line":""},{"line_number":720,"context_line":"- job:"},{"line_number":721,"context_line":"    name: system-config-run-zookeeper"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"af4db759_be9903b1","line":718,"updated":"2021-07-06 21:52:28.000000000","message":"You should probably add the test-paste.yaml playbook here as well as the test hostvars file. Also the lodgeit role.","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8c7e76375242c0944f8d8b74fc089ed97c978b56","unresolved":true,"context_lines":[{"line_number":715,"context_line":"      - playbooks/roles/lodgeit"},{"line_number":716,"context_line":"      - playbooks/roles/install-docker/"},{"line_number":717,"context_line":"      - playbooks/roles/pip3/"},{"line_number":718,"context_line":"      - testinfra/test_paste.py"},{"line_number":719,"context_line":""},{"line_number":720,"context_line":"- job:"},{"line_number":721,"context_line":"    name: system-config-run-zookeeper"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"1ee5468d_9dffde89","line":718,"in_reply_to":"af4db759_be9903b1","updated":"2021-07-06 22:26:38.000000000","message":"Done - the lodgeit role is already covered i think","commit_id":"bcabaf5c4cb567d0e43a6eb3d53026e668c35396"}]}