Patch Detail

HTTP 200 OK
Allow: GET, PUT, PATCH, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

{
    "id": 93289,
    "url": "http://patchwork.dpdk.org/api/patches/93289/?format=api",
    "web_url": "http://patchwork.dpdk.org/project/dts/patch/1621288016-63122-1-git-send-email-lijuan.tu@intel.com/",
    "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": "<1621288016-63122-1-git-send-email-lijuan.tu@intel.com>",
    "list_archive_url": "https://inbox.dpdk.org/dts/1621288016-63122-1-git-send-email-lijuan.tu@intel.com",
    "date": "2021-05-17T21:46:56",
    "name": "pktgen_ixia: fix a issue taking ownership failed",
    "commit_ref": null,
    "pull_url": null,
    "state": "accepted",
    "archived": false,
    "hash": "afbc866947d8f71f1cb013022d540b0b141e0a17",
    "submitter": {
        "id": 1098,
        "url": "http://patchwork.dpdk.org/api/people/1098/?format=api",
        "name": "Tu, Lijuan",
        "email": "lijuan.tu@intel.com"
    },
    "delegate": null,
    "mbox": "http://patchwork.dpdk.org/project/dts/patch/1621288016-63122-1-git-send-email-lijuan.tu@intel.com/mbox/",
    "series": [
        {
            "id": 17006,
            "url": "http://patchwork.dpdk.org/api/series/17006/?format=api",
            "web_url": "http://patchwork.dpdk.org/project/dts/list/?series=17006",
            "date": "2021-05-17T21:46:56",
            "name": "pktgen_ixia: fix a issue taking ownership failed",
            "version": 1,
            "mbox": "http://patchwork.dpdk.org/series/17006/mbox/"
        }
    ],
    "comments": "http://patchwork.dpdk.org/api/patches/93289/comments/",
    "check": "pending",
    "checks": "http://patchwork.dpdk.org/api/patches/93289/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 B9A92A0A02;\n\tMon, 17 May 2021 15:49:32 +0200 (CEST)",
            "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 9E69740F35;\n\tMon, 17 May 2021 15:49:32 +0200 (CEST)",
            "from mga14.intel.com (mga14.intel.com [192.55.52.115])\n by mails.dpdk.org (Postfix) with ESMTP id 1B8A14014E\n for <dts@dpdk.org>; Mon, 17 May 2021 15:49:30 +0200 (CEST)",
            "from fmsmga008.fm.intel.com ([10.253.24.58])\n by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 17 May 2021 06:49:22 -0700",
            "from dpdk-lijuan.sh.intel.com ([10.67.119.192])\n by fmsmga008.fm.intel.com with ESMTP; 17 May 2021 06:49:21 -0700"
        ],
        "IronPort-SDR": [
            "\n JKtGHkOmPx+3OSDmQXWIgTIiTXztOILtplDfch+M4lkhCUEbdc71C9cGuTrKKVU5NPdm0i1lx6\n tBmW5i3T0giw==",
            "\n iKkaOSHs7hVmDqjETaV2jHKvhtaGXJDC8D3k4IapxJzqn9eM77/k7B31TTpgjOGOhpAQtwfIcK\n zzKm6bym7zxQ=="
        ],
        "X-IronPort-AV": [
            "E=McAfee;i=\"6200,9189,9986\"; a=\"200157669\"",
            "E=Sophos;i=\"5.82,307,1613462400\"; d=\"scan'208\";a=\"200157669\"",
            "E=Sophos;i=\"5.82,307,1613462400\"; d=\"scan'208\";a=\"437987391\""
        ],
        "X-ExtLoop1": "1",
        "From": "Lijuan Tu <lijuan.tu@intel.com>",
        "To": "dts@dpdk.org",
        "Cc": "Lijuan Tu <lijuan.tu@intel.com>",
        "Date": "Tue, 18 May 2021 05:46:56 +0800",
        "Message-Id": "<1621288016-63122-1-git-send-email-lijuan.tu@intel.com>",
        "X-Mailer": "git-send-email 1.8.3.1",
        "Subject": "[dts] [PATCH] pktgen_ixia: fix a issue taking ownership failed",
        "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": "DTS has to take ownership of an ixia port, then can operate it.\nIf the port is owned by other user, DTS has to take ownership forcedly.\nIt seems DTS intend to do above but with wrong code implement.\n\nSigned-off-by: Lijuan Tu <lijuan.tu@intel.com>\n---\n framework/pktgen_ixia.py | 8 ++++----\n 1 file changed, 4 insertions(+), 4 deletions(-)",
    "diff": "diff --git a/framework/pktgen_ixia.py b/framework/pktgen_ixia.py\nindex 80c5cb5..0273f3a 100644\n--- a/framework/pktgen_ixia.py\n+++ b/framework/pktgen_ixia.py\n@@ -610,13 +610,13 @@ class Ixia(SSHConnection):\n                 for item in self.ports]),\n             \"% \", 10)\n         if out.strip()[-1] != '0':\n-            return False\n-        out = self.send_expect(\"ixTakeOwnership [list %s] force\" % ' '.join(\n+            self.logger.info(\"Force to take ownership:\")\n+            out = self.send_expect(\"ixTakeOwnership [list %s] force\" % ' '.join(\n             ['[list %d %d %d]' % (self.chasId, item['card'], item['port'])\n                 for item in self.ports]),\n             \"% \", 10)\n-        if out.strip()[-1] != '0':\n-            return False\n+            if out.strip()[-1] != '0':\n+                return False\n \n         return True\n \n",
    "prefixes": []
}