Skip to content

Multiple commits#3567

Merged
rhc54 merged 6 commits intoopenpmix:v5.0from
rhc54:cmr50/up
Mar 13, 2025
Merged

Multiple commits#3567
rhc54 merged 6 commits intoopenpmix:v5.0from
rhc54:cmr50/up

Conversation

@rhc54
Copy link
Contributor

@rhc54 rhc54 commented Mar 13, 2025

Add new pub-lookup stress example

Try to more fully explore the publish-lookup race
condition

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit c6cf013)

Just ignore any lto settings

Rather than exiting configure with an error, print a warning
if lto-based directives are detected and ignore those flags.
This may help resolve rpm problems.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit af25b7c)

Have show_help output all directories tried

When outputting a show_help message that fails to find
the specified topic file, output all locations attempted
so the user can see what was done.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit f046625)

Extend the pubstress test and add it to CI

Check that we correctly unpublish data for given
persistence levels. Add the pubstress test to
the CI.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit b19f050)

Cleanup spawn and clarify attribute comment

The use of PMIX_PREFIX in a spawn request was getting
confused, so add some text clarifying it in pmix_common.h.

(char*) prefix to be used by an app to look for its
PMIx installation on remote nodes. A NULL
value indicates that no prefix is to be given

Cleanup the client spawn code to reflect that clarification.
Specifically, we are NOT prefixing the location of the
executable - we are instead asking the host to:

(a) setup the LD_LIBRARY_PATH to start with the prefix/lib
location, and

(b) push PMIX_PREFIX=<value given> into the app's environment
If the value given is NULL, then we want the host NOT to
assign any prefix to the app, even if there is a default
one at their level

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit fc0a55b)

Properly handle OPAL_PREFIX

When we see the OPAL_PREFIX envar, we need to not only
propagate it, but we also need to modify LD_LIBRARY_PATH
accordingly so that the app can find its components.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit c445afe)

rhc54 added 6 commits March 13, 2025 07:27
Try to more fully explore the publish-lookup race
condition

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit c6cf013)
Rather than exiting configure with an error, print a warning
if lto-based directives are detected and ignore those flags.
This may help resolve rpm problems.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit af25b7c)
When outputting a show_help message that fails to find
the specified topic file, output all locations attempted
so the user can see what was done.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit f046625)
Check that we correctly unpublish data for given
persistence levels. Add the pubstress test to
the CI.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit b19f050)
The use of PMIX_PREFIX in a spawn request was getting
confused, so add some text clarifying it in pmix_common.h.
```text
(char*) prefix to be used by an app to look for its
PMIx installation on remote nodes. A NULL
value indicates that no prefix is to be given
```
Cleanup the client spawn code to reflect that clarification.
Specifically, we are NOT prefixing the location of the
executable - we are instead asking the host to:

(a) setup the LD_LIBRARY_PATH to start with the prefix/lib
    location, and

(b) push `PMIX_PREFIX=<value given>` into the app's environment
    If the value given is NULL, then we want the host NOT to
    assign any prefix to the app, even if there is a default
    one at their level

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit fc0a55b)
When we see the OPAL_PREFIX envar, we need to not only
propagate it, but we also need to modify LD_LIBRARY_PATH
accordingly so that the app can find its components.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit c445afe)
@rhc54 rhc54 merged commit 94013e3 into openpmix:v5.0 Mar 13, 2025
18 checks passed
@rhc54 rhc54 deleted the cmr50/up branch March 13, 2025 13:40
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

Successfully merging this pull request may close these issues.

1 participant