Comment 0 for bug 1627663

Revision history for this message
yatin (yatinkarel) wrote :

Source code of magnum is changed/structured too much in Newton, some code updation/deletion is missed, which can be cleaned up.

1) Cleanup exception.py: pod, rc, container are removed so there exception's can be removed.
Earlier cleanup for exception was done with this patch also: https://review.openstack.org/#/c/320843/

2) k8s_manifest and its UT: since pod, rc and service are removed from magnum code, k8s_manifest which was used by by them can be removed.

3) yamlutils.py and its UT: yamlutils was added in beginning, since then it is not used, so should be removed.

4. safe_utils.py: it was used until it's use is removed in: https://review.openstack.org/#/c/320843/
, this can be removed now.

5) ....

Other code cleanup can also be tracked within this bug report only by updating bug description and submitting patches seperately for each cleanup.