)]}'
{"playbooks/common-playbooks/cinder.yml":[{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"28a20440dede6a04b2424fad8e11890a4a57d48d","unresolved":false,"context_lines":[{"line_number":49,"context_line":"          - \"lxc.autodev\u003d0\""},{"line_number":50,"context_line":"          - \"lxc.cgroup.devices.allow\u003da *:* rmw\""},{"line_number":51,"context_line":"          - \"lxc.mount.entry\u003dudev dev devtmpfs defaults 0 0\""},{"line_number":52,"context_line":"        extra_container_config_no_restart:"},{"line_number":53,"context_line":"          - \"lxc.start.order\u003d39\""},{"line_number":54,"context_line":"      when:"},{"line_number":55,"context_line":"        - \"\u0027cinder_volume\u0027 in group_names\""},{"line_number":56,"context_line":"        - \"cinder_backend_lvm_inuse | bool\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_29274715","line":53,"range":{"start_line":52,"start_character":0,"end_line":53,"end_character":32},"updated":"2018-05-03 18:02:11.000000000","message":"We could drop this....","commit_id":"de02899c2f2f4056ccf74277055636d2be6fdb42"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"cdb1a796e2717b4a635cd680409d0cfa0141f5e7","unresolved":false,"context_lines":[{"line_number":49,"context_line":"          - \"lxc.autodev\u003d0\""},{"line_number":50,"context_line":"          - \"lxc.cgroup.devices.allow\u003da *:* rmw\""},{"line_number":51,"context_line":"          - \"lxc.mount.entry\u003dudev dev devtmpfs defaults 0 0\""},{"line_number":52,"context_line":"        extra_container_config_no_restart:"},{"line_number":53,"context_line":"          - \"lxc.start.order\u003d39\""},{"line_number":54,"context_line":"      when:"},{"line_number":55,"context_line":"        - \"\u0027cinder_volume\u0027 in group_names\""},{"line_number":56,"context_line":"        - \"cinder_backend_lvm_inuse | bool\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_72af6894","line":53,"range":{"start_line":52,"start_character":0,"end_line":53,"end_character":32},"in_reply_to":"5f7c97a3_29274715","updated":"2018-05-03 21:14:38.000000000","message":"This is ensuring that the container is started in a particular order.","commit_id":"de02899c2f2f4056ccf74277055636d2be6fdb42"},{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"28a20440dede6a04b2424fad8e11890a4a57d48d","unresolved":false,"context_lines":[{"line_number":53,"context_line":"          - \"lxc.start.order\u003d39\""},{"line_number":54,"context_line":"      when:"},{"line_number":55,"context_line":"        - \"\u0027cinder_volume\u0027 in group_names\""},{"line_number":56,"context_line":"        - \"cinder_backend_lvm_inuse | bool\""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    - name: Configure container (cinder-volume)"},{"line_number":59,"context_line":"      include: \"../common-tasks/os-{{ container_tech | default(\u0027lxc\u0027) }}-container-setup.yml\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_4924030e","line":56,"range":{"start_line":56,"start_character":0,"end_line":56,"end_character":43},"updated":"2018-05-03 18:02:11.000000000","message":"And drop this too","commit_id":"de02899c2f2f4056ccf74277055636d2be6fdb42"},{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"28a20440dede6a04b2424fad8e11890a4a57d48d","unresolved":false,"context_lines":[{"line_number":63,"context_line":"          - \"lxc.start.order\u003d39\""},{"line_number":64,"context_line":"      when:"},{"line_number":65,"context_line":"        - \"\u0027cinder_volume\u0027 in group_names\""},{"line_number":66,"context_line":"        - \"not cinder_backend_lvm_inuse | bool\""},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    - name: Configure container (other services)"},{"line_number":69,"context_line":"      include: \"../common-tasks/os-{{ container_tech | default(\u0027lxc\u0027) }}-container-setup.yml\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_e9206ffa","line":66,"range":{"start_line":66,"start_character":0,"end_line":66,"end_character":47},"updated":"2018-05-03 18:02:11.000000000","message":"And remove this, that way, we don\u0027t have to run through the start order twice?","commit_id":"de02899c2f2f4056ccf74277055636d2be6fdb42"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"cdb1a796e2717b4a635cd680409d0cfa0141f5e7","unresolved":false,"context_lines":[{"line_number":63,"context_line":"          - \"lxc.start.order\u003d39\""},{"line_number":64,"context_line":"      when:"},{"line_number":65,"context_line":"        - \"\u0027cinder_volume\u0027 in group_names\""},{"line_number":66,"context_line":"        - \"not cinder_backend_lvm_inuse | bool\""},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    - name: Configure container (other services)"},{"line_number":69,"context_line":"      include: \"../common-tasks/os-{{ container_tech | default(\u0027lxc\u0027) }}-container-setup.yml\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_3291f053","line":66,"range":{"start_line":66,"start_character":0,"end_line":66,"end_character":47},"in_reply_to":"5f7c97a3_e9206ffa","updated":"2018-05-03 21:14:38.000000000","message":"The reason for this to exist in two places and key off of \"cinder_backend_lvm_inuse\" is because we don\u0027t want to add the container config, which provides a lot of system level access, where it\u0027s not needed.","commit_id":"de02899c2f2f4056ccf74277055636d2be6fdb42"}]}