Assaf Muller has posted comments on this change.
Change subject: Unified network persistence [1/3] - Save running config
......................................................................
Patch Set 8: (1 inline comment)
....................................................
File vdsm/vdsm-store-net-config.in
Line 52: PERS_CONF_SYMLINK=$PERS_NET_CONF_PATH
Line 53: PERS_CONF_DIR_ROOTNAME="$PERS_CONF_SYMLINK."
Line 54: PERS_CONF_NEW_DIR="$PERS_CONF_DIR_ROOTNAME$TIMESTAMP"
Line 55: PERS_CONF_NEW_SYMLINK="$PERS_CONF_SYMLINK.link.$TIMESTAMP"
Line 56:
I got this issue on my machine as well. I'll start working on patches 2&3 this
Sunday. I worked on patch 1 thus far.
Line 57: cp -r "$RUN_CONF_DIR" "$PERS_CONF_NEW_DIR"
Line 58: ln -s "$PERS_CONF_NEW_DIR" "$PERS_CONF_NEW_SYMLINK"
Line 59: mv -fT "$PERS_CONF_NEW_SYMLINK" "$PERS_CONF_SYMLINK"
Line 60: find "$PERS_CONF_PATH" -type d -path
"$PERS_CONF_DIR_ROOTNAME*" | \
--
To view, visit
http://gerrit.ovirt.org/16699
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7137a96f84abd2c5e532c6c37737e36ef17567a9
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvallare(a)redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer(a)redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Petr Šebek <psebek(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server