Review: Needs Information

Hmm, filesystem features like this have been the reason why we enforce using 
older versions of Ubuntu to deploy certain OSs (e.g. Trusty for Centos 6).

Did you explore that?

Diff comments:

> diff --git a/src/provisioningserver/drivers/osystem/suse.py 
> b/src/provisioningserver/drivers/osystem/suse.py
> index c6b21d7..8eac7d3 100644
> --- a/src/provisioningserver/drivers/osystem/suse.py
> +++ b/src/provisioningserver/drivers/osystem/suse.py
> @@ -27,7 +37,8 @@ class SUSEOS(OperatingSystem):
>  
>      def is_release_supported(self, release):
>          """Return True when the release is supported, False otherwise."""
> -        return release in DISTRO_SERIES_CHOICES
> +        matched = DISTRO_MATCHER.match(release)
> +        return matched is not None

this can be one-lined

>  
>      def get_default_release(self):
>          """Gets the default release to use when a release is not


-- 
https://code.launchpad.net/~alexsander-souza/maas/+git/maas/+merge/436100
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp

Reply via email to