Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/104948/?format=api
{ "id": 104948, "url": "http://patchwork.dpdk.org/api/patches/104948/?format=api", "web_url": "http://patchwork.dpdk.org/project/dts/patch/1638793303-2955-1-git-send-email-juraj.linkes@pantheon.tech/", "project": { "id": 3, "url": "http://patchwork.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": "<1638793303-2955-1-git-send-email-juraj.linkes@pantheon.tech>", "list_archive_url": "https://inbox.dpdk.org/dts/1638793303-2955-1-git-send-email-juraj.linkes@pantheon.tech", "date": "2021-12-06T12:21:43", "name": "[v1] framework/qemu_kvm: fix pylama errors", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "cdabc50cfc652783fc9770fcc805efacd5f8131c", "submitter": { "id": 1626, "url": "http://patchwork.dpdk.org/api/people/1626/?format=api", "name": "Juraj Linkeš", "email": "juraj.linkes@pantheon.tech" }, "delegate": null, "mbox": "http://patchwork.dpdk.org/project/dts/patch/1638793303-2955-1-git-send-email-juraj.linkes@pantheon.tech/mbox/", "series": [ { "id": 20878, "url": "http://patchwork.dpdk.org/api/series/20878/?format=api", "web_url": "http://patchwork.dpdk.org/project/dts/list/?series=20878", "date": "2021-12-06T12:21:43", "name": "[v1] framework/qemu_kvm: fix pylama errors", "version": 1, "mbox": "http://patchwork.dpdk.org/series/20878/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/104948/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/104948/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 9BBF7A034F;\n\tMon, 6 Dec 2021 13:21:47 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 92AC341169;\n\tMon, 6 Dec 2021 13:21:47 +0100 (CET)", "from lb.pantheon.sk (lb.pantheon.sk [46.229.239.20])\n by mails.dpdk.org (Postfix) with ESMTP id C0DF7410E5\n for <dts@dpdk.org>; Mon, 6 Dec 2021 13:21:45 +0100 (CET)", "from localhost (localhost [127.0.0.1])\n by lb.pantheon.sk (Postfix) with ESMTP id 56D28134296;\n Mon, 6 Dec 2021 13:21:45 +0100 (CET)", "from lb.pantheon.sk ([127.0.0.1])\n by localhost (lb.pantheon.sk [127.0.0.1]) (amavisd-new, port 10024)\n with ESMTP id nuTrvQlNqcjA; Mon, 6 Dec 2021 13:21:44 +0100 (CET)", "from service-node1.lab.pantheon.local (unknown [46.229.239.141])\n by lb.pantheon.sk (Postfix) with ESMTP id 300F9130F7E;\n Mon, 6 Dec 2021 13:21:44 +0100 (CET)" ], "X-Virus-Scanned": "amavisd-new at siecit.sk", "From": "=?utf-8?q?Juraj_Linke=C5=A1?= <juraj.linkes@pantheon.tech>", "To": "lijuan.tu@intel.com,\n\tohilyard@iol.unh.edu", "Cc": "dts@dpdk.org, =?utf-8?q?Juraj_Linke=C5=A1?= <juraj.linkes@pantheon.tech>", "Subject": "[PATCH v1] framework/qemu_kvm: fix pylama errors", "Date": "Mon, 6 Dec 2021 13:21:43 +0100", "Message-Id": "<1638793303-2955-1-git-send-email-juraj.linkes@pantheon.tech>", "X-Mailer": "git-send-email 2.7.4", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "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" }, "content": "Pylama found the following errors:\nframework/qemu_kvm.py:1498: [E] E0602 Undefined variable 'assgined_pcis_info' [pylint]\nframework/qemu_kvm.py:1508: [E] E0602 Undefined variable 'assinged_pcis_info' [pylint]\nframework/qemu_kvm.py:1508: [E] E1101 Instance of 'QEMUKvm' has no '__vf_has_been_assinged' member [pylint]\nframework/qemu_kvm.py:1587: [E] E1121 Too many positional arguments for method call [pylint]\nframework/qemu_kvm.py:1733: [E] E0213 Method should have \"self\" as first argument [pylint]\nframework/qemu_kvm.py:1750: [E] E1102 func is not callable [pylint]\n\nSigned-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>\n---\nLijuan, please add additional people to review if needed.\n---\n framework/qemu_kvm.py | 7 ++++---\n 1 file changed, 4 insertions(+), 3 deletions(-)", "diff": "diff --git a/framework/qemu_kvm.py b/framework/qemu_kvm.py\nindex 515a4c04..32cd668e 100644\n--- a/framework/qemu_kvm.py\n+++ b/framework/qemu_kvm.py\n@@ -1495,7 +1495,7 @@ class QEMUKvm(VirtBase):\n if self.__is_vf_pci(pci):\n assigned_pcis_info[pci]['is_vf'] = True\n pf_pci = self.__map_vf_to_pf(pci)\n- assgined_pcis_info[pci]['pf_pci'] = pf_pci\n+ assigned_pcis_info[pci]['pf_pci'] = pf_pci\n if self.virt_pool.alloc_vf_from_pf(vm=self.vm_name,\n pf_pci=pf_pci,\n *[pci]):\n@@ -1505,7 +1505,7 @@ class QEMUKvm(VirtBase):\n else:\n # check that if any VF of specified PF has been\n # used, raise exception\n- vf_pci = self.__vf_has_been_assinged(pci, **assinged_pcis_info)\n+ vf_pci = self.__vf_has_been_assigned(pci, **assigned_pcis_info)\n if vf_pci:\n raise Exception(\n \"Error: A VF [%s] generated by PF [%s] has \" %\n@@ -1584,7 +1584,7 @@ class QEMUKvm(VirtBase):\n \"\"\"\n Get IP which VM is connected by bridge.\n \"\"\"\n- out = self.control_command('ping', '60')\n+ out = self.control_command('ping')\n if not out:\n time.sleep(10)\n out = self.control_command('ifconfig')\n@@ -1730,6 +1730,7 @@ class QEMUKvm(VirtBase):\n \n return cores\n \n+ @staticmethod\n def handle_control_session(func):\n \"\"\"\n Wrapper function to handle serial port, must return serial to host session\n", "prefixes": [ "v1" ] }