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.

Sign up for GitHub

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

Jump to bottom

docs: add some info about HPE CXI ofi provider #12162

Merged
merged 1 commit into from
Dec 14, 2023
Merged

docs: add some info about HPE CXI ofi provider #12162

merged 1 commit into from
Dec 14, 2023

Conversation

hppritcha
Copy link
Member

and also make a statement about the OFI BTL more accurate.

Related to #12038

jsquyres
jsquyres reviewed Dec 13, 2023
View reviewed changes
docs/tuning-apps/networking/ofi.rst Outdated
@@ -76,6 +78,11 @@ list. For example:
shell$ ompi_info --param mtl ofi --level 9
.. note:: When using the HPE CXI provider and ``mpirun`` as the job launcher,
Copy link
Member

Choose a reason for hiding this comment

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

Do you want to make this .. important::?

FWIW: here's all the possibilities of colored boxes: https://docs.open-mpi.org/en/v5.0.x/developers/rst-for-markdown-expats.html#brightly-colored-boxes

jsquyres
jsquyres approved these changes Dec 13, 2023
View reviewed changes
Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

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

Regardless of whether you change .. note:: to something else or not, this looks good to me.

@hppritcha
Copy link
Member Author

NVIDIA CI seems busted

Run /start build
~/CI /github/workspace/ompi/ompi
~/CI ~/CI
pod/ompi-ci-1 condition met
NAME          READY   STATUS    RESTARTS   AGE
gh-runner-0   1/1     Running   0          85d
ompi-ci-1     1/1     Running   0          63s
~/CI
Building OMPI, UCX and tests
env: '/GIT/ompi-63[4](https://github.com/open-mpi/ompi/actions/runs/7201965226/job/19619327437?pr=12162#step:2:5)-2/jenkins_scripts/jenkins/ompi/run.sh': No such file or directory
command terminated with exit code 12[7](https://github.com/open-mpi/ompi/actions/runs/7201965226/job/19619327437?pr=12162#step:2:8)
Error: Process completed with exit code [12](https://github.com/open-mpi/ompi/actions/runs/7201965226/job/19619327437?pr=12162#step:2:13)7.

ignoring and merging.

@janjust

@hppritcha
docs: add some info about HPE CXI ofi provider
2718732 
and also make a statement about the OFI BTL more accurate.

Related to open-mpi#12038

Signed-off-by: Howard Pritchard <[email protected]>
@hppritcha hppritcha merged commit 6911950 into open-mpi:main Dec 14, 2023
10 checks passed
@hppritcha hppritcha mentioned this pull request Dec 21, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsquyres jsquyres jsquyres approved these changes

Assignees
No one assigned
Labels
Target: main
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hppritcha @jsquyres