{"push_id":1216091421,"size":17,"distinct_size":17,"ref":"refs/heads/master","head":"756be395b1340f0d85fb4c141791cd1ca7d32ecd","before":"b423f16f2b13e4c14d35e89c841658be0684f58b","commits":[{"sha":"483823637cabecf30e56e0f9fe957531b9ddb3eb","author":{"name":"Mark Ryan","email":"ec2b6a3007fe920337492f22f33b64d631e4f23a@intel.com"},"message":"ciao-launcher: Send stats every 6 seconds\n\nPreviously stats were sent every 30 seconds by launcher.  This commit\nchanges the period to 6 seconds.  Instance resources are still computed\nevery 30 seconds and so will only update every 30 seconds in the UI.\nInstance state, however, should now get updated every 6 seconds.\n\nFixes #282\n\nSigned-off-by: Mark Ryan <mark.d.ryan@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/483823637cabecf30e56e0f9fe957531b9ddb3eb"},{"sha":"31a968834f685963432b71f189eaafd086be2eb6","author":{"name":"Mark Ryan","email":"ec2b6a3007fe920337492f22f33b64d631e4f23a@intel.com"},"message":"ciao-launcher: Ensure cpu profile and logs are flushed\n\nLauncher was mistakenly using defer to flush logs and to stop\nthe CPU profiling when shutting down.  However, as launcher is\nquit by calling os.Exit, deferred functions do not get called.\nThis was leading to incomplete CPU profiles being generated.\n\nFixes: #163\n\nSigned-off-by: Mark Ryan <mark.d.ryan@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/31a968834f685963432b71f189eaafd086be2eb6"},{"sha":"373c5058cdac4324adbc6c17185c42a03b4e07d2","author":{"name":"Kristen Carlson Accardi","email":"9398331d124c28724c6f2e963ff24a40c660fd37@intel.com"},"message":"Merge pull request #348 from markdryan/master\n\nciao-launcher: Update statistics interval and fix profiling","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/373c5058cdac4324adbc6c17185c42a03b4e07d2"},{"sha":"c97b38e884360e2a4fc7de49ef53fd015f5ad4a8","author":{"name":"Leoswaldo Macias","email":"0ea1e33edcce0c00c330bef1e469b65f107f5593@intel.com"},"message":"ciao-controller: make returnErrorCode func variadic\n\nreturnErrorCode function receives a message string  which it will return\nwith this design the message needed to be formatted before calling the\nfunction, making the function variadic for the message will enable the\nfunction to be call with the messages to be formatted in the function,\nso we don't have to preformat every time we call it.\n\nThe returnErrorCode func will now build the message string in the format\nspecified with the args passed to it as values.\n\nSigned-off-by: Leoswaldo Macias <leoswaldo.macias@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/c97b38e884360e2a4fc7de49ef53fd015f5ad4a8"},{"sha":"2b4f85702c62dfb9a6bced0095811bb0c2395ecf","author":{"name":"Leoswaldo Macias","email":"0ea1e33edcce0c00c330bef1e469b65f107f5593@intel.com"},"message":"ciao-controller: check instances in list belong to tenant\n\ntenantServerAction function was acting on instances when a list of\nservers was provided, the function now validates that if a servers\nlist is provided then ensure it belongs to the tenant specified\nbefore proceeding to the action.\n\nFixes #98\n\nSigned-off-by: Leoswaldo Macias <leoswaldo.macias@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/2b4f85702c62dfb9a6bced0095811bb0c2395ecf"},{"sha":"dc6a7208552711bd5738da4d76645628e688e98d","author":{"name":"markdryan","email":"5dc6744e7b965cd005e4b28e27c730a282971fa0@users.noreply.github.com"},"message":"Merge pull request #318 from leoswaldo/leoswaldo/issue/98\n\nciao-controller: check instances in list belong to tenant","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/dc6a7208552711bd5738da4d76645628e688e98d"},{"sha":"0ccf628b8741e56a6ad737c111bbd4e791ec04e4","author":{"name":"Tim Pepper","email":"9ca68eda3abf78fbf634adcaaa20e3fe0416096a@linux.intel.com"},"message":"ciao-controller: silence go -race report for TestStartWorkloadLaunchCNCI\n\nGo race is noticing the closure in the TestStartWorkloadLaunchCNCI() unit\ntest results in two separate threads reading and writing the same\nvariable.  This is resolved by use of a channel to transmit results\nfrom the goroutine back to the parent.\n\nSigned-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/0ccf628b8741e56a6ad737c111bbd4e791ec04e4"},{"sha":"7cf32c8ac7913aec9b1c11d9408c42e892927f6c","author":{"name":"Tim Pepper","email":"9ca68eda3abf78fbf634adcaaa20e3fe0416096a@linux.intel.com"},"message":"ssntp: silence go race report in ssntp client\n\nSsntp client.go threads call sync.WaitGroup functions which read\nand write the client.frameWg data member.  The waitgroups themselves\nintrinsically can be written and read from multiple contexts safely.\nBut that doesn't mean the counts are coherent.\n\nIt is possible that the Close() function starts to Wait() on the\nwaitgroup _just_ before a new frame comes in and calls Add(1).  Then\nthe wait is waiting on less than the intended count and can close the\nclient.frameRoutinesChannel which would result in a write to a\nclosed channel and a panic.  We're not hitting that panic, but that\npotential is probably what is being noted by go-race (does go-race not\nhave a VERBOSE option?).\n\nThere already is a client.status variable that is toggled to ssntpClosed\nstate.  But it is only checked in the error path of the frame processor\nnot in the non-error path.  There's a slight gap there for a race, which\nis then fixed by grabbing the client.status lock, checking for ssntpClosed\nstatus and returning rather than doing the Add(1) and processing the new\nframe (ie: discard frames that come just in the \"closing\" window).\n\nThis was observed in ciao-controller's TestDeleteInstance()\nunit test when run with go-race.  That test no longer reports\nerror in go-race after the patch.\n\nSigned-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/7cf32c8ac7913aec9b1c11d9408c42e892927f6c"},{"sha":"57f6d19248278701dffa324a06903c8a6e5fb969","author":{"name":"markdryan","email":"5dc6744e7b965cd005e4b28e27c730a282971fa0@users.noreply.github.com"},"message":"Merge pull request #350 from tpepper/unittests\n\nUnittests: race fixes","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/57f6d19248278701dffa324a06903c8a6e5fb969"},{"sha":"c88d0ef4b3b678a8cdbb035da79338dde30a9e8c","author":{"name":"Mark Ryan","email":"ec2b6a3007fe920337492f22f33b64d631e4f23a@intel.com"},"message":"ciao-launcher: Add TestExtractImageInfo\n\nAdd a new unit test case to test the code that extracts information from\nqemu-img info.\n\nPartial fix for issue #1\n\nSigned-off-by: Mark Ryan <mark.d.ryan@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/c88d0ef4b3b678a8cdbb035da79338dde30a9e8c"},{"sha":"a8d0b082716dcb329c7bd60592b8b508d590cb1e","author":{"name":"Mark Ryan","email":"ec2b6a3007fe920337492f22f33b64d631e4f23a@intel.com"},"message":"ciao-launcher: Add TestGenerateQEMULaunchParams\n\nAdd a new unit test case to test the code that generates qemu launch\nparameters.\n\nPartial fix for issue #1\n\nSigned-off-by: Mark Ryan <mark.d.ryan@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/a8d0b082716dcb329c7bd60592b8b508d590cb1e"},{"sha":"4583c0e27de4d47d4d8d7a6a9c4690128a99173e","author":{"name":"Mark Ryan","email":"ec2b6a3007fe920337492f22f33b64d631e4f23a@intel.com"},"message":"ciao-launcher: Add unit tests for the qmp go routines\n\nThis commit adds four unit tests\n\n- TestQmpConnectBadSocket\n- TestQmpLost\n- TestQmpShutdown\n- TestQmpConnect\n\nIt is a partial fix for issue #1\n\nSigned-off-by: Mark Ryan <mark.d.ryan@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/4583c0e27de4d47d4d8d7a6a9c4690128a99173e"},{"sha":"051179be4e42a73d42e7a37118168777ae9528be","author":{"name":"Kristen Carlson Accardi","email":"9398331d124c28724c6f2e963ff24a40c660fd37@intel.com"},"message":"Merge pull request #362 from markdryan/master\n\nciao-launcher: Add qemu_test.go","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/051179be4e42a73d42e7a37118168777ae9528be"},{"sha":"eb3d54341140a213af6a12a48f4949f1b5f9abf7","author":{"name":"Mark Ryan","email":"ec2b6a3007fe920337492f22f33b64d631e4f23a@intel.com"},"message":"ciao-launcher: Enable tracing\n\nThis commit adds tracing support to ciao-launcher.  When built with the\nprofile tag a new flag is added called --trace.  This flag can be used\nto provide a file into which trace data can be written.  The data can\nbe analysed at a later stage using go tool trace.\n\nSigned-off-by: Mark Ryan <mark.d.ryan@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/eb3d54341140a213af6a12a48f4949f1b5f9abf7"},{"sha":"c6dcd317625fafd116a54368850d77e31e31b598","author":{"name":"Munoz, Obed N","email":"4776e41e150fa7d0108df6f563da65b12ffe07fc@intel.com"},"message":"ciao-cli: Add tenant name evironment variable\n\nWhen user is part of more than one project (tenant), it\nallows to use environment variables for passing the proper\nproject name instead of using the ``tenant-name`` or ``tenant-id``\nparameters in ``ciao-cli`` command. It would verify if the\n``CIAO_TENANT_NAME`` environment variable exists, it would use it;\nif not, everything would normally proceed.\n\nSigned-off-by: Munoz, Obed N <obed.n.munoz@intel.com>","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/c6dcd317625fafd116a54368850d77e31e31b598"},{"sha":"6a16a5abe503ecaa4663f2db4f91fc47aa31e77c","author":{"name":"Tim Pepper","email":"9ca68eda3abf78fbf634adcaaa20e3fe0416096a@linux.intel.com"},"message":"Merge pull request #363 from markdryan/master\n\nciao-launcher: Enable tracing","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/6a16a5abe503ecaa4663f2db4f91fc47aa31e77c"},{"sha":"756be395b1340f0d85fb4c141791cd1ca7d32ecd","author":{"name":"Tim Pepper","email":"9ca68eda3abf78fbf634adcaaa20e3fe0416096a@linux.intel.com"},"message":"Merge pull request #368 from obedmr/ciao-cli\n\nciao-cli: Add tenant name evironment variable","distinct":true,"url":"https://api.github.com/repos/mrkz/ciao/commits/756be395b1340f0d85fb4c141791cd1ca7d32ecd"}]}
{"actor":{"display_login":"mrkz"}}