Public bug reported: In the line https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/cinder.py#L436, Its intent is to get ceiling of size_gb. we can use python math module math.ceil() function. This can improve the code readability. So i suggest improve it.
** Affects: glance Importance: Undecided Assignee: YaoZheng_ZTE (zheng-yao1) Status: New ** Changed in: glance Assignee: (unassigned) => YaoZheng_ZTE (zheng-yao1) ** Description changed: - In the line https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/cinder.py#L436, Its intent is to get ceiling of size_gb. we can use python math module - math.ceil() function. This can improve the code readability. So i suggest improve it. + In the line + https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/cinder.py#L436, + Its intent is to get ceiling of size_gb. we can use python math module + math.ceil() function. This can improve the code readability. So i + suggest improve it. -- You received this bug notification because you are a member of Yahoo! Engineering Team, which is subscribed to Glance. https://bugs.launchpad.net/bugs/1592376 Title: Cinder driver the function add calculate size_gb need improve Status in Glance: New Bug description: In the line https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/cinder.py#L436, Its intent is to get ceiling of size_gb. we can use python math module math.ceil() function. This can improve the code readability. So i suggest improve it. To manage notifications about this bug go to: https://bugs.launchpad.net/glance/+bug/1592376/+subscriptions -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : yahoo-eng-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp