Dan Kenigsberg has submitted this change and it was merged. Change subject: net config: make sure ifcfg files are not persisted on node ......................................................................
net config: make sure ifcfg files are not persisted on node When setting up networking, a multitude of ifcfg files may be modified. If we are using the (default) 'unified' persistence mode, we expect that none of these modification is to be persisted for the next boot. However already-persisted ifcfg files would; unpersisted files won't, which lead to the inconsistent state reported in the bug. In this patch we make sure that none of our changes to ifcfg files is to be persisted. Change-Id: Ia61b2385ce0bf44a67f627caf5b73b68992349fd Bug-Url: https://bugzilla.redhat.com/1165203 Signed-off-by: Dan Kenigsberg <dan...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/36985 Reviewed-by: Douglas Schilling Landgraf <dougsl...@redhat.com> Tested-by: Douglas Schilling Landgraf <dougsl...@redhat.com> --- M vdsm/network/configurators/ifcfg.py 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36985 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia61b2385ce0bf44a67f627caf5b73b68992349fd Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <celeb...@gmail.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabi...@redhat.com> Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com> Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com> Gerrit-Reviewer: Petr Horáček <phora...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches