Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OpenMPI 4.1.5 #588

Open
bdevcich opened this issue Aug 18, 2023 · 11 comments
Open

OpenMPI 4.1.5 #588

bdevcich opened this issue Aug 18, 2023 · 11 comments

Comments

@bdevcich
Copy link

Any plans to update to the latest stable version of OpenMPI?

@alculquicondor
Copy link
Collaborator

It should just work. Feel free to send a PR.

@tenzen-y
Copy link
Member

IIRC, we don't specify the OpenMPI version. So just rebuilding the image might be enough.

&& apt install -y --no-install-recommends openmpi-bin \

@bdevcich
Copy link
Author

It appears that 4.1.0 is the latest that is going to be provided with bullseye: https://packages.debian.org/bullseye/openmpi-bin. I didn't see any updated packages in the updates.

It looks like bookworm has 4.1.4: https://packages.debian.org/bookworm/openmpi-bin

Any concerns with building openmpi from source?

@tenzen-y
Copy link
Member

I see.

Any concerns with building openmpi from source?

I want to avoid building the OpenMPI to avoid increasing maintenance costs.

It looks like bookworm has 4.1.4: https://packages.debian.org/bookworm/openmpi-bin

Actually, we already have a PR to update the Debian version, although there are unresolved issues: #573

Can you try to update the Debian version instead of building OpenMPI?

WDYT? @alculquicondor @terrytangyuan

@alculquicondor
Copy link
Collaborator

Yes, prefer to update the debian version.

@terrytangyuan
Copy link
Member

Agreed. We should not build it from source

@bdevcich
Copy link
Author

bdevcich commented Aug 18, 2023

Thanks.

Makes sense. It adds some complexity. I think 4.1.4 (that comes with bookworm) will be fine as it contains the fix that we're interested in.

So the path forward is to get traction on #573 ?

@tenzen-y
Copy link
Member

Thanks.

Makes sense. It adds some complexity. I think 4.1.4 (that comes with bookworm) will be fine as it contains the fix that we're interested in.

So the path forward is to get traction on #573 ?

I think you can open the new PR :)

@alculquicondor
Copy link
Collaborator

In any case, you should be able to run any version of OpenMPI with the operator. You can build your containers against bookworm (or any other distro).

@abeltre1
Copy link

abeltre1 commented Nov 6, 2023

@bdevcich @tenzen-y is there a reason why we do not make the install from source?

@tenzen-y
Copy link
Member

@bdevcich @tenzen-y is there a reason why we do not make the install from source?

As I mentioned the above (#588 (comment)), we should avoid increasing maintenance costs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants