Activity log for bug #1733356
| Date | Who | What changed | Old value | New value | Message |
|---|---|---|---|---|---|
| 2017-11-20 16:32:04 | hongbin | bug | added bug | ||
| 2017-11-20 16:32:51 | hongbin | description | The "search_volume" in zun.volume.cinder_api [1] is an re-implementation of "find" in cinderclient [2]. It is better to re-use the implementation in cinderclient instead of re-implementing our own. Let's investigate if we can leverage "find" to implement "search_volume" and remove codes in our side. [1] https://github.com/openstack/zun/blob/master/zun/volume/cinder_api.py [2] https://github.com/openstack/python-cinderclient/blob/master/cinderclient/base.py#L368 | The "search_volume" in zun.volume.cinder_api [1] is a re-implementation of "find" in cinderclient [2]. It is better to re-use the implementation in cinderclient instead of re-implementing our own. Let's investigate if we can leverage "find" to implement "search_volume" and remove the codes in our side. [1] https://github.com/openstack/zun/blob/master/zun/volume/cinder_api.py [2] https://github.com/openstack/python-cinderclient/blob/master/cinderclient/base.py#L368 | |
| 2017-11-20 16:33:11 | hongbin | description | The "search_volume" in zun.volume.cinder_api [1] is a re-implementation of "find" in cinderclient [2]. It is better to re-use the implementation in cinderclient instead of re-implementing our own. Let's investigate if we can leverage "find" to implement "search_volume" and remove the codes in our side. [1] https://github.com/openstack/zun/blob/master/zun/volume/cinder_api.py [2] https://github.com/openstack/python-cinderclient/blob/master/cinderclient/base.py#L368 | The "search_volume" in zun.volume.cinder_api [1] is a re-implementation of "find" in cinderclient [2]. It is better to re-use the implementation in cinderclient instead of re-implementing our own. Let's investigate if we can leverage "find" to implement "search_volume" so that we can reduce the codes to maintain. [1] https://github.com/openstack/zun/blob/master/zun/volume/cinder_api.py [2] https://github.com/openstack/python-cinderclient/blob/master/cinderclient/base.py#L368 | |
| 2017-11-20 16:33:24 | hongbin | zun: importance | Undecided | Low | |
| 2017-11-20 16:33:26 | hongbin | zun: status | New | Triaged | |
| 2017-11-30 03:22:30 | hongbin | zun: assignee | Kevin Zhao (kevin-zhao) | ||
| 2017-11-30 03:22:37 | hongbin | zun: status | Triaged | Fix Committed |
