Added check for required keys in Redis
authorEd Bartosh <eduard.bartosh@intel.com>
Tue, 10 Sep 2013 14:47:59 +0000 (17:47 +0300)
committerHasan Wan <hasan.wan@intel.com>
Wed, 11 Sep 2013 02:11:42 +0000 (19:11 -0700)
job_pre_release_obs now checks for the required keys coming from
repos.yaml to Redis instead of crashing with KeyError exception.

Change-Id: I12544662d95bc63643bb51f2aa88bd478919299e
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/6366
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Hasan Wan <hasan.wan@intel.com>
job_pre_release_obs.py

index a38bbce..9e91969 100755 (executable)
@@ -49,6 +49,12 @@ def get_prerelease_data(backenddb, base_url, base_path='/srv/download',
         raise LocalError("Can't get info for repo '%s' : %s" \
                          % (repo_name, str(err)))
 
+    for key in ('Release', 'SnapshotDir', 'PrereleaseDir',
+                'PartOf', 'Architectures'):
+        if key not in repo:
+            raise  LocalError("Key '%s' doesn't exist for repo '%s' "\
+                              "in Redis" % (key, repo_name))
+
     release_key = '%s-%s' % (repo['PartOf'], repo['Release'])
     try:
         release_id = backenddb.get_release_ids()[release_key]