tripleo_common/image_uploader.py needs a huge refactoring
Bug #1818480 reported by
Cédric Jeanneret
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| tripleo |
Incomplete
|
Medium
|
Unassigned | ||
Bug Description
Hello,
The codebase in tripleo_common "image_uploader.py" file has a long history, and would need some refactoring and cleanup for the sake of code maintenance and evolutivity.
It might be appropriate to separate the different classes in dedicated files in order to avoid having a 1500+ lines file, and it would probably make sense to move some methods either out or in the different classes (especially inside BaseImageUploader, and the lonely methods created at the of that same file, out of any object/class).
Cheers,
C.
| tags: | added: containers tech-debt |
| Changed in tripleo: | |
| importance: | Undecided → Medium |
| Changed in tripleo: | |
| milestone: | stein-3 → train-1 |
| Changed in tripleo: | |
| milestone: | train-1 → train-2 |
| Changed in tripleo: | |
| milestone: | train-2 → train-3 |
| Changed in tripleo: | |
| milestone: | train-3 → ussuri-1 |
| Changed in tripleo: | |
| milestone: | ussuri-1 → ussuri-2 |
| Changed in tripleo: | |
| milestone: | ussuri-2 → ussuri-3 |
| Changed in tripleo: | |
| status: | Triaged → Incomplete |
| Changed in tripleo: | |
| milestone: | ussuri-3 → ussuri-rc3 |
| Changed in tripleo: | |
| milestone: | ussuri-rc3 → victoria-1 |
| Changed in tripleo: | |
| milestone: | victoria-1 → victoria-3 |
To post a comment you must log in.
