{"push_id":1203735345,"size":6,"distinct_size":4,"ref":"refs/heads/wip-check-turbostat-kill","head":"60a6e07ba3850f22458798bc809eafe090b65539","before":"82337e24d2f7b5e355ed87dbba48b84c8c15f253","commits":[{"sha":"9e0ecef70c9396521d3aa4ff802aa390d4b45d34","author":{"name":"Nick Dokos","email":"bc7f3fa5218f6d29e8a425496ff7c78b61b6d31f@redhat.com"},"message":"Fix pbench-agent-config-activate and add unit test.","distinct":false,"url":"https://api.github.com/repos/ndokos/pbench/commits/9e0ecef70c9396521d3aa4ff802aa390d4b45d34"},{"sha":"1cc0c802124738164e225c114356ee355b9cb30b","author":{"name":"Peter Portante","email":"ff9873574d55670d693c4b6ce6f4c2520c342721@gmail.com"},"message":"Merge pull request #282 from ndokos/wip-activate-unit-test\n\nFix pbench-agent-config-activate and add unit test.","distinct":false,"url":"https://api.github.com/repos/ndokos/pbench/commits/1cc0c802124738164e225c114356ee355b9cb30b"},{"sha":"b8100305a7cf93e219baced52ffd2e849a8bb37a","author":{"name":"Nick Dokos","email":"bc7f3fa5218f6d29e8a425496ff7c78b61b6d31f@redhat.com"},"message":"Safer kill: check for strange situations and deal with them.\n\nCheck that the pid corresponds to the tool before killing it.\nIf there is no process with that pid, eat the error.\nIf the kill succeeds, return success.\nOtherwise check if the process is still running: kill it forcibly\nif necessary, but return failure in that case.\n\npbench-kill-tools: Check that the list of pids to kill is not empty.","distinct":true,"url":"https://api.github.com/repos/ndokos/pbench/commits/b8100305a7cf93e219baced52ffd2e849a8bb37a"},{"sha":"5f86a4e689f2f6d473c6846a8d505077dd95b53b","author":{"name":"Nick Dokos","email":"bc7f3fa5218f6d29e8a425496ff7c78b61b6d31f@redhat.com"},"message":"Add unit tests for safe_kill.\n\nThe kvmstat, vmstat, numastat and turbostat test cases actually go\nthrough safe_kill.  The iostat and sar cases do not currently.\nAssuming that no problems are found with safe_kill, it will be\nported to the rest of the tools.","distinct":true,"url":"https://api.github.com/repos/ndokos/pbench/commits/5f86a4e689f2f6d473c6846a8d505077dd95b53b"},{"sha":"f5cd4ab37310ffacf1b4e63971e1ef3d34b2d404","author":{"name":"Nick Dokos","email":"bc7f3fa5218f6d29e8a425496ff7c78b61b6d31f@redhat.com"},"message":"Revert the change to perf.","distinct":true,"url":"https://api.github.com/repos/ndokos/pbench/commits/f5cd4ab37310ffacf1b4e63971e1ef3d34b2d404"},{"sha":"60a6e07ba3850f22458798bc809eafe090b65539","author":{"name":"Nick Dokos","email":"bc7f3fa5218f6d29e8a425496ff7c78b61b6d31f@redhat.com"},"message":"Use pidof -x to get list of pids.\n\nEach tool script belongs to a 3- or 4-deep process hierarchy of which\npidof returned one pid (and not necessarily the one that was stored in\nthe tool pid file); consequently, safe_kill would skip it and there were\ntools (and their screens) hanging around at the end.\n\nUse pidof -x to get the complete list.\n\nChange the relevant unit tests (and the mocked pidof) to accommodate\nthis change.","distinct":true,"url":"https://api.github.com/repos/ndokos/pbench/commits/60a6e07ba3850f22458798bc809eafe090b65539"}]}
{"actor":{"display_login":"ndokos"}}