)]}'
{"roles/setup-devstack-source-dirs/tasks/main.yaml":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c267996247cee957a5104394f7a5426e569400fd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- name: Create base directory"},{"line_number":2,"context_line":"  file:"},{"line_number":3,"context_line":"    path: `{{ devstack_base_dir }}`"},{"line_number":4,"context_line":"    state: directory"},{"line_number":5,"context_line":"  become: yes"},{"line_number":6,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f287b81_e13705b0","line":3,"updated":"2017-08-24 13:17:59.000000000","message":"single quote, not backtick","commit_id":"8efabcff757898ee7c7c1ef69dcb92d78682bae8"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c267996247cee957a5104394f7a5426e569400fd","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Create base/new directory"},{"line_number":8,"context_line":"  file:"},{"line_number":9,"context_line":"    path: `{{ devstack_base_dir }}/new`"},{"line_number":10,"context_line":"    state: directory"},{"line_number":11,"context_line":"  become: yes"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f287b81_614b152b","line":9,"updated":"2017-08-24 13:17:59.000000000","message":"single quote, not backtick","commit_id":"8efabcff757898ee7c7c1ef69dcb92d78682bae8"},{"author":{"_account_id":6589,"name":"Jesse Keating","email":"jkeating@j2solutions.net","username":"jesse-keating"},"change_message_id":"9295c3ec5b3932a3d64521e6f3d62da395f19b87","unresolved":false,"context_lines":[{"line_number":7,"context_line":"- name: Create base/new directory"},{"line_number":8,"context_line":"  file:"},{"line_number":9,"context_line":"    path: \u0027{{ devstack_base_dir }}/new\u0027"},{"line_number":10,"context_line":"    state: directory"},{"line_number":11,"context_line":"  become: yes"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: Find all source repos used by this job"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_5bcc7ebb","line":10,"updated":"2017-08-24 18:22:24.000000000","message":"Is there a reason why these two tasks are merged into one? If you specify a state of directory, and preceeding directories do not exist, Ansible will make them for you. It\u0027s like mkdir -p.","commit_id":"e181b67bb30f754af7faa5fe520123a0130e3b90"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"7c07c521c26bd336d75c0733f2ae521d41019887","unresolved":false,"context_lines":[{"line_number":7,"context_line":"- name: Create base/new directory"},{"line_number":8,"context_line":"  file:"},{"line_number":9,"context_line":"    path: \u0027{{ devstack_base_dir }}/new\u0027"},{"line_number":10,"context_line":"    state: directory"},{"line_number":11,"context_line":"  become: yes"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- name: Find all source repos used by this job"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_18d2b135","line":10,"updated":"2017-09-01 20:52:26.000000000","message":"Neat!","commit_id":"e181b67bb30f754af7faa5fe520123a0130e3b90"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"7c07c521c26bd336d75c0733f2ae521d41019887","unresolved":false,"context_lines":[{"line_number":20,"context_line":"  register: found_repos"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"- name: Move Zuul repos into place"},{"line_number":23,"context_line":"  command: mv {{ item.path }} {{ devstack_base_dir }}/new"},{"line_number":24,"context_line":"  with_items: \u0027{{ found_repos.files }}\u0027"},{"line_number":25,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_f8d6c527","line":23,"updated":"2017-09-01 20:52:26.000000000","message":"Gotcha.","commit_id":"e181b67bb30f754af7faa5fe520123a0130e3b90"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"cbec3f0b9dada51d6cb3f69335cce820724d9875","unresolved":false,"context_lines":[{"line_number":20,"context_line":"  register: found_repos"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"- name: Move Zuul repos into place"},{"line_number":23,"context_line":"  command: mv {{ item.path }} {{ devstack_base_dir }}/new"},{"line_number":24,"context_line":"  with_items: \u0027{{ found_repos.files }}\u0027"},{"line_number":25,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_deef151c","line":23,"updated":"2017-08-25 22:48:56.000000000","message":"I think we should copy or link rather than move if we can get away with it. Reason being that if we move then the new structure doesn\u0027t exist anymore, meaning it\u0027s harder for people to write new content that looks for things that way and perpetuates the old structure. If we copy or link, then the old structure (What devstack currently needs) can be satisfied, but new structure is there too, so people wanting to take advantage of it (including maybe devstack) can.\n\n(Thinking of things like devstack jobs for hummingbird, or integration tests that include terraform and gophercloud)","commit_id":"e181b67bb30f754af7faa5fe520123a0130e3b90"}]}