{"push_id":2409008326,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"14efab5b8c6607a8c9f9c504cd1a89929b4f422e","before":"3648fbee86afc86a11671bdf384de8af9bc99b39","commits":[{"sha":"8348fe9d43f244cd57f567852e735ac040dee964","author":{"name":"Nathan Cutler","email":"3707ad02c4b07d59901ae3003a7007b31affd51a@suse.com"},"message":"qa: common: dump systemd log when a stage fails\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":true,"url":"https://api.github.com/repos/SUSE/DeepSea/commits/8348fe9d43f244cd57f567852e735ac040dee964"},{"sha":"da1b4b4570aaef9c31a85a196f3ec3ac00fcac17","author":{"name":"Nathan Cutler","email":"3707ad02c4b07d59901ae3003a7007b31affd51a@suse.com"},"message":"qa: common: ensure minion test result can be parsed\n\nSalt 2017.7.2 has a \"feature\" where it outputs a traceback immediately after\nevery salt command. This traceback breaks the result reporting mechanism we use\nfor our test functions that run on minions. For example:\n\n2018-03-15T11:52:13.817 INFO:teuthology.orchestra.run.target217182141012.stdout:    + echo 'Result: OK'\n2018-03-15T11:52:13.886 INFO:teuthology.orchestra.run.target217182141012.stdout:    Result: OK[DEBUG   ] jid 20180315115204426028 found all minions {'target217182141012.teuthology'}\n2018-03-15T11:52:13.887 INFO:teuthology.orchestra.run.target217182141012.stdout:\n2018-03-15T11:52:13.966 INFO:teuthology.orchestra.run.target217182141012.stdout:Exception ignored in: <bound method AsyncReqMessageClient.__del__ of <salt.transport.zeromq.AsyncReqMessageClient object at 0x7f8344136e48>>\n2018-03-15T11:52:13.966 INFO:teuthology.orchestra.run.target217182141012.stdout:Traceback (most recent call last):\n2018-03-15T11:52:13.966 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"/usr/lib/python3.6/site-packages/salt/transport/zeromq.py\", line 928, in __del__\n2018-03-15T11:52:13.966 INFO:teuthology.orchestra.run.target217182141012.stdout:    self.destroy()\n2018-03-15T11:52:13.967 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"/usr/lib/python3.6/site-packages/salt/transport/zeromq.py\", line 919, in destroy\n2018-03-15T11:52:13.967 INFO:teuthology.orchestra.run.target217182141012.stdout:    self.stream.io_loop.remove_handler(self.stream.socket)\n2018-03-15T11:52:13.967 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"/usr/lib64/python3.6/site-packages/tornado/ioloop.py\", line 734, in remove_handler\n2018-03-15T11:52:13.967 INFO:teuthology.orchestra.run.target217182141012.stdout:    fd, obj = self.split_fd(fd)\n2018-03-15T11:52:13.967 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"/usr/lib64/python3.6/site-packages/tornado/ioloop.py\", line 656, in split_fd\n2018-03-15T11:52:13.967 INFO:teuthology.orchestra.run.target217182141012.stdout:    return fd.fileno(), fd\n2018-03-15T11:52:13.968 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"/usr/lib64/python3.6/site-packages/zmq/sugar/socket.py\", line 157, in fileno\n2018-03-15T11:52:13.968 INFO:teuthology.orchestra.run.target217182141012.stdout:    return self.FD\n2018-03-15T11:52:13.968 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"/usr/lib64/python3.6/site-packages/zmq/sugar/attrsettr.py\", line 45, in __getattr__\n2018-03-15T11:52:13.968 INFO:teuthology.orchestra.run.target217182141012.stdout:    return self._get_attr_opt(upper_key, opt)\n2018-03-15T11:52:13.968 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"/usr/lib64/python3.6/site-packages/zmq/sugar/attrsettr.py\", line 49, in _get_attr_opt\n2018-03-15T11:52:13.968 INFO:teuthology.orchestra.run.target217182141012.stdout:    return self.get(opt)\n2018-03-15T11:52:13.968 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"zmq/backend/cython/socket.pyx\", line 477, in zmq.backend.cython.socket.Socket.get\n2018-03-15T11:52:13.969 INFO:teuthology.orchestra.run.target217182141012.stdout:  File \"zmq/backend/cython/socket.pyx\", line 137, in zmq.backend.cython.socket._check_closed\n2018-03-15T11:52:13.969 INFO:teuthology.orchestra.run.target217182141012.stdout:zmq.error.ZMQError: Socket operation on non-socket\n2018-03-15T11:52:13.987 INFO:teuthology.orchestra.run.target217182141012.stderr:++ grep -o -P '(?<=Result: )(OK|NOT_OK)$' /tmp/test_script.log\n2018-03-15T11:52:13.987 INFO:teuthology.orchestra.run.target217182141012.stderr:++ head -1\n2018-03-15T11:52:13.989 INFO:teuthology.orchestra.run.target217182141012.stderr:+ local RESULT=\n2018-03-15T11:52:13.990 INFO:teuthology.orchestra.run.target217182141012.stderr:+ test x = xOK\n2018-03-15T11:52:13.997 ERROR:teuthology.run_tasks:Saw exception from tasks.\nTraceback (most recent call last):\n  File \"/home/ubuntu/teuthology/teuthology/run_tasks.py\", line 89, in run_tasks\n    manager.__enter__()\n  File \"/home/ubuntu/teuthology/teuthology/task/__init__.py\", line 121, in __enter__\n    self.begin()\n  File \"/home/ubuntu/src/github.com_SUSE_ceph_ses6/qa/tasks/deepsea.py\", line 196, in begin\n    cmd\n  File \"/home/ubuntu/teuthology/teuthology/orchestra/remote.py\", line 193, in run\n    r = self._runner(client=self.ssh, name=self.shortname, **kwargs)\n  File \"/home/ubuntu/teuthology/teuthology/orchestra/run.py\", line 403, in run\n    r.wait()\n  File \"/home/ubuntu/teuthology/teuthology/orchestra/run.py\", line 166, in wait\n    label=self.label)\nCommandFailedError: Command failed on target217182141012 with status 1: \"sudo sh -c 'cd DeepSea/qa ; suites/basic/health-nfs-ganesha.sh --fsal=both'\"\n\nThe breakage occurs due to the redirection of both stdout and stderr into the\nsame file, causing two processes to write to the same file at the same times.\nIn practice, the output could have been \"Result: O[DEBUG ]...K\" just as easily\nas \"Result: OK[DEBUG ]...\", rendering any regex matching meaningless.\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":true,"url":"https://api.github.com/repos/SUSE/DeepSea/commits/da1b4b4570aaef9c31a85a196f3ec3ac00fcac17"},{"sha":"14efab5b8c6607a8c9f9c504cd1a89929b4f422e","author":{"name":"Eric Jackson","email":"70fd0b24ece118bb33ef16bf15e645b66c710045@gmail.com"},"message":"Merge pull request #1014 from smithfarm/wip-qa-journalctl\n\nqa: common: dump systemd log when a stage fails","distinct":true,"url":"https://api.github.com/repos/SUSE/DeepSea/commits/14efab5b8c6607a8c9f9c504cd1a89929b4f422e"}]}
{"actor":{"display_login":"swiftgist"}}