Doron Fediuck has posted comments on this change.

Change subject: core: Make connection timeout configurable
......................................................................


Patch Set 10: Looks good to me, approved

As long as this is verified not to introduce new issues (which is what Juan 
checked) I'm fine with taking it.
Note that current values maintain previous behavior, so this will not resolve 
the reported issue. In order to resolve that more verifications needed for the 
new values.

Just a general warning; changing the current values will effect new connections 
to VDSM. This means that various flows such as storage flows, SPM election, HA 
flows needs to be checked.

Thanks Juan !

--
To view, visit http://gerrit.ovirt.org/8410
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I55a49ae5d655c2105c5840decec81ae712e40c32
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Noam Slomianko <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Kaul <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to