Change in vdsm[master]: net: always persist owned ifcfg files on ovirt node
danken at redhat.com
danken at redhat.com
Wed Jul 29 07:59:46 UTC 2015
Dan Kenigsberg has posted comments on this change.
Change subject: net: always persist owned ifcfg files on ovirt node
......................................................................
Patch Set 1:
(2 comments)
https://gerrit.ovirt.org/#/c/44112/1/vdsm/network/configurators/ifcfg.py
File vdsm/network/configurators/ifcfg.py:
Line 478
Line 479
Line 480
Line 481
Line 482
I do not think that this is correct. If a file is pre-persisted, it means that its new content is going to be persisted prior to "set safe"
https://gerrit.ovirt.org/#/c/44112/1/vdsm/vdsm-store-net-config.in
File vdsm/vdsm-store-net-config.in:
Line 36: # /etc/sysconfig as "safe".
Line 37: rm -rf "$NET_CONF_BACK_DIR"/*
Line 38: }
Line 39:
Line 40: node_persist_owned_ifcfgs() {
this could be right, but please look up in the history. I think we use to have a ifcfg_node_persist call also in the unified case.
Line 41: for f in $(find "$NET_CONF_DIR" -type f); do
Line 42: if grep -q "# Generated by VDSM version" $f; then
Line 43: ovirt_store_config "$f"
Line 44: fi
--
To view, visit https://gerrit.ovirt.org/44112
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc717b86194a32c050d346e235a5c35fd318e1ff
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list