openstack/tenks
Code Issues Proposed changes

Merge pull request #25 from jovial/usr-bin

Browse Source 
Use /usr/bin/virsh instead of /bin/virsh
This commit is contained in:
jovial 2018-10-03 17:40:30 +01:00 committed by GitHub
commit 7a04432226
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ansible/roles/ironic-enrolment/README.md
View File

@ -11,7 +11,7 @@ Requirements
the shell environment. These can be sourced from an OpenStack RC file, for the shell environment. These can be sourced from an OpenStack RC file, for
example. example.
- The `virsh` command-line tool present at `/bin/virsh`. - The `virsh` command-line tool present at `/usr/bin/virsh`.
Role Variables Role Variables
-------------- --------------

4
ansible/roles/ironic-enrolment/tasks/node.yml
View File

@ -12,9 +12,7 @@
# The output format of this command gives two lines of header, followed by # The output format of this command gives two lines of header, followed by
# (for each vNIC): # (for each vNIC):
# <name> <type> <source interface> <model> <MAC> # <name> <type> <source interface> <model> <MAC>
# The VMs will have been created with the virt module, using become: true. command: /usr/bin/virsh domiflist '{{ node.name }}'
# This targets /bin/virsh rather than /usr/bin/virsh.
command: /bin/virsh domiflist '{{ node.name }}'
register: iflist_res register: iflist_res
changed_when: false changed_when: false
become: true become: true