]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 97529ef) | patch
Check for None on service's updated_at
authorMichal Dulko <michal.dulko@intel.com>
Tue, 15 Sep 2015 13:52:57 +0000 (15:52 +0200)
committerMichal Dulko <michal.dulko@intel.com>
Wed, 16 Sep 2015 18:03:42 +0000 (20:03 +0200)
commitdde6fba200008ecf0eca2defef7361d2edd654ac
treed97ae09f1880b569fcb066f37257c8dbc8657306tree | snapshot
parent97529ef9eb8aedf06b160a026d81a3272cf46701commit | diff
Check for None on service's updated_at

We weren't checking if service's updated_at is None when passing it to
timeutils.normalize_time which caused an exception. This commit adds
the check to cinder-manage and cinder-api (services and hosts). Also
regression unit tests are added.

Change-Id: Ia6ee28dc2cd20cece1e21d07692f47e3858d707d
Closes-Bug: 1495938
cinder/api/contrib/hosts.py diff | blob | history
cinder/api/contrib/services.py diff | blob | history
cinder/cmd/manage.py diff | blob | history
cinder/tests/unit/api/contrib/test_hosts.py diff | blob | history
cinder/tests/unit/api/contrib/test_services.py diff | blob | history
cinder/tests/unit/test_cmd.py diff | blob | history