Cleanup Magnum source code
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Magnum |
In Progress
|
Undecided
|
yatin |
Bug Description
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:/
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:/
, this can be removed now.
5) Translations for Pod/rc/
6) k8s fragments enable-etcd.sh is not used, etcd is enabled in master with enable-
7) Remove "provision_state" parameters(specific to ironic) from magnum source code these were originally copied from ironic: https:/
Other code cleanup can also be tracked within this bug report only by updating bug description and submitting patches seperately for each cleanup.
Changed in magnum: | |
assignee: | nobody → yatin (yatinkarel) |
status: | New → In Progress |
description: | updated |
description: | updated |
description: | updated |
Fix proposed to branch: master /review. openstack. org/376348
Review: https:/