sts-sponsors team mailing list archive

[Adam Collard <mp+436100@xxxxxxxxxxxxxxxxxx>]

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.