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

OpenPMIx Update #1773

Open
wants to merge 3 commits into
base: 3.x
Choose a base branch
from
Open

OpenPMIx Update #1773

wants to merge 3 commits into from

Conversation

jcsiadal
Copy link
Contributor

@jcsiadal jcsiadal commented Apr 2, 2023

This is a prototype package I'm placing here for review, or to move into the development tree.

One major change is moving it under its proper name.

This update also adds PRRTE to the build, but I needed to add as a second spec. The full OpenPMIx package is required to build PRRTE, but I was not able to merge the two into a single build. There are a lot of hardcoded paths added during build and I was only able to get a successful build with PMIx in its final location. Even doing a temp build and install of PMIX introduced different errors in PRRTE.

If our build system can handle both specs in one directory, great; otherwise, we'll need to have a new package under RMS for PRRTE.

These both are using RCs, as they will likely be full releases before 3.0 and these specs also work with the current production releases.

@jcsiadal
Copy link
Contributor Author

jcsiadal commented Apr 2, 2023

@mghpcsim Prototyping the PMIx/PRRTE update. It seems to work, but see my notes in the summary.

@github-actions
Copy link

github-actions bot commented Apr 2, 2023

Test Results

0 files   -   4  0 suites   - 4   0s ⏱️ ±0s
0 tests  - 17  0 ✔️  - 17  0 💤 ±0  0 ±0 
0 runs   - 18  0 ✔️  - 18  0 💤 ±0  0 ±0 

Results for commit 8a83fbc. ± Comparison against base commit 5e38a37.

@jcsiadal
Copy link
Contributor Author

jcsiadal commented Apr 2, 2023

@adrianreber Is the devel repo not active for CI on RHEL-type builds?

@adrianreber
Copy link
Member

@adrianreber Is the devel repo not active for CI on RHEL-type builds?

No, so far it was not necessary. You can enable it.

CI can also not handle packages that dependent on each other. You would need to split this in two PRs.

Copy link

A friendly reminder that this PR had no activity for 30 days.

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

Successfully merging this pull request may close these issues.

2 participants