Skip to content

Commit

Permalink
Fix name of libvirt_vm_image_cache_path variable
Browse files Browse the repository at this point in the history
This was renamed to libvirt_image_cache_path in the defaults and README,
but the name referenced in the task files was still
libvirt_vm_image_cache_path. This change reverts to the original name,
libvirt_vm_image_cache_path.

Temporarily support the use of libvirt_image_cache_path until the next
major release.
  • Loading branch information
markgoddard committed Nov 12, 2018
1 parent 750252d commit 27cc082
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ Role Variables
VM console logs, if a VM-specific log file path is not given. Default is
"/var/log/libvirt/qemu/".

- `libvirt_image_cache_path`: The directory in which to cache downloaded
- `libvirt_vm_image_cache_path`: The directory in which to cache downloaded
images. Default is "/tmp/".

- `libvirt_vm_engine`: virtualisation engine. If not set, the role will attempt
Expand Down
5 changes: 5 additions & 0 deletions defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,11 @@ libvirt_volume_default_format: qcow2
libvirt_volume_default_device: disk

# Path to cache downloaded images.
libvirt_vm_image_cache_path: "{{ libvirt_image_cache_path }}"

# NOTE(mgoddard): Temporarily support this name in addition to
# 'libvirt_vm_image_cache_path'.
# TODO(mgoddard): Remove this for the next major release.
libvirt_image_cache_path: "/tmp/"

# CPU architecture.
Expand Down

0 comments on commit 27cc082

Please sign in to comment.