Remove some unused files, fix an import which means we don't need to
import from ipaserver, fix up Requires so it should work better running
on a different box than the IPA server.
rob
>From 9b04f60d3d0b0f2a28d1f88311e1aba815e188b9 Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcrit...@redhat.com>
Date: Tue, 6 May 2014 15:52:11 -0400
Subject: [PATCH] Clean up Smartproxy support, drop unused code
Drop the logrotate file because Apache manages the logs
Drop the systemd configuration because we run in Apache
Import json_encode_binary from ipalib
Fix Requires
---
freeipa.spec.in | 16 ++--------------
smartproxy/Makefile.am | 8 --------
smartproxy/ipa-smartproxy.logrotate | 11 -----------
smartproxy/ipa-smartproxy.py | 2 +-
smartproxy/ipa-smartproxy.service | 12 ------------
5 files changed, 3 insertions(+), 46 deletions(-)
delete mode 100644 smartproxy/ipa-smartproxy.logrotate
delete mode 100644 smartproxy/ipa-smartproxy.service
diff --git a/freeipa.spec.in b/freeipa.spec.in
index 4e3fd7351757be773fae0b02c55549910c5b37ad..68812ee350d645164b02664e4ea51d98c2454a2a 100644
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -81,7 +81,6 @@ BuildRequires: libunistring-devel
BuildRequires: python-lesscpy
BuildRequires: python-kerberos
BuildRequires: python-cherrypy
-BuildRequires: python-requests
# Find out Kerberos middle version to infer ABI changes in DAL driver
# We cannot load DAL driver into KDC with wrong ABI.
@@ -229,8 +228,8 @@ Group: System Environment/Base
Requires: %{name}-client = %version-%release
Requires: python-cherrypy
Requires: gssproxy >= 0.3.1
-Requires: python-requests
Requires: python-kerberos >= 1.1-14
+Requires: mod_wsgi
%description server-foreman-smartproxy
A Foreman-compatible REST API for managing hosts and hostgroups.
@@ -475,7 +474,6 @@ touch %{buildroot}%{_libdir}/krb5/plugins/libkrb5/winbind_krb5_locator.so
mkdir -p %{buildroot}%{_unitdir}
install -m 644 init/systemd/ipa.service %{buildroot}%{_unitdir}/ipa.service
install -m 644 init/systemd/ipa_memcached.service %{buildroot}%{_unitdir}/ipa_memcached.service
-install -m 644 smartproxy/ipa-smartproxy.service %{buildroot}%{_unitdir}/ipa-smartproxy.service
# END
mkdir -p %{buildroot}/%{_localstatedir}/lib/ipa/backup
%endif # ONLY_CLIENT
@@ -532,8 +530,6 @@ if [ $1 = 0 ]; then
# NOTE: systemd specific section
/bin/systemctl --quiet stop ipa.service || :
/bin/systemctl --quiet disable ipa.service || :
- /bin/systemctl --quiet stop ipa-smartproxy.service || :
- /bin/systemctl --quiet disable ipa-smartproxy.service || :
# END
fi
@@ -570,15 +566,9 @@ if [ $1 -eq 0 ]; then
%{_sbindir}/update-alternatives --remove winbind_krb5_locator.so /dev/null
fi
-%preun server-foreman-smartproxy
-if [ $1 = 0 ]; then
- /bin/systemctl --quiet disable ipa-smartproxy.service || :
-fi
-
%post server-foreman-smartproxy
if [ $1 -gt 1 ] ; then
- /bin/systemctl --system daemon-reload 2>&1 || :
- /bin/systemctl condrestart ipa-smartproxy.service 2>&1 || :
+ /bin/systemctl try-restart httpd.service >/dev/null 2>&1 || :
fi
%endif # ONLY_CLIENT
@@ -821,9 +811,7 @@ fi
%{_usr}/share/ipa/smartproxy/ipa-smartproxy.py*
%{_mandir}/man1/ipa-smartproxy.1.gz
%{_mandir}/man5/ipa-smartproxy.conf.5.gz
-%attr(644,root,root) %{_unitdir}/ipa-smartproxy.service
%config(noreplace) %{_sysconfdir}/ipa/ipa-smartproxy.conf
-%config(noreplace)%{_sysconfdir}/logrotate.d/ipa-smartproxy
%endif # ONLY_CLIENT
diff --git a/smartproxy/Makefile.am b/smartproxy/Makefile.am
index f79aecfc303aac12d771170badea1eaf952b7ecf..c0994cc37c9b71552a12981a5e8b42082cca7db4 100644
--- a/smartproxy/Makefile.am
+++ b/smartproxy/Makefile.am
@@ -14,11 +14,6 @@ app_DATA = \
ipa-smartproxy.py \
$(NULL)
-rotatedir = $(LOGROTATE_DIR)
-rotate_DATA = \
- ipa-smartproxy.logrotate \
- $(NULL)
-
SUBDIRS = \
man \
$(NULL)
@@ -40,6 +35,3 @@ MAINTAINERCLEANFILES = \
missing \
Makefile.in \
$(NULL)
-
-install-data-hook:
- mv $(DESTDIR)/$(LOGROTATE_DIR)/ipa-smartproxy.logrotate $(DESTDIR)/$(LOGROTATE_DIR)/ipa-smartproxy
diff --git a/smartproxy/ipa-smartproxy.logrotate b/smartproxy/ipa-smartproxy.logrotate
deleted file mode 100644
index 12e25164cd5dea23a219422e20ea07c03a6c9a8e..0000000000000000000000000000000000000000
--- a/smartproxy/ipa-smartproxy.logrotate
+++ /dev/null
@@ -1,11 +0,0 @@
-/var/log/ipa-smartproxy.access /var/log/ipa-smartproxy.errors {
- weekly
- missingok
- notifempty
- sharedscripts
- rotate 52
- compress
- postrotate
- /bin/systemctl reload ipa-smartproxy.service > /dev/null 2>/dev/null || true
- endscript
-}
diff --git a/smartproxy/ipa-smartproxy.py b/smartproxy/ipa-smartproxy.py
index 23788ecf514b9335955c3b86399e9bb6558ef024..453796a5d3c21324ad21bb3d03521c82fee1c6e4 100644
--- a/smartproxy/ipa-smartproxy.py
+++ b/smartproxy/ipa-smartproxy.py
@@ -31,7 +31,7 @@ from cherrypy import response
from ipalib import api
from ipalib import errors
from ipalib.request import context
-from ipaserver.rpcserver import json_encode_binary
+from ipalib.rpc import json_encode_binary
from ipapython.version import VERSION, API_VERSION
from ipapython.ipa_log_manager import root_logger
diff --git a/smartproxy/ipa-smartproxy.service b/smartproxy/ipa-smartproxy.service
deleted file mode 100644
index 544f55b1369943c3919f5cb61a9f3bc2d784043d..0000000000000000000000000000000000000000
--- a/smartproxy/ipa-smartproxy.service
+++ /dev/null
@@ -1,12 +0,0 @@
-[Unit]
-Description=IPA smartproxy for Foreman
-After=ipa.target
-
-[Service]
-ExecStart=/usr/sbin/ipa-smartproxy -c /etc/ipa/ipa-smartproxy.conf -d -p /var/run/ipa-smartproxy.pid
-ExecReload=/bin/kill -USR1 $MAINPID
-Type=forking
-PIDFile=/var/run/ipa-smartproxy.pid
-
-[Install]
-WantedBy=multi-user.target
--
1.9.0
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel