exclude_configs: Exclude the configurations

Message ID 20200930132623.27964-1-ummeegge@ipfire.org
State Dropped
Headers
Series exclude_configs: Exclude the configurations |

Commit Message

ummeegge Sept. 30, 2020, 1:26 p.m. UTC
  The configuration files from delivered updates of keepalived, nginx and stunnel should be excluded while updating process.

Signed-off-by: Erik Kapfer <ummeegge@ipfire.org>
---
 config/rootfiles/core/151/exclude | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

Michael Tremer Sept. 30, 2020, 1:28 p.m. UTC | #1
Hi Erik,

This should not be necessary in the Core Update. There should rather be a backup job that backs up /etc/nginx and restores it after the new version has been installed.

Do those packages not have properly functioning backup include files?

-Michael

> On 30 Sep 2020, at 14:26, Erik Kapfer <ummeegge@ipfire.org> wrote:
> 
> The configuration files from delivered updates of keepalived, nginx and stunnel should be excluded while updating process.
> 
> Signed-off-by: Erik Kapfer <ummeegge@ipfire.org>
> ---
> config/rootfiles/core/151/exclude | 3 +++
> 1 file changed, 3 insertions(+)
> 
> diff --git a/config/rootfiles/core/151/exclude b/config/rootfiles/core/151/exclude
> index e7500a03d..f68a02e4b 100644
> --- a/config/rootfiles/core/151/exclude
> +++ b/config/rootfiles/core/151/exclude
> @@ -8,10 +8,13 @@ etc/ipsec.conf
> etc/ipsec.secrets
> etc/ipsec.user.conf
> etc/ipsec.user.secrets
> +etc/keepalived/keepalived.conf
> etc/localtime
> +etc/nginx
> etc/shadow
> etc/snort/snort.conf
> etc/ssl/openssl.cnf
> +etc/stunnel/stunnel.conf
> etc/sudoers
> etc/sysconfig/firewall.local
> etc/sysconfig/rc.local
> -- 
> 2.12.2
>
  
ummeegge Sept. 30, 2020, 2:59 p.m. UTC | #2
Am Mittwoch, den 30.09.2020, 14:28 +0100 schrieb Michael Tremer:
> Hi Erik,
> 
> This should not be necessary in the Core Update. There should rather
> be a backup job that backs up /etc/nginx and restores it after the
> new version has been installed.
> 
> Do those packages not have properly functioning backup include files?
Yes they have, you are absolutely right. Patch is not needed... Sorry
for that noise.

> 
> -Michael
> 
> > On 30 Sep 2020, at 14:26, Erik Kapfer <ummeegge@ipfire.org> wrote:
> > 
> > The configuration files from delivered updates of keepalived, nginx
> > and stunnel should be excluded while updating process.
> > 
> > Signed-off-by: Erik Kapfer <ummeegge@ipfire.org>
> > ---
> > config/rootfiles/core/151/exclude | 3 +++
> > 1 file changed, 3 insertions(+)
> > 
> > diff --git a/config/rootfiles/core/151/exclude
> > b/config/rootfiles/core/151/exclude
> > index e7500a03d..f68a02e4b 100644
> > --- a/config/rootfiles/core/151/exclude
> > +++ b/config/rootfiles/core/151/exclude
> > @@ -8,10 +8,13 @@ etc/ipsec.conf
> > etc/ipsec.secrets
> > etc/ipsec.user.conf
> > etc/ipsec.user.secrets
> > +etc/keepalived/keepalived.conf
> > etc/localtime
> > +etc/nginx
> > etc/shadow
> > etc/snort/snort.conf
> > etc/ssl/openssl.cnf
> > +etc/stunnel/stunnel.conf
> > etc/sudoers
> > etc/sysconfig/firewall.local
> > etc/sysconfig/rc.local
> > -- 
> > 2.12.2
> >
  

Patch

diff --git a/config/rootfiles/core/151/exclude b/config/rootfiles/core/151/exclude
index e7500a03d..f68a02e4b 100644
--- a/config/rootfiles/core/151/exclude
+++ b/config/rootfiles/core/151/exclude
@@ -8,10 +8,13 @@  etc/ipsec.conf
 etc/ipsec.secrets
 etc/ipsec.user.conf
 etc/ipsec.user.secrets
+etc/keepalived/keepalived.conf
 etc/localtime
+etc/nginx
 etc/shadow
 etc/snort/snort.conf
 etc/ssl/openssl.cnf
+etc/stunnel/stunnel.conf
 etc/sudoers
 etc/sysconfig/firewall.local
 etc/sysconfig/rc.local