{"action":"created","issue":{"url":"https://api.github.com/repos/flux-framework/flux-sched/issues/77","labels_url":"https://api.github.com/repos/flux-framework/flux-sched/issues/77/labels{/name}","comments_url":"https://api.github.com/repos/flux-framework/flux-sched/issues/77/comments","events_url":"https://api.github.com/repos/flux-framework/flux-sched/issues/77/events","html_url":"https://github.com/flux-framework/flux-sched/pull/77","id":109187667,"number":77,"title":"jsc/etc: beefing up the jsc tests and adjusting for a recent lua binding churn","user":{"login":"dongahn","id":862123,"avatar_url":"https://avatars.githubusercontent.com/u/862123?v=3","gravatar_id":"","url":"https://api.github.com/users/dongahn","html_url":"https://github.com/dongahn","followers_url":"https://api.github.com/users/dongahn/followers","following_url":"https://api.github.com/users/dongahn/following{/other_user}","gists_url":"https://api.github.com/users/dongahn/gists{/gist_id}","starred_url":"https://api.github.com/users/dongahn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongahn/subscriptions","organizations_url":"https://api.github.com/users/dongahn/orgs","repos_url":"https://api.github.com/users/dongahn/repos","events_url":"https://api.github.com/users/dongahn/events{/privacy}","received_events_url":"https://api.github.com/users/dongahn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-30T21:56:48Z","updated_at":"2015-09-30T23:19:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/flux-framework/flux-sched/pulls/77","html_url":"https://github.com/flux-framework/flux-sched/pull/77","diff_url":"https://github.com/flux-framework/flux-sched/pull/77.diff","patch_url":"https://github.com/flux-framework/flux-sched/pull/77.patch"},"body":"This PR contains fixes for two distinct issues: 1) intermittent jsc test failures and 2) Travis CI build failures because `hostlist` and `posix` have moved to the flux namespace (e.g., `src/bindings/lua/flux/hostlist.so`). \r\n\r\n1) is captured in PR #64 along with other issues. I suspect the issue is due to some less precise synchronization mechanisms used between `flux-waitjob` and subsequent `flux-submit`s within the test script (`t/t1000-jsc.t`). To make synchronization air-tight, better synchronization support has been added within `flux-waitjob` (along with its own test case --- `t/t0002-waitjob.t`) and `t/t1000-jsc.t` has also been improved.  \r\n\r\n2) is captured in #76. To debug this better in the future, I've made a very minor modification to  `.travis.yml` such that I can see the environment variables that are exported by the `flux-core`'s deb builder script. And the other fixes are trivial changes in a sharness test script, Makefiles and lua files: e.g., `require 'hostlist'` to `require 'flux.hostlist'`. BTW, the reason that Travis CI has not detected these failures is that its test have been run with a cache. So occasional deleting of Travis caches should be encouraged to detection a condition like this.\r\n\r\nI've ran this PR on Travis CI for 10 times and all succeeded. "},"comment":{"url":"https://api.github.com/repos/flux-framework/flux-sched/issues/comments/144572048","html_url":"https://github.com/flux-framework/flux-sched/pull/77#issuecomment-144572048","issue_url":"https://api.github.com/repos/flux-framework/flux-sched/issues/77","id":144572048,"user":{"login":"dongahn","id":862123,"avatar_url":"https://avatars.githubusercontent.com/u/862123?v=3","gravatar_id":"","url":"https://api.github.com/users/dongahn","html_url":"https://github.com/dongahn","followers_url":"https://api.github.com/users/dongahn/followers","following_url":"https://api.github.com/users/dongahn/following{/other_user}","gists_url":"https://api.github.com/users/dongahn/gists{/gist_id}","starred_url":"https://api.github.com/users/dongahn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongahn/subscriptions","organizations_url":"https://api.github.com/users/dongahn/orgs","repos_url":"https://api.github.com/users/dongahn/repos","events_url":"https://api.github.com/users/dongahn/events{/privacy}","received_events_url":"https://api.github.com/users/dongahn/received_events","type":"User","site_admin":false},"created_at":"2015-09-30T23:19:46Z","updated_at":"2015-09-30T23:19:46Z","body":"I just merged #75 so let me rebase."}}