Since a successful completion of the calls to openvswitch is expected
a longer timeout should be able to be chosen to account for loaded systems.
Therefore this patch provides the ability to specify the timeout value for
openvswitch calls in the libvirtd configuration file.

Signed-off-by: Boris Fiuczynski <fiuczy@linux.vnet.ibm.com>
Reviewed-by: Bjoern Walk <bwalk@linux.vnet.ibm.com>
---
 daemon/libvirtd.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
index b6d76ed..5c30c9e 100644
--- a/daemon/libvirtd.c
+++ b/daemon/libvirtd.c
@@ -58,6 +58,7 @@
 #include "viraccessmanager.h"
 #include "virutil.h"
 #include "virgettext.h"
+#include "util/virnetdevopenvswitch.h"
 
 #ifdef WITH_DRIVER_MODULES
 # include "driver.h"
@@ -658,6 +659,16 @@ daemonSetupNetworking(virNetServerPtr srv,
 
 
 /*
+ * Set up the openvswitch timeout
+ */
+static void
+daemonSetupNetDevOpenvswitch(struct daemonConfig *config)
+{
+    virNetDevOpenvswitchSetTimeout(config->ovs_timeout);
+}
+
+
+/*
  * Set up the logging environment
  * By default if daemonized all errors go to the logfile libvirtd.log,
  * but if verbose or error debugging is asked for then also output
@@ -1267,6 +1278,8 @@ int main(int argc, char **argv) {
         exit(EXIT_FAILURE);
     }
 
+    daemonSetupNetDevOpenvswitch(config);
+
     if (daemonSetupAccessManager(config) < 0) {
         VIR_ERROR(_("Can't initialize access manager"));
         exit(EXIT_FAILURE);
-- 
2.5.5

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

On 02/07/2017 04:16 PM, Boris Fiuczynski wrote:
> Since a successful completion of the calls to openvswitch is expected
> a longer timeout should be able to be chosen to account for loaded systems.
> Therefore this patch provides the ability to specify the timeout value for
> openvswitch calls in the libvirtd configuration file.
> 
> Signed-off-by: Boris Fiuczynski <fiuczy@linux.vnet.ibm.com>
> Reviewed-by: Bjoern Walk <bwalk@linux.vnet.ibm.com>
> ---
>  daemon/libvirtd.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)

ACK

Michal

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

On 02/09/2017 09:01 AM, Michal Privoznik wrote:
> On 02/07/2017 04:16 PM, Boris Fiuczynski wrote:
>> Since a successful completion of the calls to openvswitch is expected
>> a longer timeout should be able to be chosen to account for loaded systems.
>> Therefore this patch provides the ability to specify the timeout value for
>> openvswitch calls in the libvirtd configuration file.
>>
>> Signed-off-by: Boris Fiuczynski <fiuczy@linux.vnet.ibm.com>
>> Reviewed-by: Bjoern Walk <bwalk@linux.vnet.ibm.com>
>> ---
>>  daemon/libvirtd.c | 13 +++++++++++++
>>  1 file changed, 13 insertions(+)
>
> ACK
>
> Michal
>
Thanks


-- 
Mit freundlichen Grüßen/Kind regards
    Boris Fiuczynski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martina Köderitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294

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