Hello Gabe,

thank you for your patch!

Please note that there might be a case where detection is_ipa_configured() is
broken but the user still needs to run the uninstall process to clean it up.

Could you amend the patch to respect --force option? In that case the
detection should be skipped.

Thank you for your time!

Petr^2 Spacek

On 9.10.2015 19:17, Gabe Alford wrote:
> diff --git a/ipaserver/install/server/install.py 
> b/ipaserver/install/server/install.py
> index 
> 13a59a0e6149dc22ded4a895db02516e9360e02b..ca93e7a6fd7276d9c0d82eb6f94575730759d858
>  100644
> --- a/ipaserver/install/server/install.py
> +++ b/ipaserver/install/server/install.py
> @@ -954,6 +954,12 @@ def uninstall_check(installer):
>  
>      installer._installation_cleanup = False
>  
> +    if not is_ipa_configured():
> +        print("IPA server is not configured on this system.\n" +
> +              "If you want to install the IPA server, please install " +
> +              "it using 'ipa-server-install'.")
> +        sys.exit(1)
> +
>      fstore = sysrestore.FileStore(SYSRESTORE_DIR_PATH)
>      sstore = sysrestore.StateFile(SYSRESTORE_DIR_PATH)

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to