Merge lp:~jontai/openvista-gtm-integration/bug400957 into lp:openvista-gtm-integration

[Jon Tai]

[jeff.apple (jeff-apple)]

We check for a return status of 162 when updating the page lock size. Are there any other error codes that might be useful to catch? Maybe have a default there with a warning when other values are returned.

review: Approve

[lp:~jontai/openvista-gtm-integration/bug400957]

82. By Jon Tai on 2009-07-20

handle unknown errors

[Jon Tai (jontai)]

Added a default case to catch other status codes (not 0 and not 162). Merging...

=== modified file 'packages/rpm/openvista-utils.spec'
--- packages/rpm/openvista-utils.spec	2009-07-17 08:17:11 +0000
+++ packages/rpm/openvista-utils.spec	2009-07-18 09:28:22 +0000
@@ -1,6 +1,6 @@
 Name:		openvista-utils
 Version:	0.8.4
-Release:	1
+Release:	2
 Summary:	Utilities for configuring and managing OpenVista
 License:	AGPL
 Vendor:		Medsphere Systems Corporation
@@ -157,11 +157,14 @@
 %attr(4550,root,openvista) /sbin/ovauth
 
 %pre
-/usr/sbin/useradd -G gtm openvista 2> /dev/null || :
+/usr/sbin/groupadd openvista 2> /dev/null || :
+/usr/sbin/useradd -g openvista -G gtm openvista 2> /dev/null || :
 
 # update lock space in existing databases if upgrading
 %triggerun -- openvista-utils <= 0.8.3-2
+echo "Increasing lock space of existing databases to 400 pages..."
 for instance in /opt/openvista/*; do
+    # update global directory
     if [ -L ${instance}/gtm ]; then
         gtm_dist=${instance}/gtm gtmgbldir=${instance}/globals/mumps.gld gtmroutines=${instance}/gtm ${instance}/gtm/mumps -run ^GDE > /dev/null 2>&1 <<EOF
 CHANGE -SEGMENT DEFAULT -LOCK_SPACE=400
@@ -169,8 +172,19 @@
 TEMPLATE -SEGMENT -LOCK_SPACE=400
 EXIT
 EOF
-        gtm_dist=${instance}/gtm gtmgbldir=${instance}/globals/mumps.gld ${instance}/gtm/mupip set -region -lock_space=400 DEFAULT
-        gtm_dist=${instance}/gtm gtmgbldir=${instance}/globals/mumps.gld ${instance}/gtm/mupip set -region -lock_space=400 SCRATCH
+
+        # update actual database files
+        gtm_dist=${instance}/gtm gtmgbldir=${instance}/globals/mumps.gld ${instance}/gtm/mupip set -region -lock_space=400 '*' 2>&1 \
+          | logger -p user.info
+        retval=${PIPESTATUS[0]}
+        if [ ${retval} -eq 0 ]; then
+            echo "`basename ${instance}` database's lock space increased to 400 pages"
+        elif [ ${retval} -eq 162 ]; then
+            echo ""
+            echo "Unable to increase `basename ${instance}` database's lock space to 400 pages; the database is currently in use.  Stop all processes accessing the `basename ${instance}` database, then run:"
+            echo "gtm_dist=${instance}/gtm gtmgbldir=${instance}/globals/mumps.gld ${instance}/gtm/mupip set -region -lock_space=400 '*'"
+            echo ""
+        fi
     fi
 done