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

Reply via email to