Hi!
On Mon, Jul 29, 2013 at 9:37 AM, Thomas Thrainer <[email protected]>wrote: > > > > On Thu, Jul 25, 2013 at 5:48 PM, Helga Velroyen <[email protected]> wrote: > >> This patch moves the default shared-file storage directory >> to the pathutils file. It will be removed from configure.ac >> at the end of this patch series. >> >> Signed-off-by: Helga Velroyen <[email protected]> >> --- >> lib/pathutils.py | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/lib/pathutils.py b/lib/pathutils.py >> index 496ceea..0401546 100644 >> --- a/lib/pathutils.py >> +++ b/lib/pathutils.py >> @@ -32,6 +32,7 @@ from ganeti import vcluster >> DEFAULT_FILE_STORAGE_DIR = "/srv/ganeti/file-storage" >> DEFAULT_FILE_STORAGE_DIR = >> vcluster.AddNodePrefix(DEFAULT_FILE_STORAGE_DIR) >> >> +DEFAULT_SHARED_FILE_STORAGE_DIR = "/srv/ganeti/shared-file-storage" >> > > In case we want at some point use shared file storage in VCluster QA, we > should add a vcluster.AddNodePrefix() call around the constant. But that > wouldn't change anything right now... > True, I think I did that in a later patch of this series. But thanks for pointing out! > > >> if _autoconf.ENABLE_SHARED_FILE_STORAGE: >> DEFAULT_SHARED_FILE_STORAGE_DIR = \ >> vcluster.AddNodePrefix(_autoconf.SHARED_FILE_STORAGE_DIR) >> -- >> 1.8.3 >> >> > LGTM, thanks. > Cheers, Helga
