)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"ab5b6790699fddc2682e0db84ccfd58e3c6b2697","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jean-Philippe Evrard \u003cjean-philippe.evrard@rackspace.co.uk\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-08-25 23:53:13 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Connectivity testing"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This adds a playbook, oriented for users, to automatically test"},{"line_number":10,"context_line":"their installation after the setup infrastructure test."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"7f287b81_207d19e7","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":20},"updated":"2017-09-04 09:31:50.000000000","message":"The subject is misleading - perhaps best to change it to match the playbook?","commit_id":"0ca48bcb01a6b262175114b7e51d3afcaad0df5f"}],"playbooks/healthcheck-infrastructure.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"ab5b6790699fddc2682e0db84ccfd58e3c6b2697","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        - haproxy"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# We are looking up from the first container."},{"line_number":37,"context_line":"- name: Ensure that all the repos have data"},{"line_number":38,"context_line":"  hosts: all_containers[0]"},{"line_number":39,"context_line":"  gather_facts: yes"},{"line_number":40,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"7f287b81_4094a5e8","line":37,"updated":"2017-09-04 09:31:50.000000000","message":"This isn\u0027t actually verifying whether all repo container have data as far as I can see. It only checks one. I don\u0027t think this file exists either?","commit_id":"0ca48bcb01a6b262175114b7e51d3afcaad0df5f"},{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"07419bca678ce360c0695ed53663a3085c60d915","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        - haproxy"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# We are looking up from the first container."},{"line_number":37,"context_line":"- name: Ensure that all the repos have data"},{"line_number":38,"context_line":"  hosts: all_containers[0]"},{"line_number":39,"context_line":"  gather_facts: yes"},{"line_number":40,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"7f287b81_6f7bf0f3","line":37,"in_reply_to":"7f287b81_4094a5e8","updated":"2017-09-05 15:57:14.000000000","message":"the first container checks that all the nodes in the group repo_all have data (as in the title, and see L46).\n\nShould be fine.","commit_id":"0ca48bcb01a6b262175114b7e51d3afcaad0df5f"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"ab5b6790699fddc2682e0db84ccfd58e3c6b2697","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    # Repo release path points to the internal LB vip"},{"line_number":58,"context_line":"    - name: Check the presence of upper constraints on your repos and check load balancing"},{"line_number":59,"context_line":"      uri:"},{"line_number":60,"context_line":"        url: \"{{ repo_release_path }}/requirements.txt\""},{"line_number":61,"context_line":"      tags:"},{"line_number":62,"context_line":"        - repo"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"7f287b81_20c2b9ea","line":60,"updated":"2017-09-04 09:31:50.000000000","message":"I think you\u0027re meaning to look for the file used for upper constraints, which is: {{ repo_release_path }}/requirements_absolute_requirements.txt","commit_id":"0ca48bcb01a6b262175114b7e51d3afcaad0df5f"}],"playbooks/testing-connectivity.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"b3bfa058714781c9b6be9fdf5349bd406844bcac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7f287b81_c013c35b","updated":"2017-08-25 11:02:37.000000000","message":"I think the name is a bit misleading - perhaps something along the lines of \u0027healthcheck-infra\u0027 ? Then we can do various playbooks to do a quick health check, and they can all be named \u0027healthcheck-...\u0027","commit_id":"b8a268b515479dab59a2ccdd6fdd6c8fdd06d7b3"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"b3bfa058714781c9b6be9fdf5349bd406844bcac","unresolved":false,"context_lines":[{"line_number":169,"context_line":"      tags:"},{"line_number":170,"context_line":"        - rabbitmq"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"# TODO(evrardjp): Specific checks: Etcd"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7f287b81_05155d4a","line":172,"updated":"2017-08-25 11:02:37.000000000","message":"Add something for haproxy too?","commit_id":"b8a268b515479dab59a2ccdd6fdd6c8fdd06d7b3"}]}