Ondřej Svoboda has posted comments on this change. Change subject: configurators: move disable_ipv6 code to a common function ......................................................................
Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/46984/3/vdsm/network/configurators/iproute2.py File vdsm/network/configurators/iproute2.py: Line 78: update_ipv6_sysctl > iproute2 must not touch sysctl - it should use only our own unified persist To be more specific, is it /etc/sysctl.d that only the ifcfg configurator should touch? Other configurators can enable/disable IPv6 at runtime. -- To view, visit https://gerrit.ovirt.org/46984 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I39d39f10cc2e7f593bb4ba3c65c0e04ff08715ee Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
