Code review comment for lp:~rconradharris/nova/server_progress

Revision history for this message
Trey Morris (tr3buchet) wrote :

You have valid reasons for merging, and I definitely agree about hashing it out in a formal setting. I'd be on the side against in this setting.

Ease of code removal isn't the issue. My fear is once progress bars are supported we'll never be able to get rid of them. Much the same as how they were supported in the last cloud implementation and are still plaguing us in the next. It may take you <5 minutes to remove it, but months of meetings and bickering beforehand for us to convince people to "allow" it to be removed; to decide that would be okay. But if it were never merged in the first place and we decide against, there's the end of it.

« Back to merge proposal