This utilizes the config driver I submitted to resolve the hardcoded qemu
connection string. With this, the storage driver no longer has a circular
dependency with QEMU. Without this patch, when libvirtd is restarted, QEMU
requires storage (when domains are using storage pool backings) and storage
requires QEMU. This causes issues during startup.
Signed-off-by: Adam Walters <a...@pandorasboxen.com>
---
src/storage/storage_driver.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c
index a614279..1077a66 100644
--- a/src/storage/storage_driver.c
+++ b/src/storage/storage_driver.c
@@ -70,12 +70,12 @@ storageDriverAutostart(virStorageDriverStatePtr driver) {
size_t i;
virConnectPtr conn = NULL;
- /* XXX Remove hardcoding of QEMU URI */
- if (driverState->privileged)
- conn = virConnectOpen("qemu:///system");
- else
- conn = virConnectOpen("qemu:///session");
- /* Ignoring NULL conn - let backends decide */
+ conn = virConnectOpen("config:///");
+ /* Ignoring NULL conn - let backends decide. *
+ * As long as the config driver is built, and *
+ * it should be, since it is default on and *
+ * not configurable, a NULL conn should never *
+ * happen.
for (i = 0; i < driver->pools.count; i++) {
virStoragePoolObjPtr pool = driver->pools.objs[i];
--
1.8.5.2
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
