{"push_id":779223837,"size":40,"distinct_size":40,"ref":"refs/heads/more-wreck-cleanup","head":"12fa150c67a6cfd4c77c0da748d7f441b0857ded","before":"d4d47e5e5740de82c28560803b06ead527dcfdd6","commits":[{"sha":"adcbdda7a2da61cbf1065159b5f217edc8cbf2de","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"wreck: remove more stale code from wrexec module\n\nThe rexec_session struct is no longer needed, since wrexec module\nnow just uses launch-and-forget of wrexecd.  Remove this and along\nwith it unnecessary heap activity and a rare memory leak.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/adcbdda7a2da61cbf1065159b5f217edc8cbf2de"},{"sha":"28dacb00dce9ee7140ed96d937dbdc8782b8ebed","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"cmd: fix return code of wreckrun on signal\n\nWhen remote processes in a wreckrun session were terminated by a signal,\nwreckrun was reporting the signal number as exit code. Fix this to report\nthe somewhat standard 128+signal to differentiate between killed by signal\nexited with a nonzero exit code.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/28dacb00dce9ee7140ed96d937dbdc8782b8ebed"},{"sha":"324b8f610014639a165204ad3029c897f58a137d","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"testsuite: fix wreck signal test\n\nThe test for signal forwarding in wreckrun was broken. For some reason\nexecuting under 'run_timeout' was breaking signal delivery or default\nsignal handling for flux-wreckrun.\n\nInstead of run_timeout use a shorter sleep to ensure we don't wait\nfor long if test fails, and run flux-wreckrun outside of any timeout\nwrapper.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/324b8f610014639a165204ad3029c897f58a137d"},{"sha":"333fbf526027f5262ede4b2f5eead8ec76ef4c9a","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"wreck: do not use resrc module\n\nUse kvs data from resource-hwloc instead of resrc module.\n\nThis change does nothing more than transition to gathering\ncores-per-rank information from resource.hwloc.by_rank instead\nof resrc.<rank>. No bugs are fixed, and no behavior changes\nshould have been made.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/333fbf526027f5262ede4b2f5eead8ec76ef4c9a"},{"sha":"82fdef351197eb39982cd6325451392595ffa795","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"doc: add hwloc to spellcheck dictionary","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/82fdef351197eb39982cd6325451392595ffa795"},{"sha":"7a4cf5687c9ceae11bd44894dd22088c701bcd73","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"doc: update required wreck modules\n\nThe required resource module for wreck is now resource-hwloc, not\nresrc. Update man page for flux-wreckrun to avoid any confusion.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/7a4cf5687c9ceae11bd44894dd22088c701bcd73"},{"sha":"fafa6459185b26f2a0e77d4dcb4c6fe296b58446","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"broker: do not load resrc module by default\n\nSince it no longer has users, remove the resrc module from\nbroker default module list.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/fafa6459185b26f2a0e77d4dcb4c6fe296b58446"},{"sha":"2099989b9e728f70157b5ffc6e96905b8ee3abc6","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"wreck: remove resrc module\n\nThe 'resrc' module is no longer used. Remove it.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/2099989b9e728f70157b5ffc6e96905b8ee3abc6"},{"sha":"6a99e8784bec56087b8b977bfec977c97b975569","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"wreck: remove deprecated calls from job module\n\nRemove all deprecated calls from the wreck 'job' module.\nA lot of changes were made in the sim.sched race workaround code,\nwhich I was unable to test...","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/6a99e8784bec56087b8b977bfec977c97b975569"},{"sha":"cbfa3c35258dea92b3f2cc7f981c4540b8fb7dc1","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"testsuite: improve wreck state test\n\nFor now, restrict test to checking for final 3 states\nof wreck jobs. Also, keep output in uniquely named files\nfor better debugging.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/cbfa3c35258dea92b3f2cc7f981c4540b8fb7dc1"},{"sha":"bb654c91069dbf4d2ae3e2f952b945de75593133","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"wreck: emit reserved state event\n\nIn job module, emit the `wreck.state.reserved` event when\ncreating a new job id under `lwj.<id>`. Stop and listen for\nour own event before responding to the job.create message\nto ensure synchronous behaviour.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/bb654c91069dbf4d2ae3e2f952b945de75593133"},{"sha":"2766ed065ce527faa7d7216bbf772979e68353fb","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"testsuite: wreck: check for reserved state","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/2766ed065ce527faa7d7216bbf772979e68353fb"},{"sha":"10f24fa3d478dd76a69ac5072052a3229413ca2d","author":{"email":"515aee1de6d55547211d8d97de1026df5c837e7f@gmail.com","name":"Mark Grondona"},"message":"wreck: check reactor return code in wrexecd\n\nLog non-zero return code from flux_reactor_start in wrexecd.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/10f24fa3d478dd76a69ac5072052a3229413ca2d"},{"sha":"a8f67d0fca3a2baf627ad685a5f2b7eb4465f818","author":{"email":"90eebc3ed9e472707552aa1f0850807e64b4e5e4@gmail.com","name":"Jim Garlick"},"message":"libflux/tmpdir: drop this class\n\nDrop flux_get_tmpdir(), flux_set_tmpdir(), a feeble attempt to\navoid duplicating FLUX_TMPDIR logic.  We are getting rid of FLUX_TMPDIR\nso this can go.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/a8f67d0fca3a2baf627ad685a5f2b7eb4465f818"},{"sha":"6073dc1e7fc67ac99b3d4035835c1d262079a7ff","author":{"email":"90eebc3ed9e472707552aa1f0850807e64b4e5e4@gmail.com","name":"Jim Garlick"},"message":"libflux/handle: flux_open() no default URI scheme\n\nInsist that FLUX_URI is set when calling flux_open()\nwith a NULL uri argument, or EINVAL.\n\nNo longer do we default to \"local://\" and then let the\nlocal connector read $FLUX_TMPDIR to construct the path.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/6073dc1e7fc67ac99b3d4035835c1d262079a7ff"},{"sha":"ad69ddd9f9a19a10f3bea8de2efcdb2f0312343a","author":{"email":"90eebc3ed9e472707552aa1f0850807e64b4e5e4@gmail.com","name":"Jim Garlick"},"message":"connectors/local: require path to be set\n\nIn the connector itself, don't accept a NULL path.\nThe full URI must be provided.\n\nIn modules/connector-local, obtain the URI of the local socket\nby asking the broker with flux_getattr (\"local-uri\") instead of\nderiving it from FLUX_TMPDIR.\n\nRename the socket from flux-api to local in both places (cleanup).","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/ad69ddd9f9a19a10f3bea8de2efcdb2f0312343a"},{"sha":"7f695f122bf068da5a5a8b8441ea6695f364250d","author":{"email":"90eebc3ed9e472707552aa1f0850807e64b4e5e4@gmail.com","name":"Jim Garlick"},"message":"broker: eliminate FLUX_TMPDIR, use FLUX_URI\n\nThe broker's environment is set up by its enclosing instance.\nSave the value of FLUX_URI, and make it available via\nflux_getattr (\"parent-uri\").  Then unset FLUX_URI so this is not\nused accidentally.\n\nEliminate FLUX_TMPDIR.  When launching locally, use $TMPDIR:-/tmp\nfor socket paths, and create a subdir structured as before for the\nlocal socket and broker pid.  Repeated local launches from the inital\nprogram of the first do not create sockets in subdirectories of\neach other.  They will be flat in $TMPDIR:-/tmp.\n\nHowever when launching recursively, use the parent-uri to create\nthe subdir for the local socket and broker pid in a subdirectory\nof the parent.  (Other sockets are wildcard paths shared via PMI\nso don't apply here).\n\nSet/override FLUX_URI for the initial program and cmb.exec.\nMake it available via flux_getattr (\"local-uri\").\n\nRename other attributes to avoid confsion between TBON hierarchy\nand instance hierarchy:\n  parent-uri -> tbon-parent-uri\n  request-uri -> tbon-request-uri","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/7f695f122bf068da5a5a8b8441ea6695f364250d"},{"sha":"b15f29f3c050648d32e5179b637354563dbed3c3","author":{"email":"90eebc3ed9e472707552aa1f0850807e64b4e5e4@gmail.com","name":"Jim Garlick"},"message":"cmd/flux-comms: eliminate Usage refs to old attrs\n\nThis just removes some attribute names in usage messages that\nare no longer correct, and probably shouldn't have been there\nanyway.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/b15f29f3c050648d32e5179b637354563dbed3c3"},{"sha":"c10b33a437ddc92d54731491f12d8c3c613b0760","author":{"email":"90eebc3ed9e472707552aa1f0850807e64b4e5e4@gmail.com","name":"Jim Garlick"},"message":"cmd/flux-config: test FLUX_URI not FLUX_TMPDIR\n\nTo determine whether to get config from the KVS or config file,\ntest the FLUX_URI variable, not the deprecated FLUX_TMPDIR variable.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/c10b33a437ddc92d54731491f12d8c3c613b0760"},{"sha":"1a66e9fbe4ec2c1b0a819c34ad6d6da690d83661","author":{"email":"90eebc3ed9e472707552aa1f0850807e64b4e5e4@gmail.com","name":"Jim Garlick"},"message":"cmd/flux: replace FLUX_TMPDIR with FLUX_URI\n\nDrop the -T,--tmpdir argument.\n\nTest thet FLUX_URI variable not FLUX_TMPDIR to determine whether\nto load config from the KVS or file.","distinct":true,"url":"https://api.github.com/repos/grondo/flux-core/commits/1a66e9fbe4ec2c1b0a819c34ad6d6da690d83661"}]}