Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r27526 - trunk/orte/mca/plm/rsh
From: Jeff Squyres (jsquyres_at_[hidden])
Date: 2012-10-30 15:52:30


WAIT.

This contradicts the intent of what I said on the call this morning.

The point is that any env variable that begins with "OMPI_" is supposed to be propagated out to all the remote processes. It's a cheap/easy way for users to propagate their env variables to remote nodes (without needing to "mpirun -x" every variable they want to export).

Specifically, I should be able to do something like this:

-----
$ hostname
my_localhost
$ cat myscript
:
echo `hostname`: $OMPI_foo
$ export OMPI_foo=bar
$ cat hostfile
some_remote_host
$ mpirun --hostfile hostfile -np 1 myscript
some_remote_host: bar
$
-----

This behavior has been in OMPI for a long time; please do not take it out.

If exporting non-MCA OMPI_<foo> env variables causes the problem, then it's a side effect.

On Oct 30, 2012, at 3:40 PM, <svn-commit-mailer_at_[hidden]> wrote:

> Author: hjelmn (Nathan Hjelm)
> Date: 2012-10-30 15:40:04 EDT (Tue, 30 Oct 2012)
> New Revision: 27526
> URL: https://svn.open-mpi.org/trac/ompi/changeset/27526
>
> Log:
> fix bug in plm/rsh that could add extraneous mca options to the orted argv
>
> cmr:v1.7
>
> Text files modified:
> trunk/orte/mca/plm/rsh/plm_rsh_module.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> Modified: trunk/orte/mca/plm/rsh/plm_rsh_module.c
> ==============================================================================
> --- trunk/orte/mca/plm/rsh/plm_rsh_module.c Tue Oct 30 15:23:15 2012 (r27525)
> +++ trunk/orte/mca/plm/rsh/plm_rsh_module.c 2012-10-30 15:40:04 EDT (Tue, 30 Oct 2012) (r27526)
> @@ -586,7 +586,7 @@
> * only if they aren't already present
> */
> for (i = 0; NULL != environ[i]; ++i) {
> - if (0 == strncmp("OMPI_", environ[i], 5)) {
> + if (0 == strncmp("OMPI_MCA", environ[i], 8)) {
> /* check for duplicate in app->env - this
> * would have been placed there by the
> * cmd line processor. By convention, we
> _______________________________________________
> svn-full mailing list
> svn-full_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/svn-full

-- 
Jeff Squyres
jsquyres_at_[hidden]
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/