Change in vdsm[master]: Introducing getAvailableUpdates API

ybronhei at redhat.com ybronhei at redhat.com
Sun Apr 12 14:52:31 UTC 2015


Yaniv Bronhaim has posted comments on this change.

Change subject: Introducing getAvailableUpdates API
......................................................................


Patch Set 12:

(1 comment)

https://gerrit.ovirt.org/#/c/38559/12/vdsm/API.py
File vdsm/API.py:

Line 1316:             for pkgInfo in updates:
Line 1317:                 if packages is None or pkgInfo['name'] in frozenset(packages):
Line 1318:                     info['updates'].append(pkgInfo)
Line 1319: 
Line 1320:             info['last-check'] = datetime.datetime.fromtimestamp(
the only problem with that is that I don't know how to add it into the schema .. couldn't find anything like that. almost all verbs we return a list with one type
Line 1321:                 os.path.getmtime(update_file))
Line 1322:             return {'status': doneCode, 'info': info}
Line 1323:         except:
Line 1324:             self.log.error("failed to read update info", exc_info=True)


-- 
To view, visit https://gerrit.ovirt.org/38559
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9d47d5395519fff4378ab7dd88b30c4766467427
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list