Greetings,

In the process of trying to install Robinhood 3.1.7 with Lustre 2.15.2 on
Rocky 8.8 I ran into a few issues with the source. It would not compile or
install until I made these changes. I would appreciate a developer taking a
quick look at my changes to make sure they won't break anything I am not
aware of.

Many thanks,

--Jeff

Change #1: Configure fails if there is more than one lustre-client package
installed. Example being lustre-client *and* lustre-client-dkms

diff -Naur /tmp/robinhood/autotools/m4/lustre.m4 autotools/m4/lustre.m4
--- /tmp/robinhood/autotools/m4/lustre.m4 2023-11-08 19:07:36.195248254
-0500
+++ autotools/m4/lustre.m4 2023-11-07 13:45:16.048977836 -0500
@@ -13,8 +13,11 @@
             LPACKAGE=lustre-client
             # Assume we want the same version as this package,
             # whatever 'lustre' or 'lustre-client'
+            #
+            # Added pipe to `head -1` to properly handle cases of multiple
packages; lustre-client and lustre-client-dkms
+            #
             AC_MSG_CHECKING(Lustre version)
-            LVERSION=`rpm -q --whatprovides lustre-client --qf
"%{Version}\n" 2>/dev/null | grep -v "no package" | cut -d "." -f 1-2`
+            LVERSION=`rpm -q --whatprovides lustre-client --qf
"%{Version}\n" 2>/dev/null | grep -v "no package" | cut -d "." -f 1-2 |
head -1`
             AC_MSG_RESULT($LVERSION)
         else
             AC_MSG_RESULT(no)

Change #2: Compilation fails with Lustre 2.15.2 on Lustre getfileinfo old
vs new method.

diff -Naur /tmp/robinhood/src/common/lustre_tools.c
src/common/lustre_tools.c
--- /tmp/robinhood/src/common/lustre_tools.c 2023-11-08 19:07:36.201247852
-0500
+++ src/common/lustre_tools.c 2023-11-08 19:12:11.205808334 -0500
@@ -851,11 +851,13 @@
  * but it doesn't return a struct stat.
  * Use the old (compatible) ioctl() instead.
  */
+/*
 #ifdef IOC_MDC_GETFILEINFO_OLD
 #   define IOC_MDC_GETFILEINFO_V1   IOC_MDC_GETFILEINFO_OLD
 #else
 #   define IOC_MDC_GETFILEINFO_V1   IOC_MDC_GETFILEINFO
 #endif
+*/


 /* This code is an adaptation of llapi_mds_getfileinfo() in liblustreapi.

Change #3: robinhood-webgui will not install due to rpm requires php-mysql
which was replaced in RHEL8 variant distros with php-mysqlnd

diff -Naur /tmp/robinhood/robinhood.spec.in robinhood.spec.in
--- /tmp/robinhood/robinhood.spec.in 2023-11-08 19:07:36.197248120 -0500
+++ robinhood.spec.in 2023-11-07 14:45:37.846415320 -0500
@@ -167,7 +167,7 @@
 Summary: Web interface to vizualize filesystems stats
 Group: Applications/System
 Release: @RELEASE@
-Requires: php, php-pdo, php-mysql
+Requires: php, php-pdo, php-mysqlnd

 %description webgui
 Web interface to vizualize filesystems stats.



-- 
------------------------------
Jeff Johnson
Co-Founder
Aeon Computing

jeff.john...@aeoncomputing.com
www.aeoncomputing.com
t: 858-412-3810 x1001   f: 858-412-3845
m: 619-204-9061

4170 Morena Boulevard, Suite C - San Diego, CA 92117

High-Performance Computing / Lustre Filesystems / Scale-out Storage
_______________________________________________
robinhood-support mailing list
robinhood-support@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/robinhood-support

Reply via email to