openstack/kayobe
Code Issues Proposed changes

Remove seed_hypervisor_libvirt_pool_capacity variable

Browse Source 
This is ignored by Libvirt, so there is no need to pass it to the
libvirt-host role.

Note: omitting the capacity argument requires libvirt-host 1.3.1 or
later.

Change-Id: Id94e7b514ed36c8e042e56e8f1891d98f8371e5d
Story: 2007381
Task: 38957
This commit is contained in:
Pierre Riteau 2020-03-05 16:06:49 +01:00
parent 208c46e72c
commit 7b7d39ebbe
4 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ansible/group_vars/all/seed-hypervisor
View File

@ -80,7 +80,6 @@ seed_hypervisor_libvirt_pools:
seed_hypervisor_libvirt_pool:
name: "{{ seed_hypervisor_libvirt_pool_name }}"
type: dir
capacity: "{{ seed_hypervisor_libvirt_pool_capacity }}"
path: "{{ seed_hypervisor_libvirt_pool_path }}"
mode: "{{ seed_hypervisor_libvirt_pool_mode }}"
owner: "{{ seed_hypervisor_libvirt_pool_owner }}"
@ -89,9 +88,6 @@ seed_hypervisor_libvirt_pool:
# Name of the libvirt storage pool for the seed VM.
seed_hypervisor_libvirt_pool_name: "default"
# Capacity of the libvirt storage pool for the seed VM.
seed_hypervisor_libvirt_pool_capacity: "{{ 75 * 2**30 }}"
# Directory path of the libvirt storage pool for the seed VM.
seed_hypervisor_libvirt_pool_path: "/var/lib/libvirt/images"

3
etc/kayobe/seed-hypervisor.yml
View File

@ -67,9 +67,6 @@
# Name of the libvirt storage pool for the seed VM.
#seed_hypervisor_libvirt_pool_name:
# Capacity of the libvirt storage pool for the seed VM.
#seed_hypervisor_libvirt_pool_capacity:
# Directory path of the libvirt storage pool for the seed VM.
#seed_hypervisor_libvirt_pool_path:

8
releasenotes/notes/remove-libvirt-pool-capacity-e428b893ca5bdb88.yaml Normal file
View File

@ -0,0 +1,8 @@
---
fixes:
- |
Removes the ``seed_hypervisor_libvirt_pool_capacity`` variable, which was
passed to Libvirt through the libvirt-host role when creating a storage
pool. This information is actually ignored by Libvirt, which gathers
capacity and other metadata from the storage backend. See `story 2007381
<https://storyboard.openstack.org/#!/story/2007381>`__ for details.

2
requirements.yml
View File

@ -24,7 +24,7 @@
- src: stackhpc.grafana-conf
version: 1.1.0
- src: stackhpc.libvirt-host
version: v1.3.0
version: v1.3.1
- src: stackhpc.libvirt-vm
version: v1.10.0
- src: stackhpc.mellanox-switch