Ceph » devops

So currently the version to install with yum is grabbed from sha1/$sha1/version or /ref/$branch/version

Basically we do:

version = _block_looking_for_package_version(
remote,
base_url,
config.get('wait_for_package', False))

block_looking_for_package grabs /version off the gitbuilder archive dir for branch/sha1. Then it basically uses this info (does a little removes the v if it has one and adds the . followed by dist to it) to generate the package list and installs with yum:

pkgs_with_vers = ["%s-%s" % (pkg, version) for pkg in pkgs]
args = ['sudo', 'yum', '-y', 'install']
args += pkgs_with_vers
_run_and_log_error_if_fails(remote, args)

Problem is that we are getting mis-matches, Example froM:
http://gitbuilder.ceph.com/ceph-rpm-centos6-x86_64-basic/ref/emperor/version

version file: v0.72.1
formated by teuthology: 0.72.1.el6
Actual version of the rpms: 0.72.1-0.el6 (has a -0)

Example from fedora:
http://gitbuilder.ceph.com/ceph-rpm-fc18-x86_64-basic/ref/emperor/version
version file: v0.72-3.g4d92386
formated by teuthology: 0.72-3.g4d92386.fc18
Actual version of the rpms: 0.72-1-3.g4d92386.fc18.x86_64 (has a -1)

So this needs to either be fixed in the gitbuilders that create the version text file OR we need to stop using that file on teuthology and figure out the package name a different way such as:

list available package on the repo via yum, parse output and grab it that way.
Parse xml in repodata folder and figure out correct package names that way.

I am guessing the easier solution and reducing parisng is a good thing is to fix it on the gitbuilders.

Maybe alfredo/zack have a better idea on how this should be handeled.

Gary if you think teuthology should not be using that /version file to figure out what the versio info should be then please assign this back to me or move it to teuthology (if that can be done?).