public inbox for [email protected]  
help / color / mirror / Atom feed
Bug in repmgr3 package
10+ messages / 4 participants
[nested] [flat]

* Bug in repmgr3 package
@ 2015-11-03 09:54  Guillaume Lelarge <[email protected]>
  0 siblings, 1 reply; 10+ messages in thread

From: Guillaume Lelarge @ 2015-11-03 09:54 UTC (permalink / raw)
  To: pgsql-pkg-yum

Hi,

When installing repmgr3 package on a CentOS 6 64 bits, I always end up with
an error at postinstall saying that /var/run/repmgr doesn't exist.

Here is a patch that will hopefully fix that. (The patch deals with all
distro for 9.4 release. I didn't check other releases.)

Regards.


-- 
Guillaume.
  http://blog.guillaume.lelarge.info
  http://www.dalibo.com


-- 
Sent via pgsql-pkg-yum mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-pkg-yum


Attachments:

  [text/x-patch] repmgr.patch (3.7K, 3-repmgr.patch)
  download | inline diff:
diff --git a/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec b/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec b/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec b/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-20/repmgr.spec b/rpm/redhat/9.4/repmgr/F-20/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/F-20/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-20/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-21/repmgr.spec b/rpm/redhat/9.4/repmgr/F-21/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/F-21/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-21/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-22/repmgr.spec b/rpm/redhat/9.4/repmgr/F-22/repmgr.spec
index aba8b39..5ae706f 100644
--- a/rpm/redhat/9.4/repmgr/F-22/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-22/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -r repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-23/repmgr.spec b/rpm/redhat/9.4/repmgr/F-23/repmgr.spec
index aba8b39..5ae706f 100644
--- a/rpm/redhat/9.4/repmgr/F-23/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-23/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -r repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 


^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-03 14:42  Guillaume Lelarge <[email protected]>
  parent: Guillaume Lelarge <[email protected]>
  0 siblings, 1 reply; 10+ messages in thread

From: Guillaume Lelarge @ 2015-11-03 14:42 UTC (permalink / raw)
  To: pgsql-pkg-yum

Hey,

2015-11-03 10:54 GMT+01:00 Guillaume Lelarge <[email protected]>:

> Hi,
>
> When installing repmgr3 package on a CentOS 6 64 bits, I always end up
> with an error at postinstall saying that /var/run/repmgr doesn't exist.
>
> Here is a patch that will hopefully fix that. (The patch deals with all
> distro for 9.4 release. I didn't check other releases.)
>
>
I worked a bit more with repmgr. I found that the initscript doesn't
respect the $user variable all the time. Hence this new patch (which
includes the changes from the first patch).

I'm also wondering why we do have this new OS user (repmgr). It seems to be
more a burden than anything else. Actually, they have a much better
initscript in upstream that makes use the postgres OS user. The RPM package
should embrace that change.

Regards.


-- 
Guillaume.
  http://blog.guillaume.lelarge.info
  http://www.dalibo.com


-- 
Sent via pgsql-pkg-yum mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-pkg-yum


Attachments:

  [text/x-patch] repmgr-v2.patch (8.0K, 3-repmgr-v2.patch)
  download | inline diff:
diff --git a/rpm/redhat/9.4/repmgr/EL-5/repmgr.init b/rpm/redhat/9.4/repmgr/EL-5/repmgr.init
index c900170..95e3757 100644
--- a/rpm/redhat/9.4/repmgr/EL-5/repmgr.init
+++ b/rpm/redhat/9.4/repmgr/EL-5/repmgr.init
@@ -54,7 +54,7 @@ start(){
         mkdir -p $(dirname $pidfile)
         chown $user: $(dirname $pidfile)
 
-	su -l repmgr -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
+	su -l $user -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
         retval=$?
 	[ $retval -eq 0 ] && success 
         echo
diff --git a/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec b/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/EL-5/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/EL-6/repmgr.init b/rpm/redhat/9.4/repmgr/EL-6/repmgr.init
index c900170..95e3757 100644
--- a/rpm/redhat/9.4/repmgr/EL-6/repmgr.init
+++ b/rpm/redhat/9.4/repmgr/EL-6/repmgr.init
@@ -54,7 +54,7 @@ start(){
         mkdir -p $(dirname $pidfile)
         chown $user: $(dirname $pidfile)
 
-	su -l repmgr -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
+	su -l $user -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
         retval=$?
 	[ $retval -eq 0 ] && success 
         echo
diff --git a/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec b/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/EL-6/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/EL-7/repmgr.init b/rpm/redhat/9.4/repmgr/EL-7/repmgr.init
index c900170..95e3757 100644
--- a/rpm/redhat/9.4/repmgr/EL-7/repmgr.init
+++ b/rpm/redhat/9.4/repmgr/EL-7/repmgr.init
@@ -54,7 +54,7 @@ start(){
         mkdir -p $(dirname $pidfile)
         chown $user: $(dirname $pidfile)
 
-	su -l repmgr -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
+	su -l $user -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
         retval=$?
 	[ $retval -eq 0 ] && success 
         echo
diff --git a/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec b/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/EL-7/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-20/repmgr.init b/rpm/redhat/9.4/repmgr/F-20/repmgr.init
index c900170..95e3757 100644
--- a/rpm/redhat/9.4/repmgr/F-20/repmgr.init
+++ b/rpm/redhat/9.4/repmgr/F-20/repmgr.init
@@ -54,7 +54,7 @@ start(){
         mkdir -p $(dirname $pidfile)
         chown $user: $(dirname $pidfile)
 
-	su -l repmgr -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
+	su -l $user -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
         retval=$?
 	[ $retval -eq 0 ] && success 
         echo
diff --git a/rpm/redhat/9.4/repmgr/F-20/repmgr.spec b/rpm/redhat/9.4/repmgr/F-20/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/F-20/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-20/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-21/repmgr.init b/rpm/redhat/9.4/repmgr/F-21/repmgr.init
index c900170..95e3757 100644
--- a/rpm/redhat/9.4/repmgr/F-21/repmgr.init
+++ b/rpm/redhat/9.4/repmgr/F-21/repmgr.init
@@ -54,7 +54,7 @@ start(){
         mkdir -p $(dirname $pidfile)
         chown $user: $(dirname $pidfile)
 
-	su -l repmgr -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
+	su -l $user -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
         retval=$?
 	[ $retval -eq 0 ] && success 
         echo
diff --git a/rpm/redhat/9.4/repmgr/F-21/repmgr.spec b/rpm/redhat/9.4/repmgr/F-21/repmgr.spec
index aba8b39..29c629a 100644
--- a/rpm/redhat/9.4/repmgr/F-21/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-21/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -R repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-22/repmgr.init b/rpm/redhat/9.4/repmgr/F-22/repmgr.init
index c900170..95e3757 100644
--- a/rpm/redhat/9.4/repmgr/F-22/repmgr.init
+++ b/rpm/redhat/9.4/repmgr/F-22/repmgr.init
@@ -54,7 +54,7 @@ start(){
         mkdir -p $(dirname $pidfile)
         chown $user: $(dirname $pidfile)
 
-	su -l repmgr -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
+	su -l $user -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
         retval=$?
 	[ $retval -eq 0 ] && success 
         echo
diff --git a/rpm/redhat/9.4/repmgr/F-22/repmgr.spec b/rpm/redhat/9.4/repmgr/F-22/repmgr.spec
index aba8b39..5ae706f 100644
--- a/rpm/redhat/9.4/repmgr/F-22/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-22/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -r repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 
diff --git a/rpm/redhat/9.4/repmgr/F-23/repmgr.init b/rpm/redhat/9.4/repmgr/F-23/repmgr.init
index c900170..95e3757 100644
--- a/rpm/redhat/9.4/repmgr/F-23/repmgr.init
+++ b/rpm/redhat/9.4/repmgr/F-23/repmgr.init
@@ -54,7 +54,7 @@ start(){
         mkdir -p $(dirname $pidfile)
         chown $user: $(dirname $pidfile)
 
-	su -l repmgr -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
+	su -l $user -c "$exec -f $REPMGRDCONF -p $pidfile -d ->> $REPMGRDLOG 2>&1" $user # --monitoring-history --verbose
         retval=$?
 	[ $retval -eq 0 ] && success 
         echo
diff --git a/rpm/redhat/9.4/repmgr/F-23/repmgr.spec b/rpm/redhat/9.4/repmgr/F-23/repmgr.spec
index aba8b39..5ae706f 100644
--- a/rpm/redhat/9.4/repmgr/F-23/repmgr.spec
+++ b/rpm/redhat/9.4/repmgr/F-23/repmgr.spec
@@ -115,7 +115,11 @@ fi
 # This adds the proper /etc/rc*.d links for the script
 /sbin/chkconfig --add %{sname}-%{pgpackageversion}
 %endif
-%{__chown} repmgr: %{_localstatedir}/run/%{sname}
+if [ ! -x %{_varrundir} ]
+then
+	%{__mkdir} -m 700 %{_varrundir}
+	%{__chown} -r repmgr: %{_varrundir}
+fi
 
 %postun -p /sbin/ldconfig
 


^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-09 09:19  Devrim GÜNDÜZ <[email protected]>
  parent: Guillaume Lelarge <[email protected]>
  0 siblings, 2 replies; 10+ messages in thread

From: Devrim GÜNDÜZ @ 2015-11-09 09:19 UTC (permalink / raw)
  To: Guillaume Lelarge <[email protected]>; pgsql-pkg-yum



Hi Guillaume,

Thanks for the patch!

I committed your patch:

http://git.postgresql.org/gitweb/?p=pgrpms.git;a=commit;h=0cdf38c03aebb
45a40aa5895cf0753237320f164

 I also changed the username from repmgr to postgres. I pushed this
change only to 9.5 for now. Will push and test packages today, and then
I'll update the remaining branches.

Regards, Devrim

On Tue, 2015-11-03 at 15:42 +0100, Guillaume Lelarge wrote:
> Hey,
> 
> 2015-11-03 10:54 GMT+01:00 Guillaume Lelarge <[email protected]>
> :
> 
> > Hi,
> > 
> > When installing repmgr3 package on a CentOS 6 64 bits, I always end
> > up
> > with an error at postinstall saying that /var/run/repmgr doesn't
> > exist.
> > 
> > Here is a patch that will hopefully fix that. (The patch deals with
> > all
> > distro for 9.4 release. I didn't check other releases.)
> > 
> > 
> I worked a bit more with repmgr. I found that the initscript doesn't
> respect the $user variable all the time. Hence this new patch (which
> includes the changes from the first patch).
> 
> I'm also wondering why we do have this new OS user (repmgr). It seems
> to be
> more a burden than anything else. Actually, they have a much better
> initscript in upstream that makes use the postgres OS user. The RPM
> package
> should embrace that change.
> 
> Regards.
> 
> 



-- 
Devrim GÜNDÜZ
Principal Systems Engineer @ EnterpriseDB: http://www.enterprisedb.com
PostgreSQL Danışmanı/Consultant, Red Hat Certified Engineer
Twitter: @DevrimGunduz , @DevrimGunduzTR




Attachments:

  [application/pgp-signature] signature.asc (819B, 2-signature.asc)
  download

^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-09 13:02  Guillaume Lelarge <[email protected]>
  parent: Devrim GÜNDÜZ <[email protected]>
  1 sibling, 0 replies; 10+ messages in thread

From: Guillaume Lelarge @ 2015-11-09 13:02 UTC (permalink / raw)
  To: Devrim Gündüz <[email protected]>; +Cc: pgsql-pkg-yum

Le 9 nov. 2015 10:19 AM, "Devrim GÜNDÜZ" <[email protected]> a écrit :
>
>
>
> Hi Guillaume,
>
> Thanks for the patch!
>
> I committed your patch:
>
> http://git.postgresql.org/gitweb/?p=pgrpms.git;a=commit;h=0cdf38c03aebb
> 45a40aa5895cf0753237320f164
>
>  I also changed the username from repmgr to postgres. I pushed this
> change only to 9.5 for now. Will push and test packages today, and then
> I'll update the remaining branches.
>

Great, thanks Devrim.

> Regards, Devrim
>
> On Tue, 2015-11-03 at 15:42 +0100, Guillaume Lelarge wrote:
> > Hey,
> >
> > 2015-11-03 10:54 GMT+01:00 Guillaume Lelarge <[email protected]>
> > :
> >
> > > Hi,
> > >
> > > When installing repmgr3 package on a CentOS 6 64 bits, I always end
> > > up
> > > with an error at postinstall saying that /var/run/repmgr doesn't
> > > exist.
> > >
> > > Here is a patch that will hopefully fix that. (The patch deals with
> > > all
> > > distro for 9.4 release. I didn't check other releases.)
> > >
> > >
> > I worked a bit more with repmgr. I found that the initscript doesn't
> > respect the $user variable all the time. Hence this new patch (which
> > includes the changes from the first patch).
> >
> > I'm also wondering why we do have this new OS user (repmgr). It seems
> > to be
> > more a burden than anything else. Actually, they have a much better
> > initscript in upstream that makes use the postgres OS user. The RPM
> > package
> > should embrace that change.
> >
> > Regards.
> >
> >
>
>
>
> --
> Devrim GÜNDÜZ
> Principal Systems Engineer @ EnterpriseDB: http://www.enterprisedb.com
> PostgreSQL Danışmanı/Consultant, Red Hat Certified Engineer
> Twitter: @DevrimGunduz , @DevrimGunduzTR
>
>


^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-09 15:51  Jeff Frost <[email protected]>
  parent: Devrim GÜNDÜZ <[email protected]>
  1 sibling, 1 reply; 10+ messages in thread

From: Jeff Frost @ 2015-11-09 15:51 UTC (permalink / raw)
  To: Devrim GÜNDÜZ <[email protected]>; +Cc: Guillaume Lelarge <[email protected]>; pgsql-pkg-yum



> On Nov 9, 2015, at 01:19, Devrim GÜNDÜZ <[email protected]> wrote:
> 
>  I also changed the username from repmgr to postgres. I pushed this
> change only to 9.5 for now. Will push and test packages today, and then
> I'll update the remaining branches.

Note that I haven't backed out the version bump on the 9.2 and older versions yet. 

-- 
Sent via pgsql-pkg-yum mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-pkg-yum



^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-09 22:21  Devrim Gündüz <[email protected]>
  parent: Jeff Frost <[email protected]>
  0 siblings, 1 reply; 10+ messages in thread

From: Devrim Gündüz @ 2015-11-09 22:21 UTC (permalink / raw)
  To: Jeff Frost <[email protected]>; +Cc: Guillaume Lelarge <[email protected]>; pgsql-pkg-yum

Can you do it until tomorrow evening?

On November 9, 2015 3:51:46 PM GMT+00:00, Jeff Frost <[email protected]> wrote:
>
>
>> On Nov 9, 2015, at 01:19, Devrim GÜNDÜZ <[email protected]> wrote:
>> 
>>  I also changed the username from repmgr to postgres. I pushed this
>> change only to 9.5 for now. Will push and test packages today, and
>then
>> I'll update the remaining branches.
>
>Note that I haven't backed out the version bump on the 9.2 and older
>versions yet. 

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-09 22:21  Jeff Frost <[email protected]>
  parent: Devrim Gündüz <[email protected]>
  0 siblings, 1 reply; 10+ messages in thread

From: Jeff Frost @ 2015-11-09 22:21 UTC (permalink / raw)
  To: Devrim GÜNDÜZ <[email protected]>; +Cc: Guillaume Lelarge <[email protected]>; pgsql-pkg-yum

Do you mean wait to do it until or …?

> On Nov 9, 2015, at 2:21 PM, Devrim Gündüz <[email protected]> wrote:
> 
> Can you do it until tomorrow evening?
> 
> On November 9, 2015 3:51:46 PM GMT+00:00, Jeff Frost <[email protected]> wrote:
> 
> 
>  On Nov 9, 2015, at 01:19, Devrim GÜNDÜZ <[email protected]> wrote:
>  
>   I also changed the username from repmgr to postgres. I pushed this
>  change only to 9.5 for now. Will push and test packages today, and then
>  I'll update the remaining branches.
> 
> Note that I haven't backed out the version bump on the 9.2 and older versions yet. 



^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-09 22:41  Devrim Gündüz <[email protected]>
  parent: Jeff Frost <[email protected]>
  0 siblings, 1 reply; 10+ messages in thread

From: Devrim Gündüz @ 2015-11-09 22:41 UTC (permalink / raw)
  To: Jeff Frost <[email protected]>; +Cc: Guillaume Lelarge <[email protected]>; pgsql-pkg-yum

Can you revert it today or tomorrow? :)

On November 9, 2015 10:21:59 PM GMT+00:00, Jeff Frost <[email protected]> wrote:
>Do you mean wait to do it until or …?
>
>> On Nov 9, 2015, at 2:21 PM, Devrim Gündüz <[email protected]> wrote:
>> 
>> Can you do it until tomorrow evening?
>> 
>> On November 9, 2015 3:51:46 PM GMT+00:00, Jeff Frost
><[email protected]> wrote:
>> 
>> 
>>  On Nov 9, 2015, at 01:19, Devrim GÜNDÜZ <[email protected]> wrote:
>>  
>>   I also changed the username from repmgr to postgres. I pushed this
>>  change only to 9.5 for now. Will push and test packages today, and
>then
>>  I'll update the remaining branches.
>> 
>> Note that I haven't backed out the version bump on the 9.2 and older
>versions yet. 

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-09 22:45  Jeff Frost <[email protected]>
  parent: Devrim Gündüz <[email protected]>
  0 siblings, 1 reply; 10+ messages in thread

From: Jeff Frost @ 2015-11-09 22:45 UTC (permalink / raw)
  To: Devrim GÜNDÜZ <[email protected]>; +Cc: Guillaume Lelarge <[email protected]>; pgsql-pkg-yum


> On Nov 9, 2015, at 2:41 PM, Devrim Gündüz <[email protected]> wrote:
> 
> Can you revert it today or tomorrow? :)

Can’t revert it because it’s all one big commit (at least not with my current level of git-fu), but I should be able to change them back today or tomorrow.

-- 
Sent via pgsql-pkg-yum mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-pkg-yum



^ permalink  raw  reply  [nested|flat] 10+ messages in thread

* Re: Bug in repmgr3 package
@ 2015-11-10 06:29  Jeff Frost <[email protected]>
  parent: Jeff Frost <[email protected]>
  0 siblings, 0 replies; 10+ messages in thread

From: Jeff Frost @ 2015-11-10 06:29 UTC (permalink / raw)
  To: Devrim GÜNDÜZ <[email protected]>; +Cc: Guillaume Lelarge <[email protected]>; pgsql-pkg-yum


> On Nov 9, 2015, at 2:45 PM, Jeff Frost <[email protected]> wrote:
> 
> 
>> On Nov 9, 2015, at 2:41 PM, Devrim Gündüz <[email protected]> wrote:
>> 
>> Can you revert it today or tomorrow? :)
> 
> Can’t revert it because it’s all one big commit (at least not with my current level of git-fu), but I should be able to change them back today or tomorrow.

Ok, I think  got the 9.[012] versions properly reverted to repmgr 2.0.2.

It appears to build properly, but I don’t have an environment to test it on.

-- 
Sent via pgsql-pkg-yum mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-pkg-yum




^ permalink  raw  reply  [nested|flat] 10+ messages in thread


end of thread, other threads:[~2015-11-10 06:29 UTC | newest]

Thread overview: 10+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2015-11-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Guillaume Lelarge <[email protected]>
2015-11-09 09:19   ` Devrim GÜNDÜZ <[email protected]>
2015-11-09 13:02     ` Guillaume Lelarge <[email protected]>
2015-11-09 15:51     ` Jeff Frost <[email protected]>
2015-11-09 22:21       ` Devrim Gündüz <[email protected]>
2015-11-09 22:21         ` Jeff Frost <[email protected]>
2015-11-09 22:41           ` Devrim Gündüz <[email protected]>
2015-11-09 22:45             ` Jeff Frost <[email protected]>
2015-11-10 06:29               ` Jeff Frost <[email protected]>

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox