)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"d85fc74df098735854e21cce78e286a0c2477185","unresolved":false,"context_lines":[{"line_number":14,"context_line":"  * note the data and datalog split; we\u0027ll have to move individual"},{"line_number":15,"context_line":"    files to separate them out"},{"line_number":16,"context_line":"* remove system packages"},{"line_number":17,"context_line":"* start zk containers"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Alternatives:"},{"line_number":20,"context_line":"* bind mount in the current location (/var/lib/zookeeper), keeping the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f4c43b2_266ee594","line":17,"updated":"2020-04-16 16:04:38.000000000","message":"Revised plan:\n* add zk* to emergency\n* copy data files on each node to a safe place for DR backup\n* make a json data backup: zk-shell localhost:2181 --run-once \u0027mirror / json://!tmp!zookeeper-backup.json/\u0027\n* manually run a modified playbook to set up the docker infra without starting containers\n* rolling restart; for each node:\n  * stop zk\n  * split data and log files and move them to new locations\n  * remove zk packages\n  * start zk containers\n* remove from emergency; land this change.","commit_id":"de7cc34c69aa2db7a7a53c7ccb367eccda25205f"}],"playbooks/roles/zookeeper/tasks/main.yaml":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"ef49144e306f559c5e5e2cd6a16c897d96955b41","unresolved":false,"context_lines":[{"line_number":19,"context_line":"  template:"},{"line_number":20,"context_line":"    src: myid.j2"},{"line_number":21,"context_line":"    dest: /var/zookeeper/data/myid"},{"line_number":22,"context_line":"- name: Install docker-compose"},{"line_number":23,"context_line":"  package:"},{"line_number":24,"context_line":"    name:"},{"line_number":25,"context_line":"      - docker-compose"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f4c43b2_b7785d2b","line":22,"updated":"2020-04-16 18:47:06.000000000","message":"This is going to conflict with clarkb\u0027s patch.","commit_id":"db32aab9c34c964918ac28cb2836787d2e540b95"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"9582b86ef6617e4c280502ab8db258174a834ca0","unresolved":false,"context_lines":[{"line_number":19,"context_line":"  template:"},{"line_number":20,"context_line":"    src: myid.j2"},{"line_number":21,"context_line":"    dest: /var/zookeeper/data/myid"},{"line_number":22,"context_line":"- name: Install docker-compose"},{"line_number":23,"context_line":"  package:"},{"line_number":24,"context_line":"    name:"},{"line_number":25,"context_line":"      - docker-compose"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f4c43b2_17c67149","line":22,"in_reply_to":"3f4c43b2_b7785d2b","updated":"2020-04-16 18:51:42.000000000","message":"Could replace with:\n\n  - name: Install docker-compose\n    pip:\n      name: docker-compose\n      state: present\n      executable: pip3\n\nwhich should not be a conflict with the clarkb patch, then we can drop it from here later.","commit_id":"db32aab9c34c964918ac28cb2836787d2e540b95"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"a5124d617cf121134b17c59c16ac099667c537c5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- name: Create Zookeeper goup"},{"line_number":2,"context_line":"  group:"},{"line_number":3,"context_line":"    name: \"zookeeper\""},{"line_number":4,"context_line":"    system: yes"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4c43b2_b2834ba9","line":1,"updated":"2020-04-16 20:41:04.000000000","message":"Nit: gRoup","commit_id":"5fc7372834708f86c7071dcb850df0a5f0d1182d"}],"playbooks/roles/zookeeper/templates/zoo.cfg.j2":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"46d62b44c60dcef67b4ec60e861873eb721490c0","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# snapRetainCount most recent snapshots and the corresponding"},{"line_number":13,"context_line":"# transaction logs in the dataDir and dataLogDir respectively and"},{"line_number":14,"context_line":"# deletes the rest.  Defaults to 3. Minimum value is 3."},{"line_number":15,"context_line":"autopurge.snapRetainCount\u003d3"},{"line_number":16,"context_line":"# The frequency in hours to look for and purge old snapshots,"},{"line_number":17,"context_line":"# defaults to 0 (disabled). The number of retained snapshots can"},{"line_number":18,"context_line":"# be separately controlled through snapRetainCount and"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"3f4c43b2_b76e9d93","line":15,"updated":"2020-04-16 18:40:20.000000000","message":"That\u0027s the default.","commit_id":"f92ca5ef8fb0763a28d94e1e0befcd2845ab1201"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"ca86ac91c2933c5b53a5a2180c27b37a536771f7","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# snapRetainCount most recent snapshots and the corresponding"},{"line_number":13,"context_line":"# transaction logs in the dataDir and dataLogDir respectively and"},{"line_number":14,"context_line":"# deletes the rest.  Defaults to 3. Minimum value is 3."},{"line_number":15,"context_line":"autopurge.snapRetainCount\u003d3"},{"line_number":16,"context_line":"# The frequency in hours to look for and purge old snapshots,"},{"line_number":17,"context_line":"# defaults to 0 (disabled). The number of retained snapshots can"},{"line_number":18,"context_line":"# be separately controlled through snapRetainCount and"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"3f4c43b2_d74a4925","line":15,"updated":"2020-04-16 18:39:17.000000000","message":"We also set snapCount to 10000 on puppet managed zk.","commit_id":"f92ca5ef8fb0763a28d94e1e0befcd2845ab1201"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"cf946c56382aebfc211f241f89fe4dab368da057","unresolved":false,"context_lines":[{"line_number":24,"context_line":"admin.enableServer\u003dtrue"},{"line_number":25,"context_line":"clientPort\u003d2181"},{"line_number":26,"context_line":"{% for host in groups[\u0027zookeeper\u0027] %}"},{"line_number":27,"context_line":"server.{{ loop.index }}\u003d{{ (host \u003d\u003d inventory_hostname and \u00270.0.0.0\u0027 or host) }}:2888:3888"},{"line_number":28,"context_line":"{% endfor %}"}],"source_content_type":"text/x-jinja2","patch_set":10,"id":"3f4c43b2_78c6ba9c","line":27,"range":{"start_line":27,"start_character":36,"end_line":27,"end_character":54},"updated":"2020-04-17 12:25:22.000000000","message":"I think you can use hostvars[host].ansible_default_ipv4.address to get the ip address","commit_id":"a71b1955a664f00ebf2819ada483f0dc3600825c"}],"playbooks/service-zookeeper.yaml":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"46d62b44c60dcef67b4ec60e861873eb721490c0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: \"zookeeper:!disabled\""},{"line_number":2,"context_line":"  name: \"Configure Zookeeper\""},{"line_number":3,"context_line":"  roles:"},{"line_number":4,"context_line":"    - install-docker"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f4c43b2_176e7194","line":1,"updated":"2020-04-16 18:40:20.000000000","message":"Ayup.","commit_id":"f92ca5ef8fb0763a28d94e1e0befcd2845ab1201"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"ca86ac91c2933c5b53a5a2180c27b37a536771f7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: \"zookeeper:!disabled\""},{"line_number":2,"context_line":"  name: \"Configure Zookeeper\""},{"line_number":3,"context_line":"  roles:"},{"line_number":4,"context_line":"    - install-docker"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f4c43b2_f74f8531","line":1,"updated":"2020-04-16 18:39:17.000000000","message":"Should we serialize this play like we do with gitea to prevent all zk\u0027s from being restarted at once? We use zk:latest image so we\u0027ll be doing restarts occasionally.","commit_id":"f92ca5ef8fb0763a28d94e1e0befcd2845ab1201"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"56df262317f4f8b90856a36126ad796aa2ad908f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: \"zookeeper:!disabled\""},{"line_number":2,"context_line":"  name: \"Configure Zookeeper\""},{"line_number":3,"context_line":"  roles:"},{"line_number":4,"context_line":"    - install-docker"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f4c43b2_3708cdd7","line":1,"in_reply_to":"3f4c43b2_f74f8531","updated":"2020-04-16 18:40:36.000000000","message":"That sounds like a good idea.","commit_id":"f92ca5ef8fb0763a28d94e1e0befcd2845ab1201"}]}