Skip to content

fix(openmpi): restore openmpi-devel subpackages#17843

Merged
christopherco merged 4 commits into
3.0-devfrom
mitchzhu/fix-openmpi
Jun 30, 2026
Merged

fix(openmpi): restore openmpi-devel subpackages#17843
christopherco merged 4 commits into
3.0-devfrom
mitchzhu/fix-openmpi

Conversation

@miz060

@miz060 miz060 commented Jun 29, 2026

Copy link
Copy Markdown
Member

Summary

Restore the Azure Linux 3.0 OpenMPI packaging contract after the DOCA 3.3.0 update.

The spec now:

  • keeps runtime files in openmpi
  • builds openmpi-devel and openmpi-docs
  • restores /usr/share/modulefiles/mpi/openmpi-<arch>

This fixes the golden-container flow, which installs openmpi openmpi-devel and then loads mpi/openmpi before running mpiexec --version.

Validation

Default OpenMPI builds back to split RPM mode so openmpi-devel is built
locally instead of falling back to PMC. Add matching runtime
dependencies for openmpi, openmpi-devels, and openmpi-docs.

Signed-off-by: Mitch Zhu <mitchzhu@microsoft.com>
@miz060 miz060 requested a review from a team as a code owner June 29, 2026 17:32
@microsoft-github-policy-service microsoft-github-policy-service Bot added Packaging 3.0-dev PRs Destined for AzureLinux 3.0 labels Jun 29, 2026
Comment thread SPECS/openmpi/openmpi.spec Outdated
Requires: %{mpi_selector_rpm_name}
%endif
Requires: ucx
%if !%{build_all_in_one_rpm}

@ellie-di ellie-di Jun 29, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could we just get rid of the build_all_in_one_rpm (assuming we want several sub-package RPMs)?

@miz060 miz060 Jun 29, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point. removed the build_all_in_one_rpm path

@miz060 miz060 changed the title fix(openmpi): restore split subpackages fix(openmpi): remove all-in-one packaging switch Jun 29, 2026
Always build OpenMPI as split runtime, devel, and docs RPMs. This
restores the local openmpi-devel package needed by the golden container
build.

Signed-off-by: Mitch Zhu <mitchzhu@microsoft.com>
@miz060 miz060 force-pushed the mitchzhu/fix-openmpi branch from 419450f to b60c5e5 Compare June 29, 2026 20:26
@miz060 miz060 changed the title fix(openmpi): remove all-in-one packaging switch fix(openmpi): restore runtime and devel subpackages Jun 29, 2026
@miz060 miz060 changed the title fix(openmpi): restore runtime and devel subpackages fix(openmpi): restore openmpi-devel subpackages Jun 29, 2026
Preserve the openmpi RPM while splitting devel and docs subpackages

Signed-off-by: Mitch Zhu <mitchzhu@microsoft.com>
@miz060 miz060 force-pushed the mitchzhu/fix-openmpi branch from 1abf9cd to 286a0e7 Compare June 30, 2026 05:00
Install the OpenMPI modulefile under /usr/share/modulefiles/mpi so the
golden container can load mpi/openmpi and run its mpiexec smoke test

Signed-off-by: Mitch Zhu <mitchzhu@microsoft.com>
@christopherco christopherco merged commit 70e6236 into 3.0-dev Jun 30, 2026
28 of 31 checks passed
@christopherco christopherco deleted the mitchzhu/fix-openmpi branch June 30, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.0-dev PRs Destined for AzureLinux 3.0 Packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants