Code review comment for lp:~rvb/maas-test/only-trusty

Revision history for this message
Julian Edwards (julian-edwards) wrote :

Thanks for making this change. I think it's fine, but I do have a query in [2].

[1]

11 + default=utils.get_supported_maas_series()[0],

Unexplained indexes like this are a red flag. I think it would be better to just hard-code "trusty" here.

[2]

71 + These are the series that MAAS can deploy on the nodes
72 + it manages.
73 + """
74 + return _get_supported_series('precise')

Why is trusty not included here?

review: Needs Information

« Back to merge proposal