2016-09-26 09:31:23 |
yatin |
bug |
|
|
added bug |
2016-09-26 09:31:29 |
yatin |
magnum: assignee |
|
yatin (yatinkarel) |
|
2016-09-26 09:31:34 |
yatin |
magnum: status |
New |
In Progress |
|
2016-09-27 09:20:37 |
yatin |
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://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. |
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) Translations for Pod/rc/container/svc
6) ...
Other code cleanup can also be tracked within this bug report only by updating bug description and submitting patches seperately for each cleanup. |
|
2016-09-28 09:05:06 |
yatin |
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://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) Translations for Pod/rc/container/svc
6) ...
Other code cleanup can also be tracked within this bug report only by updating bug description and submitting patches seperately for each cleanup. |
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) Translations for Pod/rc/container/svc
6) k8s fragments enable-etcd.sh is not used, etcd is enabled in master with enable-services-master.sh
7) ...
Other code cleanup can also be tracked within this bug report only by updating bug description and submitting patches seperately for each cleanup. |
|
2016-12-28 10:40:46 |
yatin |
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://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) Translations for Pod/rc/container/svc
6) k8s fragments enable-etcd.sh is not used, etcd is enabled in master with enable-services-master.sh
7) ...
Other code cleanup can also be tracked within this bug report only by updating bug description and submitting patches seperately for each cleanup. |
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) Translations for Pod/rc/container/svc
6) k8s fragments enable-etcd.sh is not used, etcd is enabled in master with enable-services-master.sh
7) Remove "provision_state" parameters(specific to ironic) from magnum source code these were originally copied from ironic: https://review.openstack.org/#/c/138280
Other code cleanup can also be tracked within this bug report only by updating bug description and submitting patches seperately for each cleanup. |
|