)]}'
{"playbooks/group_vars/nodepool-builder.yaml":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"880422dbb4f9ee22bb75e3e80df1f587f9bacbff","unresolved":false,"context_lines":[{"line_number":1,"context_line":"openstacksdk_config_owner: \"{{ nodepool_user }}\""},{"line_number":2,"context_line":"openstacksdk_config_group: \"{{ nodepool_group }}\""},{"line_number":3,"context_line":"openstacksdk_config_dir: \"~{{ openstacksdk_config_owner }}nodepool/.config/openstack\""},{"line_number":4,"context_line":"openstacksdk_config_template: clouds/nodepool_clouds.yaml.j2"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ff570b3c_e025b2c3","line":3,"range":{"start_line":3,"start_character":58,"end_line":3,"end_character":66},"updated":"2020-05-18 14:01:06.000000000","message":"I think \"nodepool\" should be dropped here.","commit_id":"e063d3014d54a85edea9dcac6ed5a2fc0b25d3ed"}],"playbooks/roles/nodepool-base/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3cdd7a737b207e8afa3bad40647ce0bbb01ff0ab","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    home: \u0027/home/{{ nodepool_user }}\u0027"},{"line_number":13,"context_line":"    create_home: yes"},{"line_number":14,"context_line":"    shell: /bin/bash"},{"line_number":15,"context_line":"    system: yes"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Sync project-config"},{"line_number":18,"context_line":"  include_role:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"ff570b3c_eeb0c045","line":15,"updated":"2020-05-20 23:34:33.000000000","message":"Its weird to me that we are indicating this is a system user but set the uid well outside of system user range and set a home dir. Is there a reason for this?","commit_id":"09935ff32823f44682c0350f1d89d6d3358174ad"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"fd594e927d1397fd5fab2f200f93c6029cdefdcc","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    home: \u0027/home/{{ nodepool_user }}\u0027"},{"line_number":13,"context_line":"    create_home: yes"},{"line_number":14,"context_line":"    shell: /bin/bash"},{"line_number":15,"context_line":"    system: yes"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Sync project-config"},{"line_number":18,"context_line":"  include_role:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"ff570b3c_f21ff3b3","line":15,"updated":"2020-05-21 17:45:35.000000000","message":"No, just making it consistent with the others.  I don\u0027t know that there was any reason for one or the other before.","commit_id":"09935ff32823f44682c0350f1d89d6d3358174ad"}],"playbooks/roles/zookeeper/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3cdd7a737b207e8afa3bad40647ce0bbb01ff0ab","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    home: \"/home/{{ zookeeper_user }}\""},{"line_number":12,"context_line":"    create_home: yes"},{"line_number":13,"context_line":"    shell: /bin/bash"},{"line_number":14,"context_line":"    system: yes"},{"line_number":15,"context_line":"- name: Synchronize compose directory"},{"line_number":16,"context_line":"  synchronize:"},{"line_number":17,"context_line":"    src: zookeeper-compose/"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"ff570b3c_0eb25449","line":14,"updated":"2020-05-20 23:34:33.000000000","message":"I guess we were already making other users system users so the nodepool change adds some consistency.","commit_id":"09935ff32823f44682c0350f1d89d6d3358174ad"}]}