diff options
| author | Daniel P. Berrange <berrange@redhat.com> | 2012-12-13 20:07:36 +0000 |
|---|---|---|
| committer | Daniel P. Berrange <berrange@redhat.com> | 2012-12-19 18:54:33 +0000 |
| commit | 4aa45d2900a3063305a7b2c727abfc2371909b21 (patch) | |
| tree | ba331927cb3a5d8122349144a98e527383d72674 /nova | |
| parent | 481d6ff142b8daeb08b22001997b5cf701a6d134 (diff) | |
| download | nova-4aa45d2900a3063305a7b2c727abfc2371909b21.tar.gz nova-4aa45d2900a3063305a7b2c727abfc2371909b21.tar.xz nova-4aa45d2900a3063305a7b2c727abfc2371909b21.zip | |
Allow loading of product/vendor/package info from external file
Currently the version.py file hardcodes some basic info for
the NOVA_PRODUCT, NOVA_VENDOR and NOVA_PACKAGE constants. It
is desirable that OS distro vendors can override/extend this
data in packages they ship. Cloud providers may also desire
to do the same. This allows Nova to optionally load this
data from a /etc/nova/release file. That file should contain
the data in a trivial "ini" format. For example:
[Nova]
vendor = Fedora Project
package = 1.fc18
Note that vendor sare not required to override all strings.
In this example, the 'product' field is not set, leaving
it on the default 'OpenStack Nova' value.
Change-Id: I3d8d32eaf35268c4b0908b0a93b7c9a2ea9c580a
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Diffstat (limited to 'nova')
| -rw-r--r-- | nova/tests/test_versions.py | 31 | ||||
| -rw-r--r-- | nova/version.py | 47 |
2 files changed, 78 insertions, 0 deletions
diff --git a/nova/tests/test_versions.py b/nova/tests/test_versions.py index 68e77ff9e..e864c00dd 100644 --- a/nova/tests/test_versions.py +++ b/nova/tests/test_versions.py @@ -14,7 +14,11 @@ # License for the specific language governing permissions and limitations # under the License. +import __builtin__ +import os +import StringIO +from nova.openstack.common import cfg from nova import test from nova import version @@ -53,3 +57,30 @@ class VersionTestCase(test.TestCase): """Ensure uninstalled code get version string""" self.assertEqual("2012.10-g9ec3421", self.version.version_string_with_package()) + + def test_release_file(self): + version.loaded = False + real_open = __builtin__.open + real_find_file = cfg.CONF.find_file + + def fake_find_file(self, name): + if name == "release": + return "/etc/nova/release" + return real_find_file(self, name) + + def fake_open(path, *args, **kwargs): + if path == "/etc/nova/release": + data = """[Nova] +vendor = ACME Corporation +product = ACME Nova +package = 1337""" + return StringIO.StringIO(data) + + return real_open(path, *args, **kwargs) + + self.stubs.Set(__builtin__, 'open', fake_open) + self.stubs.Set(cfg.ConfigOpts, 'find_file', fake_find_file) + + self.assertEqual(version.vendor_string(), "ACME Corporation") + self.assertEqual(version.product_string(), "ACME Nova") + self.assertEqual(version.package_string(), "1337") diff --git a/nova/version.py b/nova/version.py index 16b0d876a..82f3bb970 100644 --- a/nova/version.py +++ b/nova/version.py @@ -21,16 +21,63 @@ NOVA_VERSION = ['2013', '1', None] YEAR, COUNT, REVISION = NOVA_VERSION FINAL = False # This becomes true at Release Candidate time +loaded = False + + +def _load_config(): + # Don't load in global context, since we can't assume + # these modules are accessible when distutils uses + # this module + import ConfigParser + + from nova.openstack.common import cfg + from nova.openstack.common import log as logging + + global loaded, NOVA_VENDOR, NOVA_PRODUCT, NOVA_PACKAGE + if loaded: + return + + loaded = True + + cfgfile = cfg.CONF.find_file("release") + if cfgfile is None: + return + + try: + cfg = ConfigParser.RawConfigParser() + cfg.read(cfgfile) + + NOVA_VENDOR = cfg.get("Nova", "vendor") + if cfg.has_option("Nova", "vendor"): + NOVA_VENDOR = cfg.get("Nova", "vendor") + + NOVA_PRODUCT = cfg.get("Nova", "product") + if cfg.has_option("Nova", "product"): + NOVA_PRODUCT = cfg.get("Nova", "product") + + NOVA_PACKAGE = cfg.get("Nova", "package") + if cfg.has_option("Nova", "package"): + NOVA_PACKAGE = cfg.get("Nova", "package") + except Exception, ex: + LOG = logging.getLogger(__name__) + LOG.error("Failed to load %(cfgfile)s: %(ex)s" % locals()) + def vendor_string(): + _load_config() + return NOVA_VENDOR def product_string(): + _load_config() + return NOVA_PRODUCT def package_string(): + _load_config() + return NOVA_PACKAGE |
