)]}'
{"tasks/lxc_container_create.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"64624fcb33a5be69f6aace762ee47a7c8d51780a","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    mode: \"0755\""},{"line_number":34,"context_line":"  delegate_to: \"{{ physical_host }}\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# Run the lxc container creation process based on the backend store type"},{"line_number":37,"context_line":"- include: \"lxc_container_create_{{ lxc_container_backing_store }}.yml\""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"- name: Check container state"},{"line_number":40,"context_line":"  command: \"lxc-info -n {{ inventory_hostname }} --state\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7f515b1d_622436b0","line":37,"range":{"start_line":36,"start_character":0,"end_line":37,"end_character":71},"updated":"2017-10-05 12:55:25.000000000","message":"It would appear that all CoW file systems are just going to include the CoW task file, so perhaps we should rather do something like:\n\n- include: \"lxc_container_create_cow.yml\"\n  static:no\n  when:\n    - \"lxc_container_backing_store in [\u0027overlayfs\u0027, \u0027zfs\u0027, \u0027btrfs\u0027]\"\n\n- include \"lxc_container_create_{{ lxc_container_backing_store }}.yml\"\n  static: no\n  when:\n    - \"lxc_container_backing_store not in [\u0027overlayfs\u0027, \u0027zfs\u0027, \u0027btrfs\u0027]\"","commit_id":"3c2220010918aba3191709d772fb900f0ea215c8"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"864c78e53f461b63c8e13020fef1c3aa0451cf29","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    mode: \"0755\""},{"line_number":34,"context_line":"  delegate_to: \"{{ physical_host }}\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# Run the lxc container creation process based on the backend store type"},{"line_number":37,"context_line":"- include: \"lxc_container_create_{{ lxc_container_backing_store }}.yml\""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"- name: Check container state"},{"line_number":40,"context_line":"  command: \"lxc-info -n {{ inventory_hostname }} --state\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7f515b1d_223bdec8","line":37,"range":{"start_line":36,"start_character":0,"end_line":37,"end_character":71},"in_reply_to":"7f515b1d_622436b0","updated":"2017-10-05 12:55:59.000000000","message":"Alternatively, we could also just symlink the files instead of nesting the includes.","commit_id":"3c2220010918aba3191709d772fb900f0ea215c8"}],"tasks/main.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"3282b54dfd4d2cdd2a016d3d4f7236f40ae3fd0d","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    when:"},{"line_number":83,"context_line":"      - vg_result.rc !\u003d 0"},{"line_number":84,"context_line":"  when:"},{"line_number":85,"context_line":"    - lxc_container_backing_store \u003d\u003d \u0027lvm\u0027"},{"line_number":86,"context_line":"  tags:"},{"line_number":87,"context_line":"    - always"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f515b1d_5efc993d","line":85,"updated":"2017-10-04 16:22:52.000000000","message":"If the backing store is set to lvm through extra-vars then this set_fact won\u0027t work as it\u0027s a lower precedence.\n\nAlso, the default is now \u0027dir\u0027 (changed in this patch), so no need.\n\nIt might be better to just forget the auto-detection thing and just fail if the vg is not found. The \u0027legacy\u0027 behavior came form Juno. A reno can notify everyone.\n\nMy only reservation is that this does make it possible to mix and match, but I suppose that people can now use group_vars to override the backing store per groups so we don\u0027t need to auto-detect any more.","commit_id":"81528dde84546e6d5677d38ac7010f05a98c2671"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"a3185d18d7be8d948fb68f66ac8750341cf31da7","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    when:"},{"line_number":83,"context_line":"      - vg_result.rc !\u003d 0"},{"line_number":84,"context_line":"  when:"},{"line_number":85,"context_line":"    - lxc_container_backing_store \u003d\u003d \u0027lvm\u0027"},{"line_number":86,"context_line":"  tags:"},{"line_number":87,"context_line":"    - always"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f515b1d_19275bb9","line":85,"in_reply_to":"7f515b1d_5efc993d","updated":"2017-10-04 16:46:30.000000000","message":"Done","commit_id":"81528dde84546e6d5677d38ac7010f05a98c2671"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"cd732ed50c980602b72aeeacf589879a56d4f461","unresolved":false,"context_lines":[{"line_number":58,"context_line":"    - always"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# NOTE(cloudnull): Check for the LXC volume group when creating LVM backed"},{"line_number":61,"context_line":"#                  containers. If the LXC VG is not found fall back to \"dir\""},{"line_number":62,"context_line":"#                  this is being done to ensure we\u0027re not breaking away from"},{"line_number":63,"context_line":"#                  the legacy behavior."},{"line_number":64,"context_line":"- block:"},{"line_number":65,"context_line":"  - name: Check for lxc volume group"},{"line_number":66,"context_line":"    shell: \"(which vgs \u003e /dev/null \u0026\u0026 vgs | grep -o \u0027{{ lxc_container_vg_name }}\u0027) || false\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7f515b1d_c4b9bde9","line":63,"range":{"start_line":61,"start_character":31,"end_line":63,"end_character":39},"updated":"2017-10-05 09:50:36.000000000","message":"This comment is now invalid.","commit_id":"6eaf68b0bf74b1270f5ffc0365086291254081d3"}]}