Changeset 1420 for trunk/packages

Timestamp:

Oct 29, 2008, 12:44:42 AM (16 years ago)

Author:

price

Message:

eliminate choice of source and cache files in invirt config

Location:

trunk/packages/invirt-base

Files:

• debian/changelog (modified) (1 diff)
• python/invirt/config.py (modified) (1 diff)
• scripts/invirt-getconf (modified) (3 diffs)

trunk/packages/invirt-base/debian/changelog

@@ +1 @@
+invirt-base (0.0.2) unstable; urgency=low
+
+  * eliminate choice of source and cache files for invirt.config.load and
+    invirt-getconf, so we can make it more complex
+
+ -- Greg Price <price@mit.edu>  Tue, 28 Oct 2008 21:16:14 -0400
+
 invirt-base (0.0.1) unstable; urgency=low
 

trunk/packages/invirt-base/python/invirt/config.py

@@ -7 +7 @@
 from contextlib import closing
 
-default_src_path   = '/etc/invirt/master.yaml'
-default_cache_path = '/var/lib/invirt/cache.json'
-lock_path          = '/var/lib/invirt/cache.lock'
+src_path   = '/etc/invirt/master.yaml'
+cache_path = '/var/lib/invirt/cache.json'
+lock_path  = '/var/lib/invirt/cache.lock'
 
-def load(src_path = default_src_path,
-         cache_path = default_cache_path,
-         force_refresh = False):
+def load(force_refresh = False):
     """
     Try loading the configuration from the faster-to-load JSON cache at

trunk/packages/invirt-base/scripts/invirt-getconf

@@ -17 +17 @@
 """
 
-from invirt.config import load
+from invirt import config
 from sys import argv, exit, stderr, stdout
 from optparse import OptionParser
@@ -27 +27 @@
         parser = OptionParser(usage = '%prog [options] key',
                 description = __doc__.strip().split('\n\n')[0])
-        parser.add_option('-s', '--src',
-                default = '/etc/invirt/master.yaml',
-                help = 'the source YAML configuration file to read from')
-        parser.add_option('-c', '--cache',
-                default = '/var/lib/invirt/invirt.json',
-                help = 'path to the JSON cache')
         parser.add_option('-r', '--refresh',
                 action = 'store_true',
@@ -48 +42 @@
             components = []
 
-        conf = load(opts.src, opts.cache, opts.refresh)
+        conf = config.load(opts.refresh)
         for i, component in enumerate(components):
             progress = '.'.join(components[:i])