Skip to content

v3.0.x: lustre fixes #3483

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

Merged
merged 4 commits into from
May 18, 2017
Merged

Conversation

jsquyres
Copy link
Member

@jsquyres jsquyres commented May 8, 2017

Corresponding master PR: #3455

Several --with-lustre configury fixes. See #3455 for explanation.

@edgargabriel Can you review? Thanks.

jsquyres added 4 commits May 8, 2017 13:51
We check for liblustreapi.h in OMPI_CHECK_LUSTRE, so this code was
commented out here.  Might as well fully delete it, since it's
redundant and dead.

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit c81bc50)
Signed-off-by: Jeff Squyres <[email protected]>
Make sure the default Autoconf "yes" value for $with_lustre when the
user specifies --with-lustre on the command line (without a value)
does not propagate down into the directory logic.

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit 8604273)
Signed-off-by: Jeff Squyres <[email protected]>
Minor style updates; nothing of real consequence.

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit eb89712)
Signed-off-by: Jeff Squyres <[email protected]>
Follow the OMPI bias: if a human requests feature X and configure
can't deliver feature X, abort and let the human figure it out.

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit c119759)
Signed-off-by: Jeff Squyres <[email protected]>
@jsquyres jsquyres added the bug label May 8, 2017
@jsquyres jsquyres added this to the v3.0.0 milestone May 8, 2017
@jsquyres jsquyres requested a review from edgargabriel May 8, 2017 21:05
@hppritcha
Copy link
Member

bot:mellanox:retest

@jsquyres
Copy link
Member Author

@hppritcha @bwbarrett Note that the v2.0.x and v2.x versions of this PR were merged (#3481 and #3482, respectively).

@bwbarrett bwbarrett merged commit ceae9e3 into open-mpi:v3.x May 18, 2017
@jsquyres jsquyres deleted the pr/v3.0.x/with-lustre-fixes branch May 18, 2017 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants