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