[oe] [PATCH V2 1/4] postgresql: add init script and DESCRIPTION
Martin Jansa
martin.jansa at gmail.com
Wed Apr 23 17:56:35 UTC 2014
On Tue, Apr 22, 2014 at 03:07:25PM +0800, Chong Lu wrote:
> 1. Add DESCRIPTION
> 2. Add init script for starting up the PostgreSQL server.
> 3. Disable krb5 by default
>
> Signed-off-by: Chong Lu <Chong.Lu at windriver.com>
> ---
> .../postgresql/files/postgresql-bashprofile | 4 +
> .../postgresql/files/postgresql.init | 241 ++++++++++++++++++++
> meta-oe/recipes-support/postgresql/postgresql.inc | 63 ++++-
> 3 files changed, 303 insertions(+), 5 deletions(-)
> create mode 100644 meta-oe/recipes-support/postgresql/files/postgresql-bashprofile
> create mode 100644 meta-oe/recipes-support/postgresql/files/postgresql.init
>
> diff --git a/meta-oe/recipes-support/postgresql/files/postgresql-bashprofile b/meta-oe/recipes-support/postgresql/files/postgresql-bashprofile
> new file mode 100644
> index 0000000..1c931f3
> --- /dev/null
> +++ b/meta-oe/recipes-support/postgresql/files/postgresql-bashprofile
> @@ -0,0 +1,4 @@
> +[ -f /etc/profile ] && source /etc/profile
> +
> +PGDATA=/var/lib/postgresql/data
> +export PGDATA
> diff --git a/meta-oe/recipes-support/postgresql/files/postgresql.init b/meta-oe/recipes-support/postgresql/files/postgresql.init
> new file mode 100644
> index 0000000..ab46477
> --- /dev/null
> +++ b/meta-oe/recipes-support/postgresql/files/postgresql.init
> @@ -0,0 +1,241 @@
> +#!/bin/sh
> +#
> +# postgresql This is the init script for starting up the PostgreSQL
> +# server.
> +#
> +# chkconfig: - 64 36
> +# description: PostgreSQL database server.
> +# processname: postmaster
> +# pidfile: /var/run/postmaster.PORT.pid
> +
> +# This script is slightly unusual in that the name of the daemon (postmaster)
> +# is not the same as the name of the subsystem (postgresql)
> +
> +# PGVERSION is the full package version, e.g., 8.4.0
> +# Note: the specfile inserts the correct value during package build
> +PGVERSION=9.2.4
> +# PGMAJORVERSION is major version, e.g., 8.4 (this should match PG_VERSION)
> +PGMAJORVERSION=`echo "$PGVERSION" | sed 's/^\([0-9]*\.[0-9]*\).*$/\1/'`
> +
> +# Source function library.
> +. /etc/init.d/functions
> +
> +# Find the name of the script
> +NAME=`basename $0`
> +if [ ${NAME:0:1} = "S" -o ${NAME:0:1} = "K" ]
> +then
> + NAME=${NAME:3}
> +fi
> +
> +# For SELinux we need to use 'runuser' not 'su'
> +if [ -x /sbin/runuser ]
> +then
> + SU=runuser
> +else
> + SU=su
> +fi
> +
> +
> +# Set defaults for configuration variables
> +PGENGINE=/usr/bin
> +PGPORT=5432
> +PGDATA=/var/lib/postgresql/data
> +PGLOG=/var/lib/postgresql/pgstartup.log
> +# Value to set as postmaster process's oom_adj
> +PG_OOM_ADJ=-17
> +
> +# Override defaults from /etc/sysconfig/postgresql if file is present
> +[ -f /etc/default/postgresql/${NAME} ] && . /etc/default/postgresql/${NAME}
> +
> +export PGDATA
> +export PGPORT
> +
> +lockfile="/var/lock/subsys/${NAME}"
> +pidfile="/var/run/postmaster.${PGPORT}.pid"
> +
> +script_result=0
> +
> +start(){
> + [ -x "$PGENGINE/postmaster" ] || exit 5
> +
> + PSQL_START=$"Starting ${NAME} service: "
> +
> + # Make sure startup-time log file is valid
> + if [ ! -e "$PGLOG" -a ! -h "$PGLOG" ]
> + then
> + touch "$PGLOG" || exit 4
> + chown postgres:postgres "$PGLOG"
> + chmod go-rwx "$PGLOG"
> + [ -x /sbin/restorecon ] && /sbin/restorecon "$PGLOG"
> + fi
> +
> + # Check for the PGDATA structure
> + if [ -f "$PGDATA/PG_VERSION" ] && [ -d "$PGDATA/base" ]
> + then
> + # Check version of existing PGDATA
> + if [ x`cat "$PGDATA/PG_VERSION"` != x"$PGMAJORVERSION" ]
> + then
> + SYSDOCDIR="(Your System's documentation directory)"
> + if [ -d "/usr/doc/postgresql-$PGVERSION" ]
> + then
> + SYSDOCDIR=/usr/doc
> + fi
> + if [ -d "/usr/share/doc/postgresql-$PGVERSION" ]
> + then
> + SYSDOCDIR=/usr/share/doc
> + fi
> + if [ -d "/usr/doc/packages/postgresql-$PGVERSION" ]
> + then
> + SYSDOCDIR=/usr/doc/packages
> + fi
> + if [ -d "/usr/share/doc/packages/postgresql-$PGVERSION" ]
> + then
> + SYSDOCDIR=/usr/share/doc/packages
> + fi
> + echo
> + echo $"An old version of the database format was found."
> + echo $"You need to upgrade the data format before using PostgreSQL."
> + echo $"See $SYSDOCDIR/postgresql-$PGVERSION/README.rpm-dist for more information."
> + exit 1
> + fi
> + else
> + # No existing PGDATA! Warn the user to initdb it.
> + echo
> + echo "$PGDATA is missing. Use \"service postgresql initdb\" to initialize the cluster first."
> + echo -n " [FAILED] "
> + echo
> + exit 1
> + fi
> +
> + echo -n "$PSQL_START"
> + test x"$PG_OOM_ADJ" != x && echo "$PG_OOM_ADJ" > /proc/self/oom_score_adj
> + $SU -l postgres -c "$PGENGINE/postmaster -p '$PGPORT' -D '$PGDATA' ${PGOPTS} &" >> "$PGLOG" 2>&1 < /dev/null
> + sleep 2
> + pid=`head -n 1 "$PGDATA/postmaster.pid" 2>/dev/null`
> + if [ "x$pid" != x ]
> + then
> + echo -n " [ OK ]"
> + touch "$lockfile"
> + echo $pid > "$pidfile"
> + echo
> + else
> + echo -n " [FAILED]"
> + echo
> + script_result=1
> + fi
> +}
> +
> +stop(){
> + echo -n $"Stopping ${NAME} service: "
> + if [ -e "$lockfile" ]
> + then
> + $SU -l postgres -c "$PGENGINE/pg_ctl stop -D '$PGDATA' -s -m fast" > /dev/null 2>&1 < /dev/null
> + ret=$?
> + if [ $ret -eq 0 ]
> + then
> + echo -n " [ OK ] "
> + rm -f "$pidfile"
> + rm -f "$lockfile"
> + else
> + echo -n " [FAILED] "
> + script_result=1
> + fi
> + else
> + # not running; per LSB standards this is "ok"
> + echo -n " [ OK ] "
> + fi
> + echo
> +}
> +
> +restart(){
> + stop
> + start
> +}
> +
> +condrestart(){
> + [ -e "$lockfile" ] && restart || :
> +}
> +
> +reload(){
> + $SU -l postgres -c "$PGENGINE/pg_ctl reload -D '$PGDATA' -s" > /dev/null 2>&1 < /dev/null
> +}
> +
> +initdb(){
> + if [ -f "$PGDATA/PG_VERSION" ]
> + then
> + echo -n "Data directory is not empty!"
> + echo -n " [FAILED] "
> + echo
> + script_result=1
> + else
> + echo -n $"Initializing database: "
> + if [ ! -e "$PGDATA" -a ! -h "$PGDATA" ]
> + then
> + mkdir -p "$PGDATA" || exit 1
> + chown postgres:postgres "$PGDATA"
> + chmod go-rwx "$PGDATA"
> + fi
> + # Clean up SELinux tagging for PGDATA
> + [ -x /sbin/restorecon ] && /sbin/restorecon "$PGDATA"
> +
> + # Make sure the startup-time log file is OK, too
> + if [ ! -e "$PGLOG" -a ! -h "$PGLOG" ]
> + then
> + touch "$PGLOG" || exit 1
> + chown postgres:postgres "$PGLOG"
> + chmod go-rwx "$PGLOG"
> + [ -x /sbin/restorecon ] && /sbin/restorecon "$PGLOG"
> + fi
> +
> + # Initialize the database
> + $SU -l postgres -c "$PGENGINE/initdb --pgdata='$PGDATA' --auth='ident'" >> "$PGLOG" 2>&1 < /dev/null
> +
> + # Create directory for postmaster log
> + mkdir "$PGDATA/pg_log"
> + chown postgres:postgres "$PGDATA/pg_log"
> + chmod go-rwx "$PGDATA/pg_log"
> +
> + if [ -f "$PGDATA/PG_VERSION" ]
> + then
> + echo -n " [ OK ] "
> + else
> + echo -n " [FAILED] "
> + script_result=1
> + fi
> + echo
> + fi
> +}
> +
> +# See how we were called.
> +case "$1" in
> + start)
> + start
> + ;;
> + stop)
> + stop
> + ;;
> + status)
> + status postmaster
> + script_result=$?
> + ;;
> + restart)
> + restart
> + ;;
> + condrestart|try-restart)
> + condrestart
> + ;;
> + reload)
> + reload
> + ;;
> + force-reload)
> + restart
> + ;;
> + initdb)
> + initdb
> + ;;
> + *)
> + echo $"Usage: $0 {start|stop|status|restart|condrestart|try-restart|reload|force-reload|initdb}"
> + exit 2
> +esac
> +
> +exit $script_result
> diff --git a/meta-oe/recipes-support/postgresql/postgresql.inc b/meta-oe/recipes-support/postgresql/postgresql.inc
> index 5f2088e..db425e7 100644
> --- a/meta-oe/recipes-support/postgresql/postgresql.inc
> +++ b/meta-oe/recipes-support/postgresql/postgresql.inc
> @@ -1,4 +1,22 @@
> -DESCRIPTION = "PostgreSQL is a powerful, open source relational database system."
> +SUMMARY = "PostgreSQL is a powerful, open source relational database system."
> +DESCRIPTION = "\
> + PostgreSQL is an advanced Object-Relational database management system \
> + (DBMS) that supports almost all SQL constructs (including \
> + transactions, subselects and user-defined types and functions). The \
> + postgresql package includes the client programs and libraries that \
> + you'll need to access a PostgreSQL DBMS server. These PostgreSQL \
> + client programs are programs that directly manipulate the internal \
> + structure of PostgreSQL databases on a PostgreSQL server. These client \
> + programs can be located on the same machine with the PostgreSQL \
> + server, or may be on a remote machine which accesses a PostgreSQL \
> + server over a network connection. This package contains the docs \
> + in HTML for the whole package, as well as command-line utilities for \
> + managing PostgreSQL databases on a PostgreSQL server. \
> + \
> + If you want to manipulate a PostgreSQL database on a local or remote \
> + PostgreSQL server, you need this package. You also need to install \
> + this package if you're installing the postgresql-server package. \
> + "
> HOMEPAGE = "http://www.postgresql.com"
> LICENSE = "BSD"
> DEPENDS = "zlib readline tzcode-native"
> @@ -8,23 +26,58 @@ ARM_INSTRUCTION_SET = "arm"
>
> #WARNING: this recipe assumes you have the timezone compiler present in /usr/sbin/zic
>
> -SRC_URI = "http://ftp.postgresql.org/pub/source/v${PV}/${P}.tar.bz2"
> +SRC_URI = "http://ftp.postgresql.org/pub/source/v${PV}/${P}.tar.bz2 \
> + file://postgresql.init \
> + file://postgresql-bashprofile \
> +"
>
> LEAD_SONAME = "libpq.so"
>
> # LDFLAGS for shared libraries
> export LDFLAGS_SL = "${LDFLAGS}"
>
> -inherit autotools pkgconfig
> +inherit autotools pkgconfig useradd
>
> -EXTRA_OECONF = "--disable-rpath"
> -EXTRA_OECONF_sh4 = "--disable-spinlocks --disable-rpath"
> +EXTRA_OECONF += "--enable-thread-safety --disable-rpath \
> + --datadir=${datadir}/${BPN} \
> + --sysconfdir=${sysconfdir}/${BPN} \
> + --without-krb5 \
> +"
> +EXTRA_OECONF_sh4 += "--disable-spinlocks"
> EXTRA_OECONF_aarch64 += "--disable-spinlocks"
>
> do_compile_append() {
> cp /usr/sbin/zic ${S}/src/timezone/
> }
>
> +# server needs to configure user and group
> +usernum = "28"
> +groupnum = "28"
> +USERADD_PACKAGES = "${PN}"
> +USERADD_PARAM_${PN} = "-M -g postgres -o -r -d ${localstatedir}/lib/${BPN} \
> + -s /bin/bash -c 'PostgreSQL Server' -u ${usernum} postgres"
> +GROUPADD_PARAM_${PN} = "-g ${groupnum} -o -r postgres"
> +
> +INITSCRIPT_PACKAGES = "${PN}"
> +INITSCRIPT_NAME = "${BPN}-server"
> +INITSCRIPT_PARAMS = "start 64 . stop 36 0 1 2 3 4 5 6 ."
> +
> +do_install_append() {
> + # install dirs and server init
> + install -d ${D}${sysconfdir}/init.d
> + install -m 0755 ${WORKDIR}/${BPN}.init \
> + ${D}${sysconfdir}/init.d/${BPN}-server
> + sed -i -e "s/^PGVERSION=.*$/PGVERSION=${PV}/g" \
> + ${D}${sysconfdir}/init.d/${BPN}-server
> + install -d -m 700 ${D}${localstatedir}/lib/${BPN}/data
> + install -d -m 700 ${D}${localstatedir}/lib/${BPN}/backups
> + install -m 644 ${WORKDIR}/${BPN}-bashprofile \
> + ${D}${localstatedir}/lib/${BPN}/.bash_profile
> + chown -R postgres:postgres ${D}${localstatedir}/lib/${BPN}
> + # multiple server config directory
> + install -d -m 700 ${D}${sysconfdir}/default/${BPN}
> +}
We're using 4 spaces for indentation, I've changed all 4 patches to use
that when cherry-picking them to master-next, but next time please use
correct indentation.
> +
> SSTATE_SCAN_FILES += "Makefile.global"
>
> PACKAGES =+ "${PN}-client ${PN}-server-dev ${PN}-timezone \
> --
> 1.7.9.5
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Martin 'JaMa' Jansa jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20140423/430ff64f/attachment-0002.sig>
More information about the Openembedded-devel
mailing list