Bug 6598 - Allow '--with ucx' in rpmbuild
Summary: Allow '--with ucx' in rpmbuild
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Build System and Packaging (show other bugs)
Version: 19.05.x
Hardware: Linux Linux
: --- C - Contributions
Assignee: Tim Wickberg
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2019-02-27 05:21 MST by dani
Modified: 2020-03-10 04:15 MDT (History)
1 user (show)

See Also:
Site: -Other-
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: 19.05.0rc1
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---


Attachments
Add support for --with ucx to rpmbuild (2.27 KB, patch)
2019-02-27 05:21 MST, dani
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description dani 2019-02-27 05:21:35 MST
Created attachment 9331 [details]
Add support for --with ucx to rpmbuild

Currently even if ucx and ucx-devel packages are installed, configure by default ignores them unless specified via ./configure --with-ucx.
There is no very easy way to provide this configuration option to rpmbuild using current defines, hence added 'with_ucx" option.

Other part of this patch adds requires to slurmd package on pmix and ucx.
This was done to avoid confusion by users expecting srun --mpi=list to correctly transmit system capabilities.
consider slurm build with pmix, but pmix is not installed on compute nodes. In that case a user running srun --mpi=list will get pmix as an option, but it will fail if pmix was not present to wireup on the nodes. The error message is quite confusing in this regard as it tells the user to check --mpi=list for acceptable types.

Last issue is version coherence - without further knowledge of true dependencies, I have opted to require same pmix, ucx versions as were used during build.
Comment 1 Tim Wickberg 2019-04-29 14:26:07 MDT
Thank you Daniel. These are both included in 19.05rc1 which will be released on Tuesday:

commit fa2e22c8205cd7b689a2edf926b86f301b037d29
Author:     Daniel Letai <dani@letai.org.il>
AuthorDate: Mon Apr 29 14:24:18 2019 -0600

    slurm.spec - allow --with-ucx in rpmbuild.
    
    And require the compute nodes to have an identical installation version
    to avoid issues with mismatched libraries.
    
    Bug 6598.

commit 35bb9afb94619382e8e4059a08b205bdbbda6e99
Author:     Daniel Letai <dani@letai.org.il>
AuthorDate: Mon Apr 29 14:22:13 2019 -0600

    slurm.spec - allow --with-pmix in rpmbuild.
    
    And require the compute nodes to have an identical installation version
    to avoid issues with mismatched libraries.
    
    Bug 6598.