From be396339ba976be372c344a220e1efdf9c3d8b45 Mon Sep 17 00:00:00 2001
From: Pierre-Louis Bonicoli <pierre-louis@libregerbil.fr>
Date: Wed, 24 Aug 2016 14:09:00 +0200
Subject: [PATCH] Install configuration files using RPM instead of pip
 packaging

pbr doesn't handle configuration files:
http://lists.openstack.org/pipermail/openstack-dev/2015-September/075350.html
---
 lago.spec.in | 18 +++++++++++++++---
 setup.cfg    |  6 ------
 2 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/lago.spec.in b/lago.spec.in
index ddc0eb7d..ffabca51 100644
--- a/lago.spec.in
+++ b/lago.spec.in
@@ -27,12 +27,22 @@ install -d -m 755 %{buildroot}/%{_sysconfdir}/lago
 PYTHONPATH="$PYTHONPATH:%{buildroot}/%{python_sitelib}"\
              %{buildroot}/usr/bin/lago --ignore-warnings generate-config > "%{buildroot}/%{_sysconfdir}/lago/lago.conf"
 sed -i 's/^\([^#]\)\(.*\)/#\0/' "%{buildroot}/%{_sysconfdir}/lago/lago.conf"
+chmod 0644 "%{buildroot}/%{_sysconfdir}/lago/lago.conf"
 
 install -d %{buildroot}/var/lib/lago/subnets
 install -d %{buildroot}/var/lib/lago/store
 install -d %{buildroot}/var/lib/lago/repos
+
+install -d -m 755 %{buildroot}%{_sysconfdir}/polkit-1/localauthority/50-local.d
+install -d -m 755 %{buildroot}%{_sysconfdir}/sudoers.d
+
+install -p -D -m 644 etc/polkit/*.pkla %{buildroot}%{_sysconfdir}/polkit-1/localauthority/50-local.d/
+install -p -D -m 644 etc/sudo/* %{buildroot}%{_sysconfdir}/sudoers.d/
+
 # this is for python-lago-ovirt
 install -d %{buildroot}/var/lib/lago/reposync
+install -d -m 755 %{buildroot}%{_sysconfdir}/firewalld/services
+install -p -D -m 644 etc/firewalld/services/* %{buildroot}%{_sysconfdir}/firewalld/services/
 
 %files
 
@@ -104,11 +114,13 @@ Requires: sudo
 %{python2_sitelib}/%{name}_template_repo/*.py*
 %{python2_sitelib}/%{name}/*.xml
 %{python2_sitelib}/%{name}-%{version}-py*.egg-info
-%{_sysconfdir}/polkit-1/localauthority/50-local.d/
 %{_bindir}/lagocli
 %{_bindir}/lago
-%attr(0775, root, root) %{_sysconfdir}/sudoers.d/*
+
 %config(noreplace) %{_sysconfdir}/lago/lago.conf
+%config(noreplace) %{_sysconfdir}/polkit-1/localauthority/50-local.d/*
+%config(noreplace) %{_sysconfdir}/sudoers.d/*
+
 %dir %attr(2775, root, lago) /var/lib/lago/
 %dir %attr(2775, root, lago) /var/lib/lago/subnets/
 %dir %attr(2775, root, lago) /var/lib/lago/store/
@@ -159,7 +171,7 @@ Requires: xz
 
 %files -n python-%{name}-ovirt
 %{python2_sitelib}/ovirt%{name}/*.py*
-%{_sysconfdir}/firewalld/services/*
+%config(noreplace) %{_sysconfdir}/firewalld/services/*
 %dir %attr(2775, root, lago) /var/lib/lago/reposync/
 
 %post -n python-%{name}-ovirt
diff --git a/setup.cfg b/setup.cfg
index 2df85155..fca74ee4 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -13,12 +13,6 @@ packages =
     lago.plugins
     lago_template_repo
     ovirtlago
-data_files =
-# core lago
-    /etc/sudoers.d/ = etc/sudo/lago
-    /etc/polkit-1/localauthority/50-local.d = etc/polkit/*
-# ovirt lago plugin
-    /etc/firewalld/services = etc/firewalld/services/*
 
 [flake8]
 # ignore line break before operator