)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"bc89ade18254cac21e12442a67ba135844b5f20c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Instead construct the string using psutil.Process.cmdline() method that"},{"line_number":10,"context_line":"will parse the data from kernel for us."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"TODO: add tests"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I780ea10bd1870c86c1a048071863843bc2993958"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a1ff146_7126cd17","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":15},"updated":"2017-03-15 03:49:00.000000000","message":"do we need more of them ?","commit_id":"8674f593a718e16d944198542ae7231558849603"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"16f0ef424ed0b51060690d942acc9db059388f78","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Instead construct the string using psutil.Process.cmdline() method that"},{"line_number":10,"context_line":"will parse the data from kernel for us."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"TODO: add tests"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I780ea10bd1870c86c1a048071863843bc2993958"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a1ff146_0c8d8231","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":15},"in_reply_to":"3a1ff146_7126cd17","updated":"2017-03-15 04:25:14.000000000","message":"Nah, I guess it\u0027s ok as-is.","commit_id":"8674f593a718e16d944198542ae7231558849603"}],"neutron/agent/linux/external_process.py":[{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"6e122018f307d617e14d1191916154bd754a62fc","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        if not pid:"},{"line_number":149,"context_line":"            return"},{"line_number":150,"context_line":"        try:"},{"line_number":151,"context_line":"            return \u0027 \u0027.join(psutil.Process(pid).cmdline())"},{"line_number":152,"context_line":"        except (psutil.NoSuchProcess, psutil.AccessDenied):"},{"line_number":153,"context_line":"            return"},{"line_number":154,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3a1ff146_d7484a30","line":151,"range":{"start_line":151,"start_character":48,"end_line":151,"end_character":57},"updated":"2017-03-15 08:46:40.000000000","message":"cmdline is not a callable in putils\u003d\u003d1.2.1 which is the minimum required version for functional testing [0] (and the one I have installed in my laptop so I realized :). Shall we bump it? Or will it still respect psutil\u003e\u003d3.2.2 from global-requirements.txt? [1]\n[0] https://github.com/openstack/neutron/blob/master/neutron/tests/functional/requirements.txt#L7\n[1] https://github.com/openstack/requirements/blob/master/global-requirements.txt#L172","commit_id":"e5225ae7708307dd30ea7c1c38d6f191465f9dca"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"ad9b323bc85ab7c0aac69df221c51312e034a9e6","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        if not pid:"},{"line_number":149,"context_line":"            return"},{"line_number":150,"context_line":"        try:"},{"line_number":151,"context_line":"            return \u0027 \u0027.join(psutil.Process(pid).cmdline())"},{"line_number":152,"context_line":"        except (psutil.NoSuchProcess, psutil.AccessDenied):"},{"line_number":153,"context_line":"            return"},{"line_number":154,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3a1ff146_8e94f137","line":151,"range":{"start_line":151,"start_character":48,"end_line":151,"end_character":57},"in_reply_to":"3a1ff146_d7484a30","updated":"2017-03-15 14:16:52.000000000","message":"Done","commit_id":"e5225ae7708307dd30ea7c1c38d6f191465f9dca"},{"author":{"_account_id":18051,"name":"David Shaughnessy","email":"david.shaughnessy@intel.com","username":"davidsha"},"change_message_id":"43aa1aeaf44a8e3ff1dc4ae2d951fe89e769fccb","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        if not pid:"},{"line_number":149,"context_line":"            return"},{"line_number":150,"context_line":"        try:"},{"line_number":151,"context_line":"            return \u0027 \u0027.join(psutil.Process(pid).cmdline())"},{"line_number":152,"context_line":"        except (psutil.NoSuchProcess, psutil.AccessDenied):"},{"line_number":153,"context_line":"            return"},{"line_number":154,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a1ff146_54a8c61f","line":151,"range":{"start_line":151,"start_character":20,"end_line":151,"end_character":21},"updated":"2017-03-15 15:00:53.000000000","message":"Is there meant to be a space inside?","commit_id":"9c7ca5cc020a93a65fc817ae960d221d43e0e924"},{"author":{"_account_id":18051,"name":"David Shaughnessy","email":"david.shaughnessy@intel.com","username":"davidsha"},"change_message_id":"c1f728fe7ecce108b676a936e29e0472ac43e3de","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        if not pid:"},{"line_number":149,"context_line":"            return"},{"line_number":150,"context_line":"        try:"},{"line_number":151,"context_line":"            return \u0027 \u0027.join(psutil.Process(pid).cmdline())"},{"line_number":152,"context_line":"        except (psutil.NoSuchProcess, psutil.AccessDenied):"},{"line_number":153,"context_line":"            return"},{"line_number":154,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a1ff146_2194ebe0","line":151,"range":{"start_line":151,"start_character":20,"end_line":151,"end_character":21},"in_reply_to":"3a1ff146_4f49135f","updated":"2017-03-15 15:37:29.000000000","message":"ack","commit_id":"9c7ca5cc020a93a65fc817ae960d221d43e0e924"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"adac1329f2c1ed5b32fba4c85ae489f9f78f5e19","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        if not pid:"},{"line_number":149,"context_line":"            return"},{"line_number":150,"context_line":"        try:"},{"line_number":151,"context_line":"            return \u0027 \u0027.join(psutil.Process(pid).cmdline())"},{"line_number":152,"context_line":"        except (psutil.NoSuchProcess, psutil.AccessDenied):"},{"line_number":153,"context_line":"            return"},{"line_number":154,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a1ff146_4f49135f","line":151,"range":{"start_line":151,"start_character":20,"end_line":151,"end_character":21},"in_reply_to":"3a1ff146_54a8c61f","updated":"2017-03-15 15:04:23.000000000","message":"@David: Yes, so that all elements in the list are separated by spaces reconstructing the actual commandline","commit_id":"9c7ca5cc020a93a65fc817ae960d221d43e0e924"}]}