URL: https://github.com/freeipa/freeipa/pull/645
Author: dkupka
 Title: #645: Create temporaty directories at the begining of uninstall
Action: opened

PR body:
"""
Since commit 38c6689 temporary directories are no longer created at package
install time. Instead they're created at server install time.
Some steps in uninstall also assume that temporary direcories exist. Creating
the directories in the begining of server uninstall ensure that the uninstall
will go through.

https://pagure.io/freeipa/issue/6715
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/645/head:pr645
git checkout pr645
From aa2074218c0aa37fd9711dd6e4fd5d8e992b5084 Mon Sep 17 00:00:00 2001
From: David Kupka <dku...@redhat.com>
Date: Thu, 23 Mar 2017 12:48:06 +0100
Subject: [PATCH] Create temporaty directories at the begining of uninstall

Since commit 38c6689 temporary directories are no longer created at package
install time. Instead they're created at server install time.
Some steps in uninstall also assume that temporary direcories exist. Creating
the directories in the begining of server uninstall ensure that the uninstall
will go through.

https://pagure.io/freeipa/issue/6715
---
 ipaserver/install/server/install.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/ipaserver/install/server/install.py b/ipaserver/install/server/install.py
index de6b5b3..d7eb0bf 100644
--- a/ipaserver/install/server/install.py
+++ b/ipaserver/install/server/install.py
@@ -1042,6 +1042,10 @@ def uninstall(installer):
 
     rv = 0
 
+    # further steps assumes that temporary directories exists so rather
+    # ensure they are created
+    tasks.create_tmpfiles_dirs()
+
     print("Shutting down all IPA services")
     try:
         services.knownservices.ipa.stop()
-- 
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