[PATCH] Add missing version bumps for RHEL7 command control map (#1032738)

Vratislav Podzimek vpodzime at redhat.com
Mon Nov 25 08:05:02 UTC 2013


On Wed, 2013-11-20 at 18:24 +0100, Martin Kolman wrote:
> Looks like the F20 versions for VolGroup and FireWall
The name of the command is "Firewall"

> were not added to the RHEL7 control map.
> While not an issue for Anaconda, which refers directly
> to a specific command class version, it could
> be an issue elsewhere (kickstart validation, etc.).
> 
> Signed-off-by: Martin Kolman <mkolman at redhat.com>
> ---
>  pykickstart/handlers/control.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/pykickstart/handlers/control.py b/pykickstart/handlers/control.py
> index 58188d4..2dd0709 100644
> --- a/pykickstart/handlers/control.py
> +++ b/pykickstart/handlers/control.py
> @@ -1329,7 +1329,7 @@ commandMap = {
>          "driverdisk": driverdisk.F14_DriverDisk,
>          "eula": eula.F20_Eula,
>          "fcoe": fcoe.F13_Fcoe,
> -        "firewall": firewall.F14_Firewall,
> +        "firewall": firewall.F20_Firewall,
>          "firstboot": firstboot.FC3_Firstboot,
>          "graphical": displaymode.FC3_DisplayMode,
>          "group": group.F12_Group,
> @@ -1371,7 +1371,7 @@ commandMap = {
>          "url": url.F18_Url,
>          "user": user.F19_User,
>          "vnc": vnc.F9_Vnc,
> -        "volgroup": volgroup.FC16_VolGroup,
> +        "volgroup": volgroup.F20_VolGroup,
Other than that neatpick, this looks good to me.

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic



More information about the anaconda-patches mailing list