Disallowing them broke a use case of testing multipath configurations
for storage. The original intent was added as it made it impossible to
use certain /dev/disk-by... links.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1349895
---
 src/conf/domain_conf.c | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 9443281..68e89ed 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -24906,20 +24906,6 @@ virDomainDiskDefCheckDuplicateInfo(const 
virDomainDiskDef *a,
         return -1;
     }

-    if (a->wwn && b->wwn && STREQ(a->wwn, b->wwn)) {
-        virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
-                       _("Disks '%s' and '%s' have identical WWN"),
-                       a->dst, b->dst);
-        return -1;
-    }
-
-    if (a->serial && b->serial && STREQ(a->serial, b->serial)) {
-        virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
-                       _("Disks '%s' and '%s' have identical serial"),
-                       a->dst, b->dst);
-        return -1;
-    }
-
     return 0;
 }

-- 
2.9.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to