public inbox for [email protected]
help / color / mirror / Atom feedBug in repmgr3 package
10+ messages / 4 participants
[nested] [flat]
* Bug in repmgr3 package
@ 2015-11-03 09:54 Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package 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 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
@ 2015-11-03 14:42 ` Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
@ 2015-11-09 09:19 ` Devrim GÜNDÜZ <[email protected]>
2015-11-09 13:02 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 15:51 ` Re: Bug in repmgr3 package Jeff Frost <[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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[email protected]>
@ 2015-11-09 13:02 ` Guillaume Lelarge <[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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[email protected]>
@ 2015-11-09 15:51 ` Jeff Frost <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package 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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[email protected]>
2015-11-09 15:51 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
@ 2015-11-09 22:21 ` Devrim Gündüz <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package 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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[email protected]>
2015-11-09 15:51 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package Devrim Gündüz <[email protected]>
@ 2015-11-09 22:21 ` Jeff Frost <[email protected]>
2015-11-09 22:41 ` Re: Bug in repmgr3 package 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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[email protected]>
2015-11-09 15:51 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package Devrim Gündüz <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
@ 2015-11-09 22:41 ` Devrim Gündüz <[email protected]>
2015-11-09 22:45 ` Re: Bug in repmgr3 package 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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[email protected]>
2015-11-09 15:51 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package Devrim Gündüz <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
2015-11-09 22:41 ` Re: Bug in repmgr3 package Devrim Gündüz <[email protected]>
@ 2015-11-09 22:45 ` Jeff Frost <[email protected]>
2015-11-10 06:29 ` Re: Bug in repmgr3 package Jeff Frost <[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-03 09:54 Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-03 14:42 ` Re: Bug in repmgr3 package Guillaume Lelarge <[email protected]>
2015-11-09 09:19 ` Re: Bug in repmgr3 package Devrim GÜNDÜZ <[email protected]>
2015-11-09 15:51 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package Devrim Gündüz <[email protected]>
2015-11-09 22:21 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
2015-11-09 22:41 ` Re: Bug in repmgr3 package Devrim Gündüz <[email protected]>
2015-11-09 22:45 ` Re: Bug in repmgr3 package Jeff Frost <[email protected]>
@ 2015-11-10 06:29 ` 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