{"action":"created","issue":{"url":"https://api.github.com/repos/idaholab/moose/issues/12020","repository_url":"https://api.github.com/repos/idaholab/moose","labels_url":"https://api.github.com/repos/idaholab/moose/issues/12020/labels{/name}","comments_url":"https://api.github.com/repos/idaholab/moose/issues/12020/comments","events_url":"https://api.github.com/repos/idaholab/moose/issues/12020/events","html_url":"https://github.com/idaholab/moose/pull/12020","id":352176748,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA5NTUxOTA0","number":12020,"title":"Don't test with pthreads on 1D radial example","user":{"login":"brianmoose","id":13151520,"node_id":"MDQ6VXNlcjEzMTUxNTIw","avatar_url":"https://avatars1.githubusercontent.com/u/13151520?v=4","gravatar_id":"","url":"https://api.github.com/users/brianmoose","html_url":"https://github.com/brianmoose","followers_url":"https://api.github.com/users/brianmoose/followers","following_url":"https://api.github.com/users/brianmoose/following{/other_user}","gists_url":"https://api.github.com/users/brianmoose/gists{/gist_id}","starred_url":"https://api.github.com/users/brianmoose/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brianmoose/subscriptions","organizations_url":"https://api.github.com/users/brianmoose/orgs","repos_url":"https://api.github.com/users/brianmoose/repos","events_url":"https://api.github.com/users/brianmoose/events{/privacy}","received_events_url":"https://api.github.com/users/brianmoose/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-08-20T15:05:51Z","updated_at":"2018-08-21T14:53:03Z","closed_at":"2018-08-20T19:22:27Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/idaholab/moose/pulls/12020","html_url":"https://github.com/idaholab/moose/pull/12020","diff_url":"https://github.com/idaholab/moose/pull/12020.diff","patch_url":"https://github.com/idaholab/moose/pull/12020.patch"},"body":"New example introduced in #12015 doesn't skip when using pthreads, which breaks our min clang testing.\r\n`FAIL porous_flow/examples/co2_intercomparison/1Dradial.1Dradial FAILED`\r\nSee https://civet.inl.gov/job/222404/\r\n\r\nThis seems to be a somewhat frequent problem, we probably need to catch this on PRs...\r\n\r\n<!--\r\nINCLUDE THE FOLLOWING IN THE PR DESCRIPTION\r\n- Explain relevant design information for your change.\r\n- Follow the [Coding Standards](http://mooseframework.org/wiki/CodeStandards/).\r\n- Submit or improve [Test Cases](http://mooseframework.org/wiki/MooseTraining/testing/).\r\n- Reference a specific issue, place \"refs #<issue>\" or \"closes #<issue>\" (e.g., #closes #1234).\r\n-->\r\n"},"comment":{"url":"https://api.github.com/repos/idaholab/moose/issues/comments/414702972","html_url":"https://github.com/idaholab/moose/pull/12020#issuecomment-414702972","issue_url":"https://api.github.com/repos/idaholab/moose/issues/12020","id":414702972,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNDcwMjk3Mg==","user":{"login":"permcody","id":3036960,"node_id":"MDQ6VXNlcjMwMzY5NjA=","avatar_url":"https://avatars2.githubusercontent.com/u/3036960?v=4","gravatar_id":"","url":"https://api.github.com/users/permcody","html_url":"https://github.com/permcody","followers_url":"https://api.github.com/users/permcody/followers","following_url":"https://api.github.com/users/permcody/following{/other_user}","gists_url":"https://api.github.com/users/permcody/gists{/gist_id}","starred_url":"https://api.github.com/users/permcody/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/permcody/subscriptions","organizations_url":"https://api.github.com/users/permcody/orgs","repos_url":"https://api.github.com/users/permcody/repos","events_url":"https://api.github.com/users/permcody/events{/privacy}","received_events_url":"https://api.github.com/users/permcody/received_events","type":"User","site_admin":false},"created_at":"2018-08-21T14:53:03Z","updated_at":"2018-08-21T14:53:03Z","author_association":"MEMBER","body":"We might want to consider changing our PR testing a bit. If this\nThreadedUserObject becomes more widely used, it would be nice to catch\nthese easy to forget restrictions before they get merged.\n\nOn Tue, Aug 21, 2018 at 8:03 AM David Andrs <notifications@github.com>\nwrote:\n\n> Why do they pass the pthreads test that is run on the PR's as a matter of\n> interest?\n>\n> We do not use pthreads in any configuration when we test a PR. They are\n> however used in the minimal compiler target on a merge to devel. That's\n> when the problem shows up. Or you can add those targets by hand in civet\n> when you open a PR.\n>\n> —\n> You are receiving this because you commented.\n>\n>\n> Reply to this email directly, view it on GitHub\n> <https://github.com/idaholab/moose/pull/12020#issuecomment-414685245>, or mute\n> the thread\n> <https://github.com/notifications/unsubscribe-auth/AC5XIPC7rQN0m7jKl8QUDwgdUdH1gbbbks5uTBNAgaJpZM4WEL3c>\n> .\n>\n"}}
{"actor":{"display_login":"permcody"}}