Ticket 17706

Actually, the precondition RPM package including libraries is installed by:

$ yum install freeipmi-devel

FYI: The freeipmi-devel RPM contains the header file required by the Slurm configure script:

$ rpm -qf /usr/include/freeipmi/freeipmi.h
freeipmi-devel-1.5.7-3.el7.x86_64

Hi Ole,

Just updating here to let you know that I will work on that. Since these prereqs are not only for IPMI, and all of them are already specified in[1], I may add some generic note encouraging to check if the libraries were installed, will discuss with my colleagues.

I did also find in this page:

"If the respective plugin for an option is not loaded then that option will appear to be unknown by Slurm and silently ignored. If you decide to change plugin types you may also have to change the related options."

As we have already seen, this is not exactly true. It will fatal, rather than ignoring silently. Will also check this and update documentation if needed.

Cheers,
Oscar

[1]https://slurm.schedmd.com/quickstart_admin.html

(In reply to Oscar Hernández from comment #3)
> Just updating here to let you know that I will work on that. Since these
> prereqs are not only for IPMI, and all of them are already specified in[1],
> I may add some generic note encouraging to check if the libraries were
> installed, will discuss with my colleagues.

The quickstart is *very* high-level and general.  It would be an advantage if the quickstart guide specifically could state how to install prerequite packages on RPM-based Linux systems.  

If SchedMD will not offer instructions for prerequisite RPMs, people can always find detailed advice in my Wiki page :-)
https://wiki.fysik.dtu.dk/Niflheim_system/Slurm_installation/#install-prerequisites

IMHO, it would still be good to copy the prerequisites information from quickstart to the acct_gather.conf manual page so that it will become more self-contained and complete.  Or maybe just refer to the quickstart guide for prerequistes!

> "If the respective plugin for an option is not loaded then that option will
> appear to be unknown by Slurm and silently ignored. If you decide to change
> plugin types you may also have to change the related options."
> 
> As we have already seen, this is not exactly true. It will fatal, rather
> than ignoring silently. Will also check this and update documentation if
> needed.

Yes, it would be wonderful if you could update that documentation :-)

Thanks a lot,
Ole

There is a new item which would be great if it could be added to the documentation from 23.11:

In bug 17900 a new option will be added to slurm.spec from 23.11 so that users can now specify building RPMs including the freeipmi prerequisite:

$ rpmbuild <...> --with freeipmi

Thanks,
Ole

Hi Ole,

Changes in the documentation suggested have been already committed. You will find them in:

commit db69661333faa92921da55a4e9bbd20ae1dee08c
Author: Oscar Hernández <oscar.hernandez@schedmd.com>
Date:   Thu Oct 12 15:19:41 2023 +0200

    Docs - Improve acct_gather.conf documentation
    
    Clarify that Slurm will fatal when settings in acct_gather.conf do not
    match the plugins loaded. Also mention that freeipmi devel package is
    needed in order for the IPMI plugin to be built.
    
    Bug 17706

and:

commit 3f61e578242598f5686526af9d875a00a3270e40
Author: Oscar Hernández <oscar.hernandez@schedmd.com>
Date:   Fri Oct 20 16:55:44 2023 +0200

    Docs - Document new "--with freeipmi" slurm.spec option
    
    Bug 17706

I am closing as fixed. But let me know if you have any comments on them.

Thanks,
Oscar

Hi Oscar,

(In reply to Oscar Hernández from comment #11)
> Changes in the documentation suggested have been already committed. You will
> find them in:
> 
> commit db69661333faa92921da55a4e9bbd20ae1dee08c
> Author: Oscar Hernández <oscar.hernandez@schedmd.com>
> Date:   Thu Oct 12 15:19:41 2023 +0200
> 
>     Docs - Improve acct_gather.conf documentation
>     
>     Clarify that Slurm will fatal when settings in acct_gather.conf do not
>     match the plugins loaded. Also mention that freeipmi devel package is
>     needed in order for the IPMI plugin to be built.

The updated docs look fine!

> commit 3f61e578242598f5686526af9d875a00a3270e40
> Author: Oscar Hernández <oscar.hernandez@schedmd.com>
> Date:   Fri Oct 20 16:55:44 2023 +0200
> 
>     Docs - Document new "--with freeipmi" slurm.spec option

Thanks a lot for documenting the "--with freeipmi"!

Best regards,
Ole