diff mbox

[2/4,for,opensm] Make it possible to enable opensm with chkconfig

Message ID 505C7CF7.8010804@acm.org (mailing list archive)
State Accepted
Delegated to: Alex Netes
Headers show

Commit Message

Bart Van Assche Sept. 21, 2012, 2:43 p.m. UTC
"chkconfig $service on" enables a service in the runlevels mentioned
next to "Default-Start" and only in those runlevels.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Cc: Doug Ledford <dledford@redhat.com>
---
 configure.in                  |   10 ++++++++++
 scripts/opensm.init.in        |    4 ++--
 scripts/redhat-opensm.init.in |    3 +++
 3 files changed, 15 insertions(+), 2 deletions(-)

Comments

Alex Netes Oct. 24, 2012, 1:19 p.m. UTC | #1
Hi Bart,

On 16:43 Fri 21 Sep     , Bart Van Assche wrote:
> "chkconfig $service on" enables a service in the runlevels mentioned
> next to "Default-Start" and only in those runlevels.
> 
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> Cc: Doug Ledford <dledford@redhat.com>
> ---
>  configure.in                  |   10 ++++++++++
>  scripts/opensm.init.in        |    4 ++--
>  scripts/redhat-opensm.init.in |    3 +++
>  3 files changed, 15 insertions(+), 2 deletions(-)
> 
> diff --git a/configure.in b/configure.in
> index eb252ff..f660a5b 100644
> --- a/configure.in
> +++ b/configure.in
> @@ -11,6 +11,16 @@ AM_INIT_AUTOMAKE
>  AC_SUBST(RELEASE, ${RELEASE:-unknown})
>  AC_SUBST(TARBALL, ${TARBALL:-${PACKAGE}-${VERSION}.tar.gz})
>  
> +if { rpm -q sles-release || rpm -q openSUSE-release; } >/dev/null 2>&1; then
> +   default_start="2 3 5"
> +   default_stop="0 1 4 6"
> +else
> +   default_start="2 3 4 5"
> +   default_stop="0 1 6"
> +fi
> +AC_SUBST(DEFAULT_START, $default_start)
> +AC_SUBST(DEFAULT_STOP, $default_stop)
> +

Can you please elaborate more why there is a difference with runlevels
selection between SUSE and RH?

-- Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Bart Van Assche Oct. 24, 2012, 1:39 p.m. UTC | #2
On 10/24/12 15:19, Alex Netes wrote:
> Hi Bart,
>
> On 16:43 Fri 21 Sep     , Bart Van Assche wrote:
>> "chkconfig $service on" enables a service in the runlevels mentioned
>> next to "Default-Start" and only in those runlevels.
>>
>> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
>> Cc: Doug Ledford <dledford@redhat.com>
>> ---
>>   configure.in                  |   10 ++++++++++
>>   scripts/opensm.init.in        |    4 ++--
>>   scripts/redhat-opensm.init.in |    3 +++
>>   3 files changed, 15 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure.in b/configure.in
>> index eb252ff..f660a5b 100644
>> --- a/configure.in
>> +++ b/configure.in
>> @@ -11,6 +11,16 @@ AM_INIT_AUTOMAKE
>>   AC_SUBST(RELEASE, ${RELEASE:-unknown})
>>   AC_SUBST(TARBALL, ${TARBALL:-${PACKAGE}-${VERSION}.tar.gz})
>>
>> +if { rpm -q sles-release || rpm -q openSUSE-release; } >/dev/null 2>&1; then
>> +   default_start="2 3 5"
>> +   default_stop="0 1 4 6"
>> +else
>> +   default_start="2 3 4 5"
>> +   default_stop="0 1 6"
>> +fi
>> +AC_SUBST(DEFAULT_START, $default_start)
>> +AC_SUBST(DEFAULT_STOP, $default_stop)
>> +
>
> Can you please elaborate more why there is a difference with runlevels
> selection between SUSE and RH?

Most services in SLES and openSUSE, including syslog, are enabled in 
runlevels 2, 3 and 5 but not in runlevel 4. RHEL systems however follow 
the LFS and have services like syslog enabled in runlevels 2, 3, 4 and 
5. One of the patches I posted adds $syslog to the list of services that 
must be started before opensm is started. If the Default-Start runlevels 
specified in /etc/init.d/opensm do not match the runlevels in which 
syslog is started then chkconfig prints a warning when enabling the 
opensm service. Hence the above code to make the Default-Start runlevels 
of opensm match the Default-Start runlevels of syslog.

Bart.

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/configure.in b/configure.in
index eb252ff..f660a5b 100644
--- a/configure.in
+++ b/configure.in
@@ -11,6 +11,16 @@  AM_INIT_AUTOMAKE
 AC_SUBST(RELEASE, ${RELEASE:-unknown})
 AC_SUBST(TARBALL, ${TARBALL:-${PACKAGE}-${VERSION}.tar.gz})
 
+if { rpm -q sles-release || rpm -q openSUSE-release; } >/dev/null 2>&1; then
+   default_start="2 3 5"
+   default_stop="0 1 4 6"
+else
+   default_start="2 3 4 5"
+   default_stop="0 1 6"
+fi
+AC_SUBST(DEFAULT_START, $default_start)
+AC_SUBST(DEFAULT_STOP, $default_stop)
+
 dnl NOTE: AC_DEFINE's and AC_DEFINE_UNQUOTED's which are used in header files
 dnl       MUST have a corresponding entry in include/opensm/osm_config.h.in to
 dnl       ensure plugin compatibility.
diff --git a/scripts/opensm.init.in b/scripts/opensm.init.in
index 0c84bd3..007dae4 100644
--- a/scripts/opensm.init.in
+++ b/scripts/opensm.init.in
@@ -9,8 +9,8 @@ 
 # Provides: opensm
 # Required-Start: $syslog
 # Required-Stop:
-# Default-Start: none
-# Default-Stop: 0 1 6
+# Default-Start: @DEFAULT_START@
+# Default-Stop: @DEFAULT_STOP@
 # Description:  Manage OpenSM
 ### END INIT INFO
 #
diff --git a/scripts/redhat-opensm.init.in b/scripts/redhat-opensm.init.in
index 9c22275..bea0f0d 100755
--- a/scripts/redhat-opensm.init.in
+++ b/scripts/redhat-opensm.init.in
@@ -7,6 +7,9 @@ 
 #
 ### BEGIN INIT INFO
 # Provides:       opensm
+# Default-Start: @DEFAULT_START@
+# Default-Stop: @DEFAULT_STOP@
+# Description:  Manage OpenSM
 ### END INIT INFO
 #
 # Copyright (c) 2008 Voltaire, Inc. All rights reserved.