Ticket 2246 - lua support broken
Summary: lua support broken
Status: RESOLVED DUPLICATE of ticket 2243
Alias: None
Product: Slurm
Classification: Unclassified
Component: slurmctld (show other tickets)
Version: 15.08.5
Hardware: Linux Linux
: --- 3 - Medium Impact
Assignee: Tim Wickberg
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2015-12-11 03:36 MST by David Matthews
Modified: 2015-12-11 05:25 MST (History)
0 users

See Also:
Site: Met Office
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: ---
DS9 clusters: ---
HPCnow Sites: ---
HPE Sites: ---
IBM Sites: ---
NOAA SIte: ---
OCF Sites: ---
Recursion Pharma Sites: ---
SFW Sites: ---
SNIC sites: ---
Linux Distro: ---
Machine Name:
CLE Version:
Version Fixed:
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---


Attachments

Note You need to log in before you can comment on or make changes to this ticket.
Description David Matthews 2015-12-11 03:36:09 MST
We're trying to upgrade to 15.08.5 on RHEL6 (our previous attempt to upgrade to 15.08.4 failed, see http://bugs.schedmd.com/show_bug.cgi?id=2237).

It appears the changes introduced in 15.08.5 have broken support for lua. I've tracked this down to errors in "configure". The test for x_ac_lua_pkg_name looks flawed, see:
https://github.com/SchedMD/slurm/blob/slurm-15.08/configure#L23385-L23403
I've no prior experience with pkg_config but I don't think it's valid to check for "lua5.2" (should be something like "lua >= 5.2"?). So, x_ac_lua_pkg_name is left set as "lua". At 15.08.04 this was fine but the changes introduced at 15.08.5 mean we get the following error when we try to start the controller:
error: xlua_dlopen: Not compiled with lua support!  Please install a lua dev package.
which comes from here:
https://github.com/SchedMD/slurm/blob/slurm-15.08/src/common/xlua.c#L58

Note that modifying configure so that x_ac_lua_pkg_name gets set to "lua5.1" causes my build to fail (I haven't had time to investigate this).

So, my temporary workaround is as follows:

--- slurm-15.08.5.orig/configure	2015-12-10 23:55:01.000000000 +0000
+++ slurm-15.08.5/configure	2015-12-11 16:01:14.555747000 +0000
@@ -23531,7 +23531,7 @@ fi
 
 $as_echo "#define HAVE_LUA_5_2 1" >>confdefs.h
 
-		elif test "x$x_ac_lua_pkg_name" = "xlua5.1"; then
+		elif test "x$x_ac_lua_pkg_name" = "xlua"; then
 
 $as_echo "#define HAVE_LUA_5_1 1" >>confdefs.h

Everything seems to be OK with this patched version.

As an aside, note that the 15.08.5 tarball appears to contain various temporary files, e.g.
META~
NEWS~
x
x~
Comment 1 Tim Wickberg 2015-12-11 05:25:29 MST
We're looking into a better solution for this. I'm marking this a duplicate as we're already tracking this exact issue on 2243.

*** This ticket has been marked as a duplicate of ticket 2243 ***