Hi, Thanks for the recent patch.
- It fixes 'start' and 'restart'. I'm not sure about 'reload' but I couldn't make it fail, so OK :) - However 'stop' is not fixed (still async). - Also after service postgresql stop ; rm -rf /var/lib/postgresql ; service postgresql start -> 'status' will still happily declare the service 'active' These tests show the bug is not completely fixed, so I'm reopening it. Cheers! Sylvain On Tue, Oct 07, 2014 at 07:39:19PM +0000, Debian Bug Tracking System wrote: > This is an automatic notification regarding your Bug report > which was filed against the postgresql-common package: > > #759725: postgresql-common: non-synchronous service postgresql > start/stop/reload > > It has been closed by Christoph Berg <m...@debian.org>. > > Their explanation is attached below along with your original report. > If this explanation is unsatisfactory and you have not received a > better one in a separate message then please contact Christoph Berg > <m...@debian.org> by > replying to this email. > > > -- > 759725: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=759725 > Debian Bug Tracking System > Contact ow...@bugs.debian.org with problems > Date: Tue, 07 Oct 2014 19:34:49 +0000 > From: Christoph Berg <m...@debian.org> > To: 759725-cl...@bugs.debian.org > Subject: Bug#759725: fixed in postgresql-common 162 > > Source: postgresql-common > Source-Version: 162 > > We believe that the bug you reported is fixed in the latest version of > postgresql-common, which is due to be installed in the Debian FTP archive. > > A summary of the changes between this version and the previous one is > attached. > > Thank you for reporting the bug, which will now be closed. If you > have further comments please address them to 759...@bugs.debian.org, > and the maintainer will reopen the bug report if appropriate. > > Debian distribution maintenance software > pp. > Christoph Berg <m...@debian.org> (supplier of updated postgresql-common > package) > > (This message was generated automatically at their request; if you > believe that there is a problem with it please contact the archive > administrators by mailing ftpmas...@ftp-master.debian.org) > > > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA256 > > Format: 1.8 > Date: Tue, 07 Oct 2014 21:25:25 +0200 > Source: postgresql-common > Binary: postgresql-common postgresql-client-common postgresql-server-dev-all > postgresql postgresql-client postgresql-doc postgresql-contrib > Architecture: source all > Version: 162 > Distribution: unstable > Urgency: medium > Maintainer: Debian PostgreSQL Maintainers > <pkg-postgresql-pub...@lists.alioth.debian.org> > Changed-By: Christoph Berg <m...@debian.org> > Description: > postgresql - object-relational SQL database (supported version) > postgresql-client - front-end programs for PostgreSQL (supported version) > postgresql-client-common - manager for multiple PostgreSQL client versions > postgresql-common - PostgreSQL database-cluster manager > postgresql-contrib - additional facilities for PostgreSQL (supported version) > postgresql-doc - documentation for the PostgreSQL database management system > postgresql-server-dev-all - extension build tool for multiple PostgreSQL > versions > Closes: 759725 761020 762632 > Changes: > postgresql-common (162) unstable; urgency=medium > . > [ Christoph Berg ] > * Move PgCommon.pm to /usr/share/perl5. > * t/020_create_sql_remove.t: Test psql -tx alignment to catch a bug in > 9.4beta2. > * supported-versions: Set 9.4 as pgdg default on Ubuntu 14.10. > * Debconf translation updates, thanks! > + nl by Frans Spiesschaert. (Closes: #762632) > . > [ Peter Michael Green ] > * Use ID_LIKE to identify deriviatives of Debian and Ubuntu. > (Closes: #761020) > . > [ Richard Hughes ] > * Use Type=forking in postgresql@.service and start before postgresql. > (Closes: #759725) > Checksums-Sha1: > d16d77dcd70a0e5d3c39795089894f6b336605ee 2240 postgresql-common_162.dsc > 72700311d17f9f5b0ab33dac3c2ee4b1f691a83e 186148 postgresql-common_162.tar.xz > f1d9c51af5a8d18e3ff2f93911a2b66702e3eef8 58446 > postgresql-server-dev-all_162_all.deb > c5aa9355d15073352377250fc34fe71f8676f846 51340 postgresql_9.4+162_all.deb > 890006a4d530ef8b80d061bc4042e6cd798a3f00 51360 > postgresql-client_9.4+162_all.deb > 34b2d39d1f6022cd96876f9b97ad987afee38ff9 51350 postgresql-doc_9.4+162_all.deb > 8ee08a4781b7e6a4b8e02dcd61fccf1313754fdf 51348 > postgresql-contrib_9.4+162_all.deb > 37cdc985a272f14601430965e0a4b0c51077945b 200776 postgresql-common_162_all.deb > 4fd33e920efd5753e91b33bf9c616e3934b56534 72972 > postgresql-client-common_162_all.deb > Checksums-Sha256: > 60ad532f39e13d151ef7f1fcd41c827c881db8a6930bdf293cb6bc7e895aab8b 2240 > postgresql-common_162.dsc > b5346242d7c3704002d6ca9f1339cdd029a7835ef91edadeb33c78efe3b4b96b 186148 > postgresql-common_162.tar.xz > a8ae6a6a98bbf55e6ce7bda3ad9f80d72b74363b0178d450daeaeac1e746c62c 58446 > postgresql-server-dev-all_162_all.deb > e4fa20fae0095a92b18673edc31f26c93b8dddecb4a5f0dfdea3ab661d29173e 51340 > postgresql_9.4+162_all.deb > 58c7da68cf06e36cdd98442a937c671278d5f6f8cdd24a062c2b80a05966f315 51360 > postgresql-client_9.4+162_all.deb > 3f2078e2beeb6d8ad9155415772fd650a85852a502cc3a8d9e360bcbcbc3b089 51350 > postgresql-doc_9.4+162_all.deb > 3234f193a5a1645c56f860776a1ab9420d0e0eff932864ad93776d1bcf60db63 51348 > postgresql-contrib_9.4+162_all.deb > aac562a550a831661d477b20cbe43510fadb699fdcc330a50b6ca71e510ddbb6 200776 > postgresql-common_162_all.deb > fd0ec54bdab7318795134ec98bba6127c656fe38efefe3d8a52f1241e203a79b 72972 > postgresql-client-common_162_all.deb > Files: > abfe957bd81f3d68aec0205233ff609e 58446 database optional > postgresql-server-dev-all_162_all.deb > 9c3efc223baa8258cad326470d523b74 51340 database optional > postgresql_9.4+162_all.deb > d5bc60b95fbee06b77f930ce66425ec9 51360 database optional > postgresql-client_9.4+162_all.deb > 5b651fecba32a54ab5b316a168f72b66 51350 doc optional > postgresql-doc_9.4+162_all.deb > fc840fdb116d0081c38d157e85d8f073 51348 database optional > postgresql-contrib_9.4+162_all.deb > 3d6c3f20e3e1a91bfb2ad1ad3461f617 200776 database optional > postgresql-common_162_all.deb > bdd30524c1444521f3eb9db7823153cd 72972 database optional > postgresql-client-common_162_all.deb > 62e59e4d5652fe62dd9b836d0e78c901 2240 database optional > postgresql-common_162.dsc > 9c6f016609e8a4f74228b3f024d7f5c2 186148 database optional > postgresql-common_162.tar.xz > > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1 > > iQIcBAEBCAAGBQJUND53AAoJEExaa6sS0qeudI4P/26SGgCyyzCpp9td6F2hlLwF > /pNLvS87A0fNlKz6GzRbd9osKxLe9SWteZwKdgjYtUzeCvr8awQLrLNdpLtzg6ru > VJZWyiA/ixX8KEIXL1Ze5EfnDCdxyS5keTmzZG/mmoyzH4qeAeTMcLYQc38r4w2o > VXB6wgizt4Ua4mzKLOU7OlhfwfAY9e5VQjEDoYpsJ24Xdt8kLTD8tXkn1HgeajO7 > GCKrU0YLkd9Q2aj9B1+qtMYwTf7PLqH1Lzu+Fwg+2mee14bo22cMecROmSKV7yFp > BtlNCEtBV1z1AHHrK0kzwzsME4BzKbXNxKoSA35YWzll/DOIeO3uUMDitHVIK/AL > pyK7pMyeUkdyHHKFJ+YriUelX3Q1yMePxZqp6O541LkWje5aA/8ZTZjyLGbHhQ/5 > /w4cJVHAO3rG4hByc6pAXCrtJOjNFYGewHb+oc8mTdbEHRbPK+aRgC2ugDN8+JKA > xNVTfykF/0vdPH6+Qsm0deQDgHuHkx58jawivjaSuoVKsCKCmKjGH46CzuLfqfkK > +Upr8oX0lCJFfgGapVKySjk31BRAFmGicnTzYCSzWwXFOkaoTQydzUJ4gYug5pfK > mQGGLz3J9RiSJaAyF5BOAfespx9PnAz2H24gb7iPazfzxWm31WIz+mzt00F09UCg > ukCm6ezkAR8adS0f/lqH > =vmXR > -----END PGP SIGNATURE----- > Date: Fri, 29 Aug 2014 20:28:50 +0200 > From: beuc <b...@debian.org> > To: Debian Bug Tracking System <sub...@bugs.debian.org> > Subject: postgresql-common: non-synchronous service postgresql > start/stop/reload > X-Mailer: reportbug 6.5.0 > > Package: postgresql-common > Version: 160 > Severity: important > > Hi, > > When working on the FusionForge installation system today I noticed > that in Debian Jessie, running: > > service postgresql start > > (or stop, or reload), is now asynchronous, due to using the new > PostgreSQL systemd init scripts. > > Previously I could rely on the init script to come back when the > database was properly stopped and flushed. Right now the command > returns immediately and starts/stops/reloads in the background. > > My scripts need to modify the PostgreSQL listen address and reload it > before populating the database through the PHP application. They also > need to stop/backup/start the server for quick load/restore during our > testsuite. Due to this change the installation system fails randomly > due to race condition. > > I found it basically impossible to work-around this issue in a > portable manner: > > - 'service postgresql status' is not reliable: it usually says the > service is stopped far before the shutdown is complete. I also got a > few cases where it reported active service with no running daemon. > > - the 'postgresql' process may be stopped already, but pg_ctl still > doing a faststop (especially when there's data to flush to disk) - > there may also be other PostgreSQL processes I don't know about; > so 'ps' is not reliable either. > > - the postgresql control commands vary between Debian and RedHat > (pg_ctl vs. pg_clusterctl), and they need a data directory that can > be in varied, possibly multiple, locations. Using 'pg_*ctl' manually > is error-prone and long. > > - in any case that will require fare more code and testing than > 'service postgresql xxx' > > Please consider maintaining 'service postgresql start/stop/reload' > synchronous even with systemd. > > Cheers! > Sylvain > > > -- System Information: > Debian Release: jessie/sid > APT prefers testing-updates > APT policy: (500, 'testing-updates'), (500, 'testing') > Architecture: amd64 (x86_64) > Foreign Architectures: i386 > > Kernel: Linux 3.14-2-amd64 (SMP w/8 CPU cores) > Locale: LANG=eo.UTF-8, LC_CTYPE=eo.UTF-8 (charmap=UTF-8) > Shell: /bin/sh linked to /bin/dash > > Versions of packages postgresql-common depends on: > ii adduser 3.113+nmu3 > ii debconf [debconf-2.0] 1.5.53 > ii init-system-helpers 1.21 > ii lsb-base 4.1+Debian13 > ii postgresql-client-common 160 > ii procps 1:3.3.9-7 > ii ssl-cert 1.0.34 > ii ucf 3.0030 > > Versions of packages postgresql-common recommends: > ii logrotate 3.8.7-1 > > postgresql-common suggests no packages. > > -- debconf information excluded -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
