yahoo-eng-team team mailing list archive

[YaoZheng_ZTE <zheng.yao1@xxxxxxxxxx>]

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