)]}'
{"ansible/docker-devicemapper.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"b3b0d1ac75a7f01c33ebbf02b68bba6ebe56ed0b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"        lineinfile:"},{"line_number":10,"context_line":"          path: /etc/docker/daemon.json"},{"line_number":11,"context_line":"          regexp: \u0027storage-driver.*devicemapper\u0027"},{"line_number":12,"context_line":"          state: absent"},{"line_number":13,"context_line":"        become: True"},{"line_number":14,"context_line":"        changed_when: False"},{"line_number":15,"context_line":"        check_mode: False"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_1907b01a","line":12,"range":{"start_line":12,"start_character":17,"end_line":12,"end_character":23},"updated":"2020-09-21 08:03:55.000000000","message":"I think this will actually make the change. You could use a simple grep, or a stat followed by slurp with from_json.","commit_id":"25ee518de4cc8b1aa85bcce018fac998c2e4fe74"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"41993f989ffe0ce7c104615c44f4cc9ffbc84cf0","unresolved":false,"context_lines":[{"line_number":9,"context_line":"        lineinfile:"},{"line_number":10,"context_line":"          path: /etc/docker/daemon.json"},{"line_number":11,"context_line":"          regexp: \u0027storage-driver.*devicemapper\u0027"},{"line_number":12,"context_line":"          state: absent"},{"line_number":13,"context_line":"        become: True"},{"line_number":14,"context_line":"        changed_when: False"},{"line_number":15,"context_line":"        check_mode: False"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_792eac8c","line":12,"range":{"start_line":12,"start_character":17,"end_line":12,"end_character":23},"in_reply_to":"9f560f44_1907b01a","updated":"2020-09-21 08:08:00.000000000","message":"I just realised that and pushed a new version which uses check_mode: True.","commit_id":"25ee518de4cc8b1aa85bcce018fac998c2e4fe74"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"b3b0d1ac75a7f01c33ebbf02b68bba6ebe56ed0b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"  tags:"},{"line_number":26,"context_line":"    - docker"},{"line_number":27,"context_line":"    - docker-devicemapper"},{"line_number":28,"context_line":"  tasks:"},{"line_number":29,"context_line":"    - name: Ensure docker devicemapper storage is configured"},{"line_number":30,"context_line":"      include_role:"},{"line_number":31,"context_line":"        name: docker-devicemapper"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_f9091c2b","line":28,"updated":"2020-09-21 08:03:55.000000000","message":"Any reason not to reuse the play?","commit_id":"25ee518de4cc8b1aa85bcce018fac998c2e4fe74"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"41993f989ffe0ce7c104615c44f4cc9ffbc84cf0","unresolved":false,"context_lines":[{"line_number":25,"context_line":"  tags:"},{"line_number":26,"context_line":"    - docker"},{"line_number":27,"context_line":"    - docker-devicemapper"},{"line_number":28,"context_line":"  tasks:"},{"line_number":29,"context_line":"    - name: Ensure docker devicemapper storage is configured"},{"line_number":30,"context_line":"      include_role:"},{"line_number":31,"context_line":"        name: docker-devicemapper"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_d97378b3","line":28,"in_reply_to":"9f560f44_f9091c2b","updated":"2020-09-21 08:08:00.000000000","message":"The description used in `name` doesn\u0027t match what\u0027s being done.","commit_id":"25ee518de4cc8b1aa85bcce018fac998c2e4fe74"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"648b3cd8d2bebb89bfa92f79be07f47109403248","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check for existing docker configuration using devicemapper"},{"line_number":3,"context_line":"  hosts: docker"},{"line_number":4,"context_line":"  tags:"},{"line_number":5,"context_line":"    - docker"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - block:"},{"line_number":8,"context_line":"      - name: Check for a Docker configuration file"},{"line_number":9,"context_line":"        become: True"},{"line_number":10,"context_line":"        stat:"},{"line_number":11,"context_line":"          path: /etc/docker/daemon.json"},{"line_number":12,"context_line":"        register: docker_config_file"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      - block:"},{"line_number":15,"context_line":"        - name: Check whether devicemapper is in use"},{"line_number":16,"context_line":"          lineinfile:"},{"line_number":17,"context_line":"            path: /etc/docker/daemon.json"},{"line_number":18,"context_line":"            regexp: \u0027storage-driver.*devicemapper\u0027"},{"line_number":19,"context_line":"            state: absent"},{"line_number":20,"context_line":"          become: True"},{"line_number":21,"context_line":"          changed_when: False"},{"line_number":22,"context_line":"          # `check_mode: True` ensures that lineinfile doesn\u0027t modify"},{"line_number":23,"context_line":"          # daemon.json"},{"line_number":24,"context_line":"          check_mode: True"},{"line_number":25,"context_line":"          register: devicemapper_docker"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        - name: Fail if devicemapper is in use while another storage driver was requested"},{"line_number":28,"context_line":"          fail:"},{"line_number":29,"context_line":"            msg: \"Docker storage driver {{ docker_storage_driver }} was requested, but devicemapper is in use\""},{"line_number":30,"context_line":"          when: devicemapper_docker.found \u003d\u003d 1"},{"line_number":31,"context_line":"        when: docker_config_file.stat.exists"},{"line_number":32,"context_line":"      when: docker_storage_driver !\u003d \u0027devicemapper\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Ensure docker devicemapper storage is configured"},{"line_number":35,"context_line":"  hosts: docker"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_b9d2045a","line":32,"range":{"start_line":2,"start_character":0,"end_line":32,"end_character":51},"updated":"2020-09-21 08:15:26.000000000","message":"why not docker_host_info? it\u0027s available from 2.8?","commit_id":"066b845349b928e742b12133e325d233a333b605"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"55a8f5d97de70daa57f09198bc25f1dc73f1b150","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check for existing docker configuration using devicemapper"},{"line_number":3,"context_line":"  hosts: docker"},{"line_number":4,"context_line":"  tags:"},{"line_number":5,"context_line":"    - docker"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - block:"},{"line_number":8,"context_line":"      - name: Check for a Docker configuration file"},{"line_number":9,"context_line":"        become: True"},{"line_number":10,"context_line":"        stat:"},{"line_number":11,"context_line":"          path: /etc/docker/daemon.json"},{"line_number":12,"context_line":"        register: docker_config_file"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      - block:"},{"line_number":15,"context_line":"        - name: Check whether devicemapper is in use"},{"line_number":16,"context_line":"          lineinfile:"},{"line_number":17,"context_line":"            path: /etc/docker/daemon.json"},{"line_number":18,"context_line":"            regexp: \u0027storage-driver.*devicemapper\u0027"},{"line_number":19,"context_line":"            state: absent"},{"line_number":20,"context_line":"          become: True"},{"line_number":21,"context_line":"          changed_when: False"},{"line_number":22,"context_line":"          # `check_mode: True` ensures that lineinfile doesn\u0027t modify"},{"line_number":23,"context_line":"          # daemon.json"},{"line_number":24,"context_line":"          check_mode: True"},{"line_number":25,"context_line":"          register: devicemapper_docker"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        - name: Fail if devicemapper is in use while another storage driver was requested"},{"line_number":28,"context_line":"          fail:"},{"line_number":29,"context_line":"            msg: \"Docker storage driver {{ docker_storage_driver }} was requested, but devicemapper is in use\""},{"line_number":30,"context_line":"          when: devicemapper_docker.found \u003d\u003d 1"},{"line_number":31,"context_line":"        when: docker_config_file.stat.exists"},{"line_number":32,"context_line":"      when: docker_storage_driver !\u003d \u0027devicemapper\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Ensure docker devicemapper storage is configured"},{"line_number":35,"context_line":"  hosts: docker"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_ec5afcef","line":32,"range":{"start_line":2,"start_character":0,"end_line":32,"end_character":51},"in_reply_to":"9f560f44_79c22c55","updated":"2020-09-21 09:17:17.000000000","message":"It doesn\u0027t work if docker is stopped though.","commit_id":"066b845349b928e742b12133e325d233a333b605"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6d7869c58d74857327bca3047bd4e6a8037aee25","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check for existing docker configuration using devicemapper"},{"line_number":3,"context_line":"  hosts: docker"},{"line_number":4,"context_line":"  tags:"},{"line_number":5,"context_line":"    - docker"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - block:"},{"line_number":8,"context_line":"      - name: Check for a Docker configuration file"},{"line_number":9,"context_line":"        become: True"},{"line_number":10,"context_line":"        stat:"},{"line_number":11,"context_line":"          path: /etc/docker/daemon.json"},{"line_number":12,"context_line":"        register: docker_config_file"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      - block:"},{"line_number":15,"context_line":"        - name: Check whether devicemapper is in use"},{"line_number":16,"context_line":"          lineinfile:"},{"line_number":17,"context_line":"            path: /etc/docker/daemon.json"},{"line_number":18,"context_line":"            regexp: \u0027storage-driver.*devicemapper\u0027"},{"line_number":19,"context_line":"            state: absent"},{"line_number":20,"context_line":"          become: True"},{"line_number":21,"context_line":"          changed_when: False"},{"line_number":22,"context_line":"          # `check_mode: True` ensures that lineinfile doesn\u0027t modify"},{"line_number":23,"context_line":"          # daemon.json"},{"line_number":24,"context_line":"          check_mode: True"},{"line_number":25,"context_line":"          register: devicemapper_docker"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        - name: Fail if devicemapper is in use while another storage driver was requested"},{"line_number":28,"context_line":"          fail:"},{"line_number":29,"context_line":"            msg: \"Docker storage driver {{ docker_storage_driver }} was requested, but devicemapper is in use\""},{"line_number":30,"context_line":"          when: devicemapper_docker.found \u003d\u003d 1"},{"line_number":31,"context_line":"        when: docker_config_file.stat.exists"},{"line_number":32,"context_line":"      when: docker_storage_driver !\u003d \u0027devicemapper\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Ensure docker devicemapper storage is configured"},{"line_number":35,"context_line":"  hosts: docker"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_79c22c55","line":32,"range":{"start_line":2,"start_character":0,"end_line":32,"end_character":51},"in_reply_to":"9f560f44_b9d2045a","updated":"2020-09-21 08:29:39.000000000","message":"Nice idea. We do need to make sure this works when docker and its SDK have not been installed yet though.","commit_id":"066b845349b928e742b12133e325d233a333b605"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"63ccb8ae9c98ae5e7f112c76fa817748b8aedb77","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check for existing docker configuration using devicemapper"},{"line_number":3,"context_line":"  hosts: docker"},{"line_number":4,"context_line":"  tags:"},{"line_number":5,"context_line":"    - docker"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - block:"},{"line_number":8,"context_line":"      - name: Check for a Docker configuration file"},{"line_number":9,"context_line":"        become: True"},{"line_number":10,"context_line":"        stat:"},{"line_number":11,"context_line":"          path: /etc/docker/daemon.json"},{"line_number":12,"context_line":"        register: docker_config_file"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      - block:"},{"line_number":15,"context_line":"        - name: Check whether devicemapper is in use"},{"line_number":16,"context_line":"          lineinfile:"},{"line_number":17,"context_line":"            path: /etc/docker/daemon.json"},{"line_number":18,"context_line":"            regexp: \u0027storage-driver.*devicemapper\u0027"},{"line_number":19,"context_line":"            state: absent"},{"line_number":20,"context_line":"          become: True"},{"line_number":21,"context_line":"          changed_when: False"},{"line_number":22,"context_line":"          # `check_mode: True` ensures that lineinfile doesn\u0027t modify"},{"line_number":23,"context_line":"          # daemon.json"},{"line_number":24,"context_line":"          check_mode: True"},{"line_number":25,"context_line":"          register: devicemapper_docker"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        - name: Fail if devicemapper is in use while another storage driver was requested"},{"line_number":28,"context_line":"          fail:"},{"line_number":29,"context_line":"            msg: \"Docker storage driver {{ docker_storage_driver }} was requested, but devicemapper is in use\""},{"line_number":30,"context_line":"          when: devicemapper_docker.found \u003d\u003d 1"},{"line_number":31,"context_line":"        when: docker_config_file.stat.exists"},{"line_number":32,"context_line":"      when: docker_storage_driver !\u003d \u0027devicemapper\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Ensure docker devicemapper storage is configured"},{"line_number":35,"context_line":"  hosts: docker"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_eac08a80","line":32,"range":{"start_line":2,"start_character":0,"end_line":32,"end_character":51},"in_reply_to":"9f560f44_d28caa36","updated":"2020-09-29 07:59:51.000000000","message":"I uploaded a new patch which uses docker_host_info, falling back to reading the configuration file if Docker is not reachable.","commit_id":"066b845349b928e742b12133e325d233a333b605"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"dde173fc192c1aae7871a9ae8cc7362463cf33cd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check for existing docker configuration using devicemapper"},{"line_number":3,"context_line":"  hosts: docker"},{"line_number":4,"context_line":"  tags:"},{"line_number":5,"context_line":"    - docker"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - block:"},{"line_number":8,"context_line":"      - name: Check for a Docker configuration file"},{"line_number":9,"context_line":"        become: True"},{"line_number":10,"context_line":"        stat:"},{"line_number":11,"context_line":"          path: /etc/docker/daemon.json"},{"line_number":12,"context_line":"        register: docker_config_file"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      - block:"},{"line_number":15,"context_line":"        - name: Check whether devicemapper is in use"},{"line_number":16,"context_line":"          lineinfile:"},{"line_number":17,"context_line":"            path: /etc/docker/daemon.json"},{"line_number":18,"context_line":"            regexp: \u0027storage-driver.*devicemapper\u0027"},{"line_number":19,"context_line":"            state: absent"},{"line_number":20,"context_line":"          become: True"},{"line_number":21,"context_line":"          changed_when: False"},{"line_number":22,"context_line":"          # `check_mode: True` ensures that lineinfile doesn\u0027t modify"},{"line_number":23,"context_line":"          # daemon.json"},{"line_number":24,"context_line":"          check_mode: True"},{"line_number":25,"context_line":"          register: devicemapper_docker"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        - name: Fail if devicemapper is in use while another storage driver was requested"},{"line_number":28,"context_line":"          fail:"},{"line_number":29,"context_line":"            msg: \"Docker storage driver {{ docker_storage_driver }} was requested, but devicemapper is in use\""},{"line_number":30,"context_line":"          when: devicemapper_docker.found \u003d\u003d 1"},{"line_number":31,"context_line":"        when: docker_config_file.stat.exists"},{"line_number":32,"context_line":"      when: docker_storage_driver !\u003d \u0027devicemapper\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Ensure docker devicemapper storage is configured"},{"line_number":35,"context_line":"  hosts: docker"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f560f44_d28caa36","line":32,"range":{"start_line":2,"start_character":0,"end_line":32,"end_character":51},"in_reply_to":"9f560f44_ec5afcef","updated":"2020-09-24 05:54:40.000000000","message":"Well, if it\u0027s started - you are not sure that nobody changed daemon.json in the meantime - no perfect solution :)","commit_id":"066b845349b928e742b12133e325d233a333b605"}]}