[pykickstart][PATCH] Add --remove-service option for the firewall command (#957809)

Vratislav Podzimek vpodzime at redhat.com
Mon Oct 7 08:52:57 UTC 2013


On Fri, 2013-10-04 at 18:07 +0200, Martin Kolman wrote:
> The --remove-service option makes it possible to remove services
> that were enabled on the firewall.
> 
> Signed-off-by: Martin Kolman <mkolman at redhat.com>
> ---
>  pykickstart/commands/firewall.py | 42 ++++++++++++++++++++++++++++++++++++++++
>  pykickstart/handlers/control.py  |  2 +-
>  tests/commands/firewall.py       | 10 ++++++++++
>  3 files changed, 53 insertions(+), 1 deletion(-)
> 
> diff --git a/pykickstart/commands/firewall.py b/pykickstart/commands/firewall.py
> index 664bea6..1384d63 100644
> --- a/pykickstart/commands/firewall.py
> +++ b/pykickstart/commands/firewall.py
> @@ -191,3 +191,45 @@ class F14_Firewall(F10_Firewall):
>          op = F10_Firewall._getParser(self)
>          op.remove_option("--telnet")
>          return op
> +
> +class F20_Firewall(F14_Firewall):
> +    removedKeywords = F14_Firewall.removedKeywords
> +    removedAttrs = F14_Firewall.removedAttrs
I think you don't need to do this, these two attributes get inherited.

-- 
Vratislav Podzimek

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



More information about the anaconda-patches mailing list