)]}'
{"playbooks/roles/setup_tempest_user/tasks/main.yaml":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"f9553f074ff7a3f27b7da16c278c988b4b13e835","unresolved":false,"context_lines":[{"line_number":2,"context_line":"- name: Create tempest group"},{"line_number":3,"context_line":"  group: name\u003dtempest state\u003dpresent"},{"line_number":4,"context_line":"  become: yes"},{"line_number":5,"context_line":"- name: Create tempest user"},{"line_number":6,"context_line":"  user: name\u003dtempest shell\u003d/bin/bash group\u003dtempest"},{"line_number":7,"context_line":"  become: yes"},{"line_number":8,"context_line":"- name: Copy 51_tempest_sh to /etc/sudoers.d"},{"line_number":9,"context_line":"  copy: src\u003d51_tempest_sh dest\u003d/etc/sudoers.d owner\u003droot group\u003droot mode\u003d0440"},{"line_number":10,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5a74a57a_e44a5654","line":7,"range":{"start_line":5,"start_character":0,"end_line":7,"end_character":13},"updated":"2016-11-28 20:28:51.000000000","message":"This is not really a nit but more of a question, is there a documented standard for writing ansible stuff ?\n\nFor example, this particular bit I would typically write like so:\n\n    - name: Create tempest user\n      user:\n        name: \"tempest\"\n        shell: \"/bin/bash\"\n        group: \"tempest\"\n      become: \"yes\"\n\nNot trying to say one way is better than the other but it\u0027s more about agreeing on the same way of writing things so that eventually it doesn\u0027t become inconsistent as it organically grows from different contributors.","commit_id":"eda447c19db5c424620ee6c6d3e042d6d5d83d5f"},{"author":{"_account_id":1106,"name":"Ricardo Carrillo Cruz","email":"ricardo.carrillo.cruz@gmail.com","username":"rcc"},"change_message_id":"dbdacdeb872029876bfb94f0dcdb4f8adff0af6f","unresolved":false,"context_lines":[{"line_number":2,"context_line":"- name: Create tempest group"},{"line_number":3,"context_line":"  group: name\u003dtempest state\u003dpresent"},{"line_number":4,"context_line":"  become: yes"},{"line_number":5,"context_line":"- name: Create tempest user"},{"line_number":6,"context_line":"  user: name\u003dtempest shell\u003d/bin/bash group\u003dtempest"},{"line_number":7,"context_line":"  become: yes"},{"line_number":8,"context_line":"- name: Copy 51_tempest_sh to /etc/sudoers.d"},{"line_number":9,"context_line":"  copy: src\u003d51_tempest_sh dest\u003d/etc/sudoers.d owner\u003droot group\u003droot mode\u003d0440"},{"line_number":10,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5a74a57a_5e5492c8","line":7,"range":{"start_line":5,"start_character":0,"end_line":7,"end_character":13},"in_reply_to":"5a74a57a_e44a5654","updated":"2016-11-29 14:19:56.000000000","message":"yep, aware of the dict notation.\n\nIMHO, these roles won\u0027t change much, but we can revisit if we keep adding more params to not have very long lines","commit_id":"eda447c19db5c424620ee6c6d3e042d6d5d83d5f"}]}