Ticket 2246

Summary:	lua support broken
Product:	Slurm	Reporter:	David Matthews <david.matthews>
Component:	slurmctld	Assignee:	Tim Wickberg <tim>
Status:	RESOLVED DUPLICATE	QA Contact:
Severity:	3 - Medium Impact
Priority:	---
Version:	15.08.5
Hardware:	Linux
OS:	Linux
Site:	Met Office	Slinky Site:	---
Alineos Sites:	---	Atos/Eviden Sites:	---
Confidential Site:	---	Coreweave sites:	---
Cray Sites:	---	DS9 clusters:	---
Google sites:	---	HPCnow Sites:	---
HPE Sites:	---	IBM Sites:	---
NOAA SIte:	---	NoveTech Sites:	---
Nvidia HWinf-CS Sites:	---	OCF Sites:	---
Recursion Pharma Sites:	---	SFW Sites:	---
SNIC sites:	---	Tzag Elita Sites:	---
Linux Distro:	---	Machine Name:
CLE Version:		Version Fixed:
Target Release:	---	DevPrio:	---
Emory-Cloud Sites:	---

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 ***