[PATCH rhel7-branch] Take "RHEL Atomic Host" as rhel installclass.

Chris Lumens clumens at redhat.com
Tue Oct 7 14:03:56 UTC 2014


> In near future we'll want to configure different default autopartitioning for
> Atomic. I'm inclined to do it by updating fedora and rhel installclasses for
> atomic case. If we wanted to go more classy, perhaps using mixin would be ok.
> Maybe we'll need to rework installclasses for other new products anyway, Seems
> like a place deserving clever cleanup.

Installclasses may also inherit from each other as classes ought to, so
maybe that would also be helpful.

> diff --git a/pyanaconda/installclasses/rhel.py b/pyanaconda/installclasses/rhel.py
> index b1e4375..de5608d 100644
> --- a/pyanaconda/installclasses/rhel.py
> +++ b/pyanaconda/installclasses/rhel.py
> @@ -29,7 +29,7 @@ class InstallClass(BaseInstallClass):
>      id = "rhel"
>      name = N_("Red Hat Enterprise Linux")
>      sortPriority = 20000
> -    if not productName.startswith("Red Hat "):
> +    if not productName.startswith(("Red Hat ", "RHEL Atomic")):
>          hidden = 1
>      defaultFS = "xfs"

Huh, I had no idea you could do that.  ACK.

- Chris


More information about the anaconda-patches mailing list