On 6 February 2013 14:38, Michael Hanselmann <[email protected]> wrote: > Disk template names shouldn't be hardcoded. > --- > qa/qa_cluster.py | 2 +- > qa/qa_instance.py | 6 ++++-- > 2 files changed, 5 insertions(+), 3 deletions(-)
LGTM, except that you skipped one instance of "drbd" and two of "plain". What about adding an assertion in QA to check that value doesn't change (since that would change the user interface too)? Bernardo
