Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/96668/?format=api
https://patches.dpdk.org/api/patches/96668/?format=api", "web_url": "https://patches.dpdk.org/project/dts/patch/1628156514-12397-1-git-send-email-lijuan.tu@intel.com/", "project": { "id": 3, "url": "https://patches.dpdk.org/api/projects/3/?format=api", "name": "DTS", "link_name": "dts", "list_id": "dts.dpdk.org", "list_email": "dts@dpdk.org", "web_url": "", "scm_url": "git://dpdk.org/tools/dts", "webscm_url": "http://git.dpdk.org/tools/dts/", "list_archive_url": "https://inbox.dpdk.org/dts", "list_archive_url_format": "https://inbox.dpdk.org/dts/{}", "commit_url_format": "" }, "msgid": "<1628156514-12397-1-git-send-email-lijuan.tu@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dts/1628156514-12397-1-git-send-email-lijuan.tu@intel.com", "date": "2021-08-05T09:41:54", "name": "[v2] framework: fix session output", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "1ac6a6334ef492ea1565132300dd925508f1c14d", "submitter": { "id": 1098, "url": "https://patches.dpdk.org/api/people/1098/?format=api", "name": "Tu, Lijuan", "email": "lijuan.tu@intel.com" }, "delegate": null, "mbox": "https://patches.dpdk.org/project/dts/patch/1628156514-12397-1-git-send-email-lijuan.tu@intel.com/mbox/", "series": [ { "id": 18193, "url": "https://patches.dpdk.org/api/series/18193/?format=api", "web_url": "https://patches.dpdk.org/project/dts/list/?series=18193", "date": "2021-08-05T09:41:54", "name": "[v2] framework: fix session output", "version": 2, "mbox": "https://patches.dpdk.org/series/18193/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/96668/comments/", "check": "fail", "checks": "https://patches.dpdk.org/api/patches/96668/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dts-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 164C1A0C43;\n\tThu, 5 Aug 2021 03:52:08 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id D6CA14014F;\n\tThu, 5 Aug 2021 03:52:07 +0200 (CEST)", "from mga06.intel.com (mga06.intel.com [134.134.136.31])\n by mails.dpdk.org (Postfix) with ESMTP id 666E740143\n for <dts@dpdk.org>; Thu, 5 Aug 2021 03:52:06 +0200 (CEST)", "from fmsmga008.fm.intel.com ([10.253.24.58])\n by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 04 Aug 2021 18:52:04 -0700", "from unknown (HELO dpdk-lijuan.sh.intel.com) ([10.67.119.192])\n by fmsmga008.fm.intel.com with ESMTP; 04 Aug 2021 18:52:03 -0700" ], "X-IronPort-AV": [ "E=McAfee;i=\"6200,9189,10066\"; a=\"275100587\"", "E=Sophos;i=\"5.84,296,1620716400\"; d=\"scan'208\";a=\"275100587\"", "E=Sophos;i=\"5.84,296,1620716400\"; d=\"scan'208\";a=\"480489856\"" ], "X-ExtLoop1": "1", "From": "Lijuan Tu <lijuan.tu@intel.com>", "To": "dts@dpdk.org,\n\treshma.pattan@intel.com", "Cc": "Lijuan Tu <lijuan.tu@intel.com>", "Date": "Thu, 5 Aug 2021 17:41:54 +0800", "Message-Id": "<1628156514-12397-1-git-send-email-lijuan.tu@intel.com>", "X-Mailer": "git-send-email 1.8.3.1", "Subject": "[dts] [v2,] framework: fix session output", "X-BeenThere": "dts@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "test suite reviews and discussions <dts.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dts>,\n <mailto:dts-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dts/>", "List-Post": "<mailto:dts@dpdk.org>", "List-Help": "<mailto:dts-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dts>,\n <mailto:dts-request@dpdk.org?subject=subscribe>", "Errors-To": "dts-bounces@dpdk.org", "Sender": "\"dts\" <dts-bounces@dpdk.org>" }, "content": "* it should flush buffer before getting output\n* fix strip space in all session output\n* fix some output don't have \\r\\n before [PEXPECT]\n\nSigned-off-by: Lijuan Tu <lijuan.tu@intel.com>\nReported-by: Reshma Pattan <reshma.pattan@intel.com>\n---\n framework/crb.py | 7 ++++---\n framework/ssh_pexpect.py | 17 ++++++++---------\n 2 files changed, 12 insertions(+), 12 deletions(-)", "diff": "diff --git a/framework/crb.py b/framework/crb.py\nindex 3964e21..e029e71 100644\n--- a/framework/crb.py\n+++ b/framework/crb.py\n@@ -94,10 +94,11 @@ class Crb(object):\n \n # sometimes there will be no alt_session like VM dut\n if alt_session and self.alt_session:\n- return self.alt_session.session.send_expect(cmds, expected,\n+ out = self.alt_session.session.send_expect(cmds, expected,\n timeout, verify)\n-\n- return self.session.send_expect(cmds, expected, timeout, verify)\n+ else:\n+ out = self.session.send_expect(cmds, expected, timeout, verify)\n+ return out.strip()\n \n def create_session(self, name=\"\"):\n \"\"\"\ndiff --git a/framework/ssh_pexpect.py b/framework/ssh_pexpect.py\nindex fbc7dd4..adc97b5 100644\n--- a/framework/ssh_pexpect.py\n+++ b/framework/ssh_pexpect.py\n@@ -78,6 +78,8 @@ class SSHPexpect:\n self.clean_session()\n self.session.PROMPT = expected\n self.__sendline(command)\n+ # flush buffer before prompt, else buffer may impact output\n+ self.__flush()\n self.__prompt(command, timeout)\n aware_keyintr()\n \n@@ -127,13 +129,13 @@ class SSHPexpect:\n ignore_keyintr()\n self.session.PROMPT = self.magic_prompt\n try:\n+ self.__flush()\n self.session.prompt(timeout)\n except Exception as e:\n- pass\n+ raise(e)\n \n aware_keyintr()\n before = self.get_output_all()\n- self.__flush()\n \n return before\n \n@@ -159,16 +161,13 @@ class SSHPexpect:\n def get_output_before(self):\n if not self.isalive():\n raise SSHSessionDeadException(self.host)\n- before = self.session.before.rsplit('\\r\\n', 1)\n- if before[0] == \"[PEXPECT]\":\n- before[0] = \"\"\n \n- return before[0]\n+ before = self.session.before.strip('[PEXPECT]')\n+ return before.strip('\\r\\n')\n \n def get_output_all(self):\n- output = self.session.before\n- output.replace(\"[PEXPECT]\", \"\")\n- return output\n+ output = self.session.before.strip('[PEXPECT]')\n+ return output.strip('\\r\\n')\n \n def close(self, force=False):\n if force is True:\n", "prefixes": [ "v2" ] }{ "id": 96668, "url": "