)]}'
{"tripleo_ansible/playbooks/cli-overcloud-node-provide.yaml":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"5d1f21679536e9f30093662d24e6b08294d2e510","unresolved":false,"context_lines":[{"line_number":17,"context_line":"- name: Overcloud Node Provide"},{"line_number":18,"context_line":"  connection: local"},{"line_number":19,"context_line":"  hosts: localhost"},{"line_number":20,"context_line":"  pre_tasks:"},{"line_number":21,"context_line":"    - name: Check for required inputs"},{"line_number":22,"context_line":"      fail:"},{"line_number":23,"context_line":"        msg: \u003e"},{"line_number":24,"context_line":"          `node_uuids` or `all_manageable` are required inputs but"},{"line_number":25,"context_line":"          currently undefined. Check you inputs and try again."},{"line_number":26,"context_line":"      when:"},{"line_number":27,"context_line":"        - node_uuids is undefined"},{"line_number":28,"context_line":"        - all_manageable is undefined"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    - name: Check for required inputs"},{"line_number":31,"context_line":"      fail:"},{"line_number":32,"context_line":"        msg: \u003e"},{"line_number":33,"context_line":"          The option `all_manageable` is \"false\" but `node_uuids` is"},{"line_number":34,"context_line":"          undefined. Check your inputs and try again."},{"line_number":35,"context_line":"      when:"},{"line_number":36,"context_line":"        - node_uuids is undefined"},{"line_number":37,"context_line":"        - not (all_manageable | bool)"},{"line_number":38,"context_line":"  tasks:"},{"line_number":39,"context_line":"    - name: Run all manageable block"},{"line_number":40,"context_line":"      block:"},{"line_number":41,"context_line":"        # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":42,"context_line":"        - name: Get node UUIDS"},{"line_number":43,"context_line":"          command: \u003e"},{"line_number":44,"context_line":"            openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":45,"context_line":"          register: command_output"},{"line_number":46,"context_line":"          changed_when: false"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        - name: Set node_uuids fact"},{"line_number":49,"context_line":"          set_fact:"},{"line_number":50,"context_line":"            node_uuids: \"{{ command_output.stdout_lines }}\""},{"line_number":51,"context_line":"      when:"},{"line_number":52,"context_line":"        - all_manageable | bool"},{"line_number":53,"context_line":"        - node_uuids is undefined"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    - name: exit if nothing to do"},{"line_number":56,"context_line":"      block:"},{"line_number":57,"context_line":"        - name: Notice"},{"line_number":58,"context_line":"          debug:"},{"line_number":59,"context_line":"            msg: No nodes are manageable at this time."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        - name: end play"},{"line_number":62,"context_line":"          meta: end_play"},{"line_number":63,"context_line":"      when:"},{"line_number":64,"context_line":"        - (node_uuids | length) \u003c 1"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # Set nodes to available"},{"line_number":67,"context_line":"    - name: Make nodes available"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_fbbfe5c6","line":64,"range":{"start_line":20,"start_character":2,"end_line":64,"end_character":35},"updated":"2019-11-25 17:48:04.000000000","message":"maybe we can extract it to separate playbook, see: https://review.opendev.org/#/c/689194/23/tripleo_ansible/playbooks/workflow-baremetal-introspect.yaml","commit_id":"d8b4496d2946e86c5e1232d325141b065f43f91a"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"23052fdd23e706cb0e25b6a10ecc20fefd5671ca","unresolved":false,"context_lines":[{"line_number":17,"context_line":"- name: Overcloud Node Provide"},{"line_number":18,"context_line":"  connection: local"},{"line_number":19,"context_line":"  hosts: localhost"},{"line_number":20,"context_line":"  pre_tasks:"},{"line_number":21,"context_line":"    - name: Check for required inputs"},{"line_number":22,"context_line":"      fail:"},{"line_number":23,"context_line":"        msg: \u003e"},{"line_number":24,"context_line":"          `node_uuids` or `all_manageable` are required inputs but"},{"line_number":25,"context_line":"          currently undefined. Check you inputs and try again."},{"line_number":26,"context_line":"      when:"},{"line_number":27,"context_line":"        - node_uuids is undefined"},{"line_number":28,"context_line":"        - all_manageable is undefined"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    - name: Check for required inputs"},{"line_number":31,"context_line":"      fail:"},{"line_number":32,"context_line":"        msg: \u003e"},{"line_number":33,"context_line":"          The option `all_manageable` is \"false\" but `node_uuids` is"},{"line_number":34,"context_line":"          undefined. Check your inputs and try again."},{"line_number":35,"context_line":"      when:"},{"line_number":36,"context_line":"        - node_uuids is undefined"},{"line_number":37,"context_line":"        - not (all_manageable | bool)"},{"line_number":38,"context_line":"  tasks:"},{"line_number":39,"context_line":"    - name: Run all manageable block"},{"line_number":40,"context_line":"      block:"},{"line_number":41,"context_line":"        # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":42,"context_line":"        - name: Get node UUIDS"},{"line_number":43,"context_line":"          command: \u003e"},{"line_number":44,"context_line":"            openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":45,"context_line":"          register: command_output"},{"line_number":46,"context_line":"          changed_when: false"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        - name: Set node_uuids fact"},{"line_number":49,"context_line":"          set_fact:"},{"line_number":50,"context_line":"            node_uuids: \"{{ command_output.stdout_lines }}\""},{"line_number":51,"context_line":"      when:"},{"line_number":52,"context_line":"        - all_manageable | bool"},{"line_number":53,"context_line":"        - node_uuids is undefined"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    - name: exit if nothing to do"},{"line_number":56,"context_line":"      block:"},{"line_number":57,"context_line":"        - name: Notice"},{"line_number":58,"context_line":"          debug:"},{"line_number":59,"context_line":"            msg: No nodes are manageable at this time."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        - name: end play"},{"line_number":62,"context_line":"          meta: end_play"},{"line_number":63,"context_line":"      when:"},{"line_number":64,"context_line":"        - (node_uuids | length) \u003c 1"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # Set nodes to available"},{"line_number":67,"context_line":"    - name: Make nodes available"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_5b6c9206","line":64,"range":{"start_line":20,"start_character":2,"end_line":64,"end_character":35},"in_reply_to":"3fa7e38b_fbbfe5c6","updated":"2019-12-03 13:36:54.000000000","message":"we probably should, however, duplication is OK as we build out the playbook structure. Once we have the basic set of playbooks in place, and the CLI using them, we can begin optimizing the playbooks. IMO we\u0027re still establishing the pattern we\u0027re going to use and I\u0027d hate to begin optimizing too soon.","commit_id":"d8b4496d2946e86c5e1232d325141b065f43f91a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"5d1f21679536e9f30093662d24e6b08294d2e510","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        - (node_uuids | length) \u003c 1"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # Set nodes to available"},{"line_number":67,"context_line":"    - name: Make nodes available"},{"line_number":68,"context_line":"      command: \u003e-"},{"line_number":69,"context_line":"        openstack baremetal node provide {{ item }} --wait 1200"},{"line_number":70,"context_line":"      loop: \"{{ node_uuids }}\""},{"line_number":71,"context_line":"      async: 2400"},{"line_number":72,"context_line":"      poll: 0"},{"line_number":73,"context_line":"      register: node_provide"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    - name: poll for completion"},{"line_number":76,"context_line":"      async_status:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_dbeb09da","line":73,"range":{"start_line":67,"start_character":3,"end_line":73,"end_character":28},"updated":"2019-11-25 17:48:04.000000000","message":"I think we need a module for that too","commit_id":"d8b4496d2946e86c5e1232d325141b065f43f91a"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"23052fdd23e706cb0e25b6a10ecc20fefd5671ca","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        - (node_uuids | length) \u003c 1"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # Set nodes to available"},{"line_number":67,"context_line":"    - name: Make nodes available"},{"line_number":68,"context_line":"      command: \u003e-"},{"line_number":69,"context_line":"        openstack baremetal node provide {{ item }} --wait 1200"},{"line_number":70,"context_line":"      loop: \"{{ node_uuids }}\""},{"line_number":71,"context_line":"      async: 2400"},{"line_number":72,"context_line":"      poll: 0"},{"line_number":73,"context_line":"      register: node_provide"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    - name: poll for completion"},{"line_number":76,"context_line":"      async_status:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_1ba93a24","line":73,"range":{"start_line":67,"start_character":3,"end_line":73,"end_character":28},"in_reply_to":"3fa7e38b_dbeb09da","updated":"2019-12-03 13:36:54.000000000","message":"a module would be good, but in the absense of one I think we can push this along and retrofit the playbook with an appropriate module in a future iteration.","commit_id":"d8b4496d2946e86c5e1232d325141b065f43f91a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"5d1f21679536e9f30093662d24e6b08294d2e510","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    # Run cellv2 discovery of hosts"},{"line_number":86,"context_line":"    - name: Run cell_v2 host discovery"},{"line_number":87,"context_line":"      command: podman exec nova_api /bin/nova-manage cell_v2 discover_hosts --verbose"},{"line_number":88,"context_line":"      changed_when: false"},{"line_number":89,"context_line":"      become: true"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_9bdcb118","line":87,"range":{"start_line":87,"start_character":15,"end_line":87,"end_character":21},"updated":"2019-11-25 17:48:04.000000000","message":"should it be configurable for \"docker\" too?","commit_id":"d8b4496d2946e86c5e1232d325141b065f43f91a"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"23052fdd23e706cb0e25b6a10ecc20fefd5671ca","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    # Run cellv2 discovery of hosts"},{"line_number":86,"context_line":"    - name: Run cell_v2 host discovery"},{"line_number":87,"context_line":"      command: podman exec nova_api /bin/nova-manage cell_v2 discover_hosts --verbose"},{"line_number":88,"context_line":"      changed_when: false"},{"line_number":89,"context_line":"      become: true"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa7e38b_fb845eb4","line":87,"range":{"start_line":87,"start_character":15,"end_line":87,"end_character":21},"in_reply_to":"3fa7e38b_9bdcb118","updated":"2019-12-03 13:36:54.000000000","message":"going forward I don\u0027t think we support docker.","commit_id":"d8b4496d2946e86c5e1232d325141b065f43f91a"}],"tripleo_ansible/playbooks/overcloud-node-provide.yaml":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"7964bbac495c4eab9e4aa16a8352f41e4e3ec096","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":14,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"# under the License."},{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"- name: Overcloud Node Provide"},{"line_number":18,"context_line":"  connection: local"},{"line_number":19,"context_line":"  hosts: localhost"},{"line_number":20,"context_line":"  pre_tasks:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_de24fb10","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":30},"updated":"2019-11-13 00:58:39.000000000","message":"why is this not a role?","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"c9daada7ab7b0ac4cb8905098b7c75b8c04bf270","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":14,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"# under the License."},{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"- name: Overcloud Node Provide"},{"line_number":18,"context_line":"  connection: local"},{"line_number":19,"context_line":"  hosts: localhost"},{"line_number":20,"context_line":"  pre_tasks:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_2005a87a","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":30},"in_reply_to":"3fa7e38b_de24fb10","updated":"2019-11-18 16:59:47.000000000","message":"no need for this to be a role. Not really something that we\u0027d re-use in different playbooks.","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"208951310ead9252086b580aaf3310e9d9fd66f4","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"# under the License."},{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"- name: Overcloud Node Provide"},{"line_number":18,"context_line":"  connection: local"},{"line_number":19,"context_line":"  hosts: localhost"},{"line_number":20,"context_line":"  pre_tasks:"},{"line_number":21,"context_line":"    - name: Check for required inputs"},{"line_number":22,"context_line":"      fail:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_52908143","line":19,"range":{"start_line":17,"start_character":0,"end_line":19,"end_character":18},"updated":"2019-10-31 13:45:31.000000000","message":"Lets change this address http://lists.openstack.org/pipermail/openstack-discuss/2019-October/010459.html\n\n    - name: Overcloud Node Provide\n      connection: \"{{ (tripleo_target_host is defined) | ternary(\u0027ssh\u0027, \u0027local\u0027) }}\"\n      hosts: \"{{ tripleo_target_host | default(\u0027localhost\u0027) }}\"\n      user: \"{{ tripleo_target_user | default(lookup(\u0027env\u0027,\u0027USER\u0027)) }}\"\n\n\nWith this setup we\u0027d be able to run the ansible playbook with extra variables which define the target host and user. If undefined (the default) the playbook will run with the `local` connection driver on `localhost` with the current user.","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"7964bbac495c4eab9e4aa16a8352f41e4e3ec096","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":32,"context_line":"    - name: Get node UUIDS"},{"line_number":33,"context_line":"      shell: \u003e"},{"line_number":34,"context_line":"        openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":35,"context_line":"      register: command_output"},{"line_number":36,"context_line":"      changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_3e0e6f8c","line":33,"range":{"start_line":33,"start_character":7,"end_line":33,"end_character":11},"updated":"2019-11-13 00:58:39.000000000","message":"if we use shell, better to use \"command\"","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"c9daada7ab7b0ac4cb8905098b7c75b8c04bf270","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":32,"context_line":"    - name: Get node UUIDS"},{"line_number":33,"context_line":"      shell: \u003e"},{"line_number":34,"context_line":"        openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":35,"context_line":"      register: command_output"},{"line_number":36,"context_line":"      changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_a0cf5855","line":33,"range":{"start_line":33,"start_character":7,"end_line":33,"end_character":11},"in_reply_to":"3fa7e38b_3e0e6f8c","updated":"2019-11-18 16:59:47.000000000","message":"+1","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"c75217c418784ebe04ffbdc9bdcafa94c655a8ff","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":32,"context_line":"    - name: Get node UUIDS"},{"line_number":33,"context_line":"      shell: \u003e"},{"line_number":34,"context_line":"        openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":35,"context_line":"      register: command_output"},{"line_number":36,"context_line":"      changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_a87708d1","line":33,"range":{"start_line":33,"start_character":7,"end_line":33,"end_character":11},"in_reply_to":"3fa7e38b_a0cf5855","updated":"2019-11-21 18:16:42.000000000","message":"Done","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"change_message_id":"cabac1c78c88815e4ca3db6c91fedb471c843ade","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":32,"context_line":"    - name: Get node UUIDS"},{"line_number":33,"context_line":"      shell: \u003e"},{"line_number":34,"context_line":"        openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":35,"context_line":"      register: command_output"},{"line_number":36,"context_line":"      changed_when: false"},{"line_number":37,"context_line":"      when:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_52bf2160","line":34,"updated":"2019-10-31 13:56:12.000000000","message":"Would any of the os_ironic* Ansible modules would here instead of using shell commands? https://docs.ansible.com/ansible/latest/modules/list_of_cloud_modules.html#openstack","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"1971ac8180f2d906f2307cb0dab449f2e8a366c0","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":32,"context_line":"    - name: Get node UUIDS"},{"line_number":33,"context_line":"      shell: \u003e"},{"line_number":34,"context_line":"        openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":35,"context_line":"      register: command_output"},{"line_number":36,"context_line":"      changed_when: false"},{"line_number":37,"context_line":"      when:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_b22695ca","line":34,"in_reply_to":"3fa7e38b_52bf2160","updated":"2019-10-31 14:01:46.000000000","message":"sadly, for what we\u0027re doing here, those modules wont work :\u0027( While it\u0027d be wonderful if we could get a module that does a lot of this, ansible is just not there yet. That said, every effort to make this idempotent has been made, even though we\u0027re using a shell command.","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"c9daada7ab7b0ac4cb8905098b7c75b8c04bf270","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":32,"context_line":"    - name: Get node UUIDS"},{"line_number":33,"context_line":"      shell: \u003e"},{"line_number":34,"context_line":"        openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":35,"context_line":"      register: command_output"},{"line_number":36,"context_line":"      changed_when: false"},{"line_number":37,"context_line":"      when:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_20dcc816","line":34,"in_reply_to":"3fa7e38b_9e744338","updated":"2019-11-18 16:59:47.000000000","message":"+1 we should write modules to better enable this functionality. That said, we can replace these shell commands with modules when the modules are available.","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"7964bbac495c4eab9e4aa16a8352f41e4e3ec096","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    # Get Node UUIDs, if \"all_manageable\" is True"},{"line_number":32,"context_line":"    - name: Get node UUIDS"},{"line_number":33,"context_line":"      shell: \u003e"},{"line_number":34,"context_line":"        openstack baremetal node list --provision-state\u003dmanageable -c UUID -f value"},{"line_number":35,"context_line":"      register: command_output"},{"line_number":36,"context_line":"      changed_when: false"},{"line_number":37,"context_line":"      when:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_9e744338","line":34,"in_reply_to":"3fa7e38b_b22695ca","updated":"2019-11-13 00:58:39.000000000","message":"Then let\u0027s write them :D\nWe have already new ansible modules here in the repo, like podman_*  modules. We can either to write from scratch or just to add functionality for current and put it in this repo. Later we can suggest them to ansible upstream without rush.\nIn this case writing os_ironic_info module will be not hard.","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"7964bbac495c4eab9e4aa16a8352f41e4e3ec096","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        - all_manageable | bool"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    - set_fact:"},{"line_number":41,"context_line":"        node_uuids: \"{{ command_output.stdout_lines }}\""},{"line_number":42,"context_line":"      when:"},{"line_number":43,"context_line":"        - all_manageable | bool"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_9e73e316","line":41,"range":{"start_line":41,"start_character":7,"end_line":41,"end_character":55},"updated":"2019-11-13 00:58:39.000000000","message":"if you pass variable to this playbook, it will override this set_fact (see vars precedence), but only for scope of this playbook and it may create confusions. Better to add another task \"when not all_manageable\" and use different variable","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"f15223549994e34c921c966e796c7eee7b3a35b0","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        - all_manageable | bool"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    - set_fact:"},{"line_number":41,"context_line":"        node_uuids: \"{{ command_output.stdout_lines }}\""},{"line_number":42,"context_line":"      when:"},{"line_number":43,"context_line":"        - all_manageable | bool"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_ae1dd162","line":41,"range":{"start_line":41,"start_character":7,"end_line":41,"end_character":55},"in_reply_to":"3fa7e38b_9e73e316","updated":"2019-11-21 18:24:25.000000000","message":"Done","commit_id":"55634df365364ff395d23fc7ad4e1302bb3d0313"}]}