{"action":"created","issue":{"url":"https://api.github.com/repos/trilinos/Trilinos/issues/3500","repository_url":"https://api.github.com/repos/trilinos/Trilinos","labels_url":"https://api.github.com/repos/trilinos/Trilinos/issues/3500/labels{/name}","comments_url":"https://api.github.com/repos/trilinos/Trilinos/issues/3500/comments","events_url":"https://api.github.com/repos/trilinos/Trilinos/issues/3500/events","html_url":"https://github.com/trilinos/Trilinos/pull/3500","id":363720736,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE4MDgzOTc2","number":3500,"title":"Tpetra:  Added extra CMake logic for setting Tpetra_ASSUME_CUDA_AWARE_MPI","user":{"login":"kddevin","id":9597421,"node_id":"MDQ6VXNlcjk1OTc0MjE=","avatar_url":"https://avatars0.githubusercontent.com/u/9597421?v=4","gravatar_id":"","url":"https://api.github.com/users/kddevin","html_url":"https://github.com/kddevin","followers_url":"https://api.github.com/users/kddevin/followers","following_url":"https://api.github.com/users/kddevin/following{/other_user}","gists_url":"https://api.github.com/users/kddevin/gists{/gist_id}","starred_url":"https://api.github.com/users/kddevin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kddevin/subscriptions","organizations_url":"https://api.github.com/users/kddevin/orgs","repos_url":"https://api.github.com/users/kddevin/repos","events_url":"https://api.github.com/users/kddevin/events{/privacy}","received_events_url":"https://api.github.com/users/kddevin/received_events","type":"User","site_admin":false},"labels":[{"id":289606028,"node_id":"MDU6TGFiZWwyODk2MDYwMjg=","url":"https://api.github.com/repos/trilinos/Trilinos/labels/Tpetra","name":"Tpetra","color":"5319e7","default":false},{"id":318642986,"node_id":"MDU6TGFiZWwzMTg2NDI5ODY=","url":"https://api.github.com/repos/trilinos/Trilinos/labels/in%20progress","name":"in progress","color":"ededed","default":false}],"state":"open","locked":false,"assignee":{"login":"kddevin","id":9597421,"node_id":"MDQ6VXNlcjk1OTc0MjE=","avatar_url":"https://avatars0.githubusercontent.com/u/9597421?v=4","gravatar_id":"","url":"https://api.github.com/users/kddevin","html_url":"https://github.com/kddevin","followers_url":"https://api.github.com/users/kddevin/followers","following_url":"https://api.github.com/users/kddevin/following{/other_user}","gists_url":"https://api.github.com/users/kddevin/gists{/gist_id}","starred_url":"https://api.github.com/users/kddevin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kddevin/subscriptions","organizations_url":"https://api.github.com/users/kddevin/orgs","repos_url":"https://api.github.com/users/kddevin/repos","events_url":"https://api.github.com/users/kddevin/events{/privacy}","received_events_url":"https://api.github.com/users/kddevin/received_events","type":"User","site_admin":false},"assignees":[{"login":"kddevin","id":9597421,"node_id":"MDQ6VXNlcjk1OTc0MjE=","avatar_url":"https://avatars0.githubusercontent.com/u/9597421?v=4","gravatar_id":"","url":"https://api.github.com/users/kddevin","html_url":"https://github.com/kddevin","followers_url":"https://api.github.com/users/kddevin/followers","following_url":"https://api.github.com/users/kddevin/following{/other_user}","gists_url":"https://api.github.com/users/kddevin/gists{/gist_id}","starred_url":"https://api.github.com/users/kddevin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kddevin/subscriptions","organizations_url":"https://api.github.com/users/kddevin/orgs","repos_url":"https://api.github.com/users/kddevin/repos","events_url":"https://api.github.com/users/kddevin/events{/privacy}","received_events_url":"https://api.github.com/users/kddevin/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2018-09-25T18:55:45Z","updated_at":"2018-09-25T19:03:57Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/trilinos/Trilinos/pulls/3500","html_url":"https://github.com/trilinos/Trilinos/pull/3500","diff_url":"https://github.com/trilinos/Trilinos/pull/3500.diff","patch_url":"https://github.com/trilinos/Trilinos/pull/3500.patch"},"body":"For issue #3405 \r\nThese changes add logic to CMake's configuration, increasing the number of locations in which ompi_info is found.  The changes allow ompi_info to be found for ATDM builds on white.\r\nATDM builds did not set MPI_CXX_COMPILER; instead, they set CMAKE_CXX_COMPILER.  The new logic looks for ompi_info in the same directory as CMAKE_CXX_COMPILER.\r\n\r\nI also added a test program, cudaMpi_CanaryTest.cpp.  It exercises MPI with CUDA directly, without any Kokkos or Trilinos data structures.  If this test fails with a given MPI installation (e.g., OpenMPI3 on white), the problem is likely with MPI, not with Tpetra.  Is there a better place in Trilinos for such tests, rather than in the packages?  Do we need a \"stableSystemPackage\" that includes tests like these, which have no dependence on Trilinos yet flag conditions that will cause Trilinos to fail?\r\n\r\nNote that once this change is merged, ATDM tests using OpenMPI3 on white will fail unless they explicitly set Tpetra_ASSUME_CUDA_AWARE_MPI=OFF.  _**Since OpenMPI3 is known to be bad on white, users should probably use OpenMPI2.**_\r\n\r\n@bartlettroscoe @fryeguy52  To save you headaches, I will not set AT:AUTOMERGE for this PR; feel free to merge when you are ready.\r\n\r\n<!---\r\nBe sure to select `develop` as the `base` branch against which to create this\r\npull request.  Only pull requests against `develop` will undergo Trilinos'\r\nautomated testing.  Pull requests against `master` will be ignored.\r\n\r\nProvide a general summary of your changes in the Title above.  If this pull\r\nrequest pertains to a particular package in Trilinos, it's worthwhile to start\r\nthe title with \"PackageName:  \".\r\n\r\nNote that anything between these delimiters is a comment that will not appear\r\nin the pull request description once created. Most areas in this message are\r\ncommented out and can be easily added by removing the comment delimiters.\r\n\r\nPlease make sure to mark:\r\n* Reviewers\r\n* Assignees\r\n* Labels\r\n\r\nReplace <teamName> below with the appropriate Trilinos package/team name.\r\n-->\r\n@trilinos/tpetra @bartlettroscoe @fryeguy52 \r\n\r\n\r\n\r\n## How Has This Been Tested?\r\n<!---\r\nPlease describe in detail how you tested your changes.  Include details of your\r\ntesting environment and the tests you ran to see how your change affects other\r\nareas of the code.  Consider including configure, build, and test log files.\r\n-->\r\nImplemented and exercised on white.\r\nExercised on linux workstation.\r\n\r\n## Checklist\r\n\r\n- [ x] My commit messages mention the appropriate GitHub issue numbers.\r\n- [ x] My code follows the code style of the affected package(s).\r\n- [ ] My change requires a change to the documentation.\r\n- [ ] I have updated the documentation accordingly.\r\n- [ ] I have read the [code contribution guidelines](../blob/master/CONTRIBUTING.md) for this project.\r\n- [ x] I have added tests to cover my changes.\r\n- [x ] All new and existing tests passed.\r\n- [x ] No new compiler warnings were introduced.\r\n- [ ] These changes break backwards compatibility.\r\n"},"comment":{"url":"https://api.github.com/repos/trilinos/Trilinos/issues/comments/424463218","html_url":"https://github.com/trilinos/Trilinos/pull/3500#issuecomment-424463218","issue_url":"https://api.github.com/repos/trilinos/Trilinos/issues/3500","id":424463218,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNDQ2MzIxOA==","user":{"login":"trilinos-autotester","id":26070492,"node_id":"MDQ6VXNlcjI2MDcwNDky","avatar_url":"https://avatars2.githubusercontent.com/u/26070492?v=4","gravatar_id":"","url":"https://api.github.com/users/trilinos-autotester","html_url":"https://github.com/trilinos-autotester","followers_url":"https://api.github.com/users/trilinos-autotester/followers","following_url":"https://api.github.com/users/trilinos-autotester/following{/other_user}","gists_url":"https://api.github.com/users/trilinos-autotester/gists{/gist_id}","starred_url":"https://api.github.com/users/trilinos-autotester/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/trilinos-autotester/subscriptions","organizations_url":"https://api.github.com/users/trilinos-autotester/orgs","repos_url":"https://api.github.com/users/trilinos-autotester/repos","events_url":"https://api.github.com/users/trilinos-autotester/events{/privacy}","received_events_url":"https://api.github.com/users/trilinos-autotester/received_events","type":"User","site_admin":false},"created_at":"2018-09-25T19:03:57Z","updated_at":"2018-09-25T19:03:57Z","author_association":"CONTRIBUTOR","body":"Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:\n\n<details>\n<summary>\n  <b>Pull Request Auto Testing STARTING</b> (click to expand)\n</summary>\n\n<h2>Build Information</h2>\n<h3>Test Name: Trilinos_pullrequest_intel_17.0.1</h3>\n<ul>\n  <li>Build Num: 949</li>\n  <li>Status: STARTED</li>\n</ul>\n\n<h4>Jenkins Parameters</h4>\n<table>\n  <tr>\n    <th>Parameter Name</th>\n    <th>Value</th>\n  </tr>\n\n  <tr>\n    <td>PULLREQUESTNUM</td>\n    <td>3500</td>\n  </tr>\n\n  <tr>\n    <td>TEST_REPO_ALIAS</td>\n    <td>TRILINOS</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_BRANCH</td>\n    <td>issue3405</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_REPO</td>\n    <td>https://github.com/trilinos/Trilinos</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_SHA</td>\n    <td>7900eab85eefe6de216a6510a65904ccf5ea5a2b</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_BRANCH</td>\n    <td>develop</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_REPO</td>\n    <td>https://github.com/trilinos/Trilinos</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_SHA</td>\n    <td>75bf699c8ce92476885a6073be975f63645f2146</td>\n  </tr>\n</table>\n\n<h2>Build Information</h2>\n<h3>Test Name: Trilinos_pullrequest_gcc_4.9.3</h3>\n<ul>\n  <li>Build Num: 1708</li>\n  <li>Status: STARTED</li>\n</ul>\n\n<h4>Jenkins Parameters</h4>\n<table>\n  <tr>\n    <th>Parameter Name</th>\n    <th>Value</th>\n  </tr>\n\n  <tr>\n    <td>COMPILER_MODULE</td>\n    <td>sems-gcc/4.9.3</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_BUILD_TYPE</td>\n    <td>Release</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_COMM_TYPE</td>\n    <td>MPI</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_DO_COMPLEX</td>\n    <td>OFF</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_JOB_TYPE</td>\n    <td>Experimental</td>\n  </tr>\n\n  <tr>\n    <td>MPI_MODULE</td>\n    <td>sems-openmpi/1.8.7</td>\n  </tr>\n\n  <tr>\n    <td>PULLREQUESTNUM</td>\n    <td>3500</td>\n  </tr>\n\n  <tr>\n    <td>TEST_REPO_ALIAS</td>\n    <td>TRILINOS</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_BRANCH</td>\n    <td>issue3405</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_REPO</td>\n    <td>https://github.com/trilinos/Trilinos</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_SHA</td>\n    <td>7900eab85eefe6de216a6510a65904ccf5ea5a2b</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_BRANCH</td>\n    <td>develop</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_REPO</td>\n    <td>https://github.com/trilinos/Trilinos</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_SHA</td>\n    <td>75bf699c8ce92476885a6073be975f63645f2146</td>\n  </tr>\n</table>\n\n<h2>Build Information</h2>\n<h3>Test Name: Trilinos_pullrequest_gcc_4.8.4</h3>\n<ul>\n  <li>Build Num: 1403</li>\n  <li>Status: STARTED</li>\n</ul>\n\n<h4>Jenkins Parameters</h4>\n<table>\n  <tr>\n    <th>Parameter Name</th>\n    <th>Value</th>\n  </tr>\n\n  <tr>\n    <td>COMPILER_MODULE</td>\n    <td>sems-gcc/4.8.4</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_BUILD_TYPE</td>\n    <td>Release</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_COMM_TYPE</td>\n    <td>MPI</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_DO_COMPLEX</td>\n    <td>OFF</td>\n  </tr>\n\n  <tr>\n    <td>JENKINS_JOB_TYPE</td>\n    <td>Experimental</td>\n  </tr>\n\n  <tr>\n    <td>MPI_MODULE</td>\n    <td>sems-openmpi/1.8.7</td>\n  </tr>\n\n  <tr>\n    <td>PULLREQUESTNUM</td>\n    <td>3500</td>\n  </tr>\n\n  <tr>\n    <td>TEST_REPO_ALIAS</td>\n    <td>TRILINOS</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_BRANCH</td>\n    <td>issue3405</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_REPO</td>\n    <td>https://github.com/trilinos/Trilinos</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_SOURCE_SHA</td>\n    <td>7900eab85eefe6de216a6510a65904ccf5ea5a2b</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_BRANCH</td>\n    <td>develop</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_REPO</td>\n    <td>https://github.com/trilinos/Trilinos</td>\n  </tr>\n\n  <tr>\n    <td>TRILINOS_TARGET_SHA</td>\n    <td>75bf699c8ce92476885a6073be975f63645f2146</td>\n  </tr>\n</table>\n\n<h2>Using Repos:</h2>\n\n<dl>\n  <dt>Repo: TRILINOS (trilinos/Trilinos)</dt>\n  <ul>\n    <li>Branch: issue3405</li>\n    <li>SHA: 7900eab85eefe6de216a6510a65904ccf5ea5a2b</li>\n    <li>Mode: TEST_REPO</li>\n  </ul>\n</dl>\n\nPull Request Author: kddevin\n</details>\n"}}
{"actor":{"display_login":"trilinos-autotester"}}