Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/80708/?format=api
{ "id": 80708, "url": "http://patches.dpdk.org/api/patches/80708/?format=api", "web_url": "http://patches.dpdk.org/project/dts/patch/20201015094845.420591-1-qinx.sun@intel.com/", "project": { "id": 3, "url": "http://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": "<20201015094845.420591-1-qinx.sun@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dts/20201015094845.420591-1-qinx.sun@intel.com", "date": "2020-10-15T09:48:45", "name": "[V1] tests/TestSuite_cvl_fdir.py modify the log validation for wrong action according to the change of dpdk, dpdk commit id:cb97e595d9d3b316690c8d5ac688240d2ba1e551", "commit_ref": null, "pull_url": null, "state": "changes-requested", "archived": false, "hash": "a6ee33c8895d3e566809fcbf6d3a1c187ad8f080", "submitter": { "id": 1956, "url": "http://patches.dpdk.org/api/people/1956/?format=api", "name": "Sun, QinX", "email": "qinx.sun@intel.com" }, "delegate": null, "mbox": "http://patches.dpdk.org/project/dts/patch/20201015094845.420591-1-qinx.sun@intel.com/mbox/", "series": [ { "id": 12949, "url": "http://patches.dpdk.org/api/series/12949/?format=api", "web_url": "http://patches.dpdk.org/project/dts/list/?series=12949", "date": "2020-10-15T09:48:45", "name": "[V1] tests/TestSuite_cvl_fdir.py modify the log validation for wrong action according to the change of dpdk, dpdk commit id:cb97e595d9d3b316690c8d5ac688240d2ba1e551", "version": 1, "mbox": "http://patches.dpdk.org/series/12949/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/80708/comments/", "check": "pending", "checks": "http://patches.dpdk.org/api/patches/80708/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 dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id DF78DA04B7;\n\tWed, 14 Oct 2020 11:54:35 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id D6AE71DDC8;\n\tWed, 14 Oct 2020 11:54:34 +0200 (CEST)", "from mga06.intel.com (mga06.intel.com [134.134.136.31])\n by dpdk.org (Postfix) with ESMTP id 4769B1DDB9\n for <dts@dpdk.org>; Wed, 14 Oct 2020 11:54:32 +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 14 Oct 2020 02:54:30 -0700", "from unknown (HELO localhost.localdomain) ([10.240.183.61])\n by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 14 Oct 2020 02:54:28 -0700" ], "IronPort-SDR": [ "\n mA4C8BpVESjuB66OM2pbWDjYabl44Gfp4Hoa/h6GZgQRNKSJAzWCdyAMHJWQXiSNYJCH14QUkU\n tBiHbN2tfRRA==", "\n SpVhllDh1PhsFiS4h1OpQiDS/pvRiY6qiXMx0zzM8TI3iZeC+BmOiyJofOJcnayY3gp4Geq+6y\n OvTa10avjoeQ==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6000,8403,9773\"; a=\"227710132\"", "E=Sophos;i=\"5.77,374,1596524400\"; d=\"scan'208\";a=\"227710132\"", "E=Sophos;i=\"5.77,374,1596524400\"; d=\"scan'208\";a=\"299877572\"" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "From": "sunqin <qinx.sun@intel.com>", "To": "dts@dpdk.org", "Cc": "sunqin <qinx.sun@intel.com>", "Date": "Thu, 15 Oct 2020 17:48:45 +0800", "Message-Id": "<20201015094845.420591-1-qinx.sun@intel.com>", "X-Mailer": "git-send-email 2.25.1", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dts] [PATCH V1] tests/TestSuite_cvl_fdir.py modify the log\n\tvalidation for wrong action according to the change of dpdk,\n\tdpdk commit id:cb97e595d9d3b316690c8d5ac688240d2ba1e551", "X-BeenThere": "dts@dpdk.org", "X-Mailman-Version": "2.1.15", "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": "According to dpdk commit cb97e595d9d3b316690c8d5ac688240d2ba1e551 to\nmodify log validation for wrong action\n\nSigned-off-by: sunqin <qinx.sun@intel.com>\n---\n test_plans/cvl_fdir_test_plan.rst | 8 ++++----\n tests/TestSuite_cvl_fdir.py | 14 +++++++-------\n 2 files changed, 11 insertions(+), 11 deletions(-)", "diff": "diff --git a/test_plans/cvl_fdir_test_plan.rst b/test_plans/cvl_fdir_test_plan.rst\nindex 0e1521f..740cb37 100644\n--- a/test_plans/cvl_fdir_test_plan.rst\n+++ b/test_plans/cvl_fdir_test_plan.rst\n@@ -610,7 +610,7 @@ Note: there may be error message change.\n \n get the message::\n \n- Invalid input action: Invalid argument\n+ 'error' in message\n \n Discontinuous queues::\n \n@@ -2939,7 +2939,7 @@ Subcase 1: invalid parameters of queue index\n \n Failed to create flow, report message::\n \n- Invalid queue for FDIR.: Invalid argument\n+ 'error' in message\n \n 2. check there is no rule listed.\n \n@@ -3053,7 +3053,7 @@ Subcase 6: conflicted rules\n \n or::\n \n- Invalid input action number: Invalid argument\n+ 'error' in message\n \n 3. check there is only one rule listed.\n \n@@ -3066,7 +3066,7 @@ Subcase 7: conflicted actions\n \n Failed to create flow, report message::\n \n- Invalid input action: Invalid argument\n+ 'error' in message\n \n 2. check there is no rule listed.\n \ndiff --git a/tests/TestSuite_cvl_fdir.py b/tests/TestSuite_cvl_fdir.py\nindex e5a9c71..fd005b3 100644\n--- a/tests/TestSuite_cvl_fdir.py\n+++ b/tests/TestSuite_cvl_fdir.py\n@@ -2428,7 +2428,7 @@ class TestCVLFdir(TestCase):\n check_stats=False, check_msg='Bad arguments')\n self.validate_fdir_rule(\n 'flow validate 0 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions rss queues 2 3 end / rss / end',\n- check_stats=False, check_msg='Invalid input action number: Invalid argument')\n+ check_stats=False, check_msg='error')\n self.validate_fdir_rule(\n 'flow validate 0 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions passthru / mark id 4294967296 / end',\n check_stats=False, check_msg='Bad arguments')\n@@ -2580,7 +2580,7 @@ class TestCVLFdir(TestCase):\n def test_invalid_parameters_of_queue_index(self):\n rule = \"flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions queue index 64 / end\"\n out = self.dut.send_command(rule, timeout=1)\n- self.verify(\"Invalid input action: Invalid argument\" in out, \"failed with output: %s\" % out)\n+ self.verify(\"error\" in out, \"failed with output: %s\" % out)\n self.check_fdir_rule(port_id=0, stats=False)\n \n def test_invalid_parameters_of_rss_queues(self):\n@@ -2588,11 +2588,11 @@ class TestCVLFdir(TestCase):\n \"flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions rss queues 1 2 3 end / end\",\n \"flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions rss queues 0 end / end\",\n \"flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions rss queues end / end\"]\n- self.create_fdir_rule(rule=rule1, check_stats=False, msg='Invalid input action: Invalid argument')\n+ self.create_fdir_rule(rule=rule1, check_stats=False, msg='error')\n rule2 = 'flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions rss queues 1 2 3 5 end / end'\n- self.create_fdir_rule(rule2, check_stats=False, msg='Invalid input action: Invalid argument')\n+ self.create_fdir_rule(rule2, check_stats=False, msg='error')\n rule3 = 'flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions rss queues 63 64 end / end'\n- self.create_fdir_rule(rule3, check_stats=False, msg='Invalid input action: Invalid argument')\n+ self.create_fdir_rule(rule3, check_stats=False, msg='error')\n try:\n # restart testpmd\n self.dut.send_expect(\"quit\", \"# \")\n@@ -2655,13 +2655,13 @@ class TestCVLFdir(TestCase):\n 'flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv6 dst is CDCD:910A:2222:5498:8475:1111:3900:2021 / end actions mark / end',\n 'flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv6 dst is CDCD:910A:2222:5498:8475:1111:3900:2020 src is 2001::2 / udp src is 22 dst is 23 / end actions queue index 1 / mark / end']\n self.create_fdir_rule(rule2[0:4], check_stats=False, msg=\"Rule already exists!: File exists\", validate=False)\n- self.create_fdir_rule(rule2[4:7], check_stats=False, msg=\"Invalid input action number: Invalid argument\", validate=False)\n+ self.create_fdir_rule(rule2[4:7], check_stats=False, msg=\"error\", validate=False)\n self.create_fdir_rule(rule2[7:], check_stats=False, msg=\"Invalid input set: Invalid argument\", validate=False)\n self.check_fdir_rule(stats=True, rule_list=rule_li)\n \n def test_conflicted_actions(self):\n rule1 = \"flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 ttl is 2 tos is 4 / end actions queue index 1 / rss queues 2 3 end / end\"\n- self.create_fdir_rule(rule1, check_stats=False, msg=\"Invalid input action number: Invalid argument\")\n+ self.create_fdir_rule(rule1, check_stats=False, msg=\"error\")\n self.check_fdir_rule(stats=False)\n \n def test_void_action(self):\n", "prefixes": [ "V1" ] }