Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/129573/?format=api
http://patchwork.dpdk.org/api/patches/129573/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20230717110709.39220-3-juraj.linkes@pantheon.tech/", "project": { "id": 1, "url": "http://patchwork.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20230717110709.39220-3-juraj.linkes@pantheon.tech>", "list_archive_url": "https://inbox.dpdk.org/dev/20230717110709.39220-3-juraj.linkes@pantheon.tech", "date": "2023-07-17T11:07:05", "name": "[v2,2/6] dts: add traffic generator config", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "71726924431501091615895c01e2c4b4957cc0f3", "submitter": { "id": 1626, "url": "http://patchwork.dpdk.org/api/people/1626/?format=api", "name": "Juraj Linkeš", "email": "juraj.linkes@pantheon.tech" }, "delegate": { "id": 1, "url": "http://patchwork.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20230717110709.39220-3-juraj.linkes@pantheon.tech/mbox/", "series": [ { "id": 28953, "url": "http://patchwork.dpdk.org/api/series/28953/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=28953", "date": "2023-07-17T11:07:03", "name": "dts: tg abstractions and scapy tg", "version": 2, "mbox": "http://patchwork.dpdk.org/series/28953/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/129573/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/129573/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-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 16AF242E9B;\n\tMon, 17 Jul 2023 13:07:28 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 672F742D30;\n\tMon, 17 Jul 2023 13:07:18 +0200 (CEST)", "from mail-ej1-f42.google.com (mail-ej1-f42.google.com\n [209.85.218.42]) by mails.dpdk.org (Postfix) with ESMTP id BC12A42D2F\n for <dev@dpdk.org>; Mon, 17 Jul 2023 13:07:14 +0200 (CEST)", "by mail-ej1-f42.google.com with SMTP id\n a640c23a62f3a-98e39784a85so1171736566b.1\n for <dev@dpdk.org>; Mon, 17 Jul 2023 04:07:14 -0700 (PDT)", "from jlinkes-PT-Latitude-5530.. (ip-46.34.239.87.o2inet.sk.\n [46.34.239.87]) by smtp.gmail.com with ESMTPSA id\n s21-20020a170906355500b0098de7d28c34sm8995051eja.193.2023.07.17.04.07.13\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Mon, 17 Jul 2023 04:07:14 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=pantheon.tech; s=google; t=1689592034; x=1692184034;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=LYQ5fSud0HdHTdvHxZGBV6h/Mog2ynXXqUbRmQO64Po=;\n b=mn1t7z86xHswi16TL0zcZKda9i9tF0XE4p06QMpspcwyWFIU+GddKQYOKTKEXaLkMi\n Rk3CRuTFAUE+FZZluvWORkCWLnvfEpGuWcpBO9x150hZFmg7PIE8KLK35Cd612LDj4Qa\n A/95toP68NOjc8dgQUG8niy5fs5dbzg5tBKDMjluVjalBHXcPpMOBpGVfEmMOesyL/VF\n mjNnlk/rwlUb//5+jmhfUoW6rdOWKMGb8q06JIf/KE0tfPhnVzeWn6noExDetCFJPri7\n 1Sx6hYSftg5I7g9AMnXPv2QkkymX4FgLX7YgGWmzQlO4wFbU3i1Pjhs6Q3MuponjuOKH\n n9Iw==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20221208; t=1689592034; x=1692184034;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=LYQ5fSud0HdHTdvHxZGBV6h/Mog2ynXXqUbRmQO64Po=;\n b=WYM20JnH9E8567cjOt9RsxgHl7DkgJJUprTT8/iUkh9hno+O1R1fAnpMvBRtLixaGA\n 29xCXRRdLkP3ogOh8wH/eJqRQJ1OvogSFvRwzwizb/PzQdLVBGs37LM3zXyoixvUg7SO\n 1swCfYOcg2WZPwoKp7AQ0RFH77ERIJ/tApsKuBU99DheT/EcPLUyBL6sUyYHHLLWT2Lz\n N/NIG60MeeTB3jxsVjGFwHkAFUMRwk7flCOjZo3CR5oMwjK6S/cknPfxeWLw81XN2gOa\n Lnq7OL9bPnehfa+v5rzpC/KHW0+iOg8LRrnG2vwbTYfl1E2LnuDORSBXMPUh6EyCjCn9\n mULw==", "X-Gm-Message-State": "ABy/qLZPLYs4rvipCgVqvhadmHct8GByK2K+A7h6FfFhW2266/Mcjy2D\n 12kwVR0AuDgPbj4KHBgnPoQqEA==", "X-Google-Smtp-Source": "\n APBJJlHF4aEqhuPRglvKSOxZcBvSLul/4jJX6ny/hSrtOoTOsTKK8JbEjaOODv3TWp/4qsjj2CYSdg==", "X-Received": "by 2002:a17:907:9809:b0:98e:1729:aeee with SMTP id\n ji9-20020a170907980900b0098e1729aeeemr10746829ejc.2.1689592034351;\n Mon, 17 Jul 2023 04:07:14 -0700 (PDT)", "From": "=?utf-8?q?Juraj_Linke=C5=A1?= <juraj.linkes@pantheon.tech>", "To": "thomas@monjalon.net, Honnappa.Nagarahalli@arm.com, lijuan.tu@intel.com,\n jspewock@iol.unh.edu, probb@iol.unh.edu", "Cc": "dev@dpdk.org, =?utf-8?q?Juraj_Linke=C5=A1?= <juraj.linkes@pantheon.tech>", "Subject": "[PATCH v2 2/6] dts: add traffic generator config", "Date": "Mon, 17 Jul 2023 13:07:05 +0200", "Message-Id": "<20230717110709.39220-3-juraj.linkes@pantheon.tech>", "X-Mailer": "git-send-email 2.34.1", "In-Reply-To": "<20230717110709.39220-1-juraj.linkes@pantheon.tech>", "References": "<20230420093109.594704-1-juraj.linkes@pantheon.tech>\n <20230717110709.39220-1-juraj.linkes@pantheon.tech>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "Node configuration - where to connect, what ports to use and what TG to\nuse.\n\nSigned-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>\n---\n dts/conf.yaml | 26 ++++++-\n dts/framework/config/__init__.py | 87 +++++++++++++++++++---\n dts/framework/config/conf_yaml_schema.json | 29 +++++++-\n dts/framework/dts.py | 12 +--\n dts/framework/testbed_model/node.py | 4 +-\n dts/framework/testbed_model/sut_node.py | 6 +-\n 6 files changed, 141 insertions(+), 23 deletions(-)", "diff": "diff --git a/dts/conf.yaml b/dts/conf.yaml\nindex 3a5d87cb49..7f089022ba 100644\n--- a/dts/conf.yaml\n+++ b/dts/conf.yaml\n@@ -13,10 +13,11 @@ executions:\n skip_smoke_tests: false # optional flag that allow you to skip smoke tests\n test_suites:\n - hello_world\n- system_under_test:\n+ system_under_test_node:\n node_name: \"SUT 1\"\n vdevs: # optional; if removed, vdevs won't be used in the execution\n - \"crypto_openssl\"\n+ traffic_generator_node: \"TG 1\"\n nodes:\n - name: \"SUT 1\"\n hostname: sut1.change.me.localhost\n@@ -40,3 +41,26 @@ nodes:\n os_driver: i40e\n peer_node: \"TG 1\"\n peer_pci: \"0000:00:08.1\"\n+ - name: \"TG 1\"\n+ hostname: tg1.change.me.localhost\n+ user: dtsuser\n+ arch: x86_64\n+ os: linux\n+ lcores: \"\"\n+ ports:\n+ - pci: \"0000:00:08.0\"\n+ os_driver_for_dpdk: rdma\n+ os_driver: rdma\n+ peer_node: \"SUT 1\"\n+ peer_pci: \"0000:00:08.0\"\n+ - pci: \"0000:00:08.1\"\n+ os_driver_for_dpdk: rdma\n+ os_driver: rdma\n+ peer_node: \"SUT 1\"\n+ peer_pci: \"0000:00:08.1\"\n+ use_first_core: false\n+ hugepages: # optional; if removed, will use system hugepage configuration\n+ amount: 256\n+ force_first_numa: false\n+ traffic_generator:\n+ type: SCAPY\ndiff --git a/dts/framework/config/__init__.py b/dts/framework/config/__init__.py\nindex fad56cc520..72aa021b97 100644\n--- a/dts/framework/config/__init__.py\n+++ b/dts/framework/config/__init__.py\n@@ -13,7 +13,7 @@\n from dataclasses import dataclass\n from enum import Enum, auto, unique\n from pathlib import PurePath\n-from typing import Any, TypedDict\n+from typing import Any, TypedDict, Union\n \n import warlock # type: ignore\n import yaml\n@@ -55,6 +55,11 @@ class Compiler(StrEnum):\n msvc = auto()\n \n \n+@unique\n+class TrafficGeneratorType(StrEnum):\n+ SCAPY = auto()\n+\n+\n # Slots enables some optimizations, by pre-allocating space for the defined\n # attributes in the underlying data structure.\n #\n@@ -79,6 +84,29 @@ class PortConfig:\n def from_dict(node: str, d: dict) -> \"PortConfig\":\n return PortConfig(node=node, **d)\n \n+ def __str__(self) -> str:\n+ return f\"Port {self.pci} on node {self.node}\"\n+\n+\n+@dataclass(slots=True, frozen=True)\n+class TrafficGeneratorConfig:\n+ traffic_generator_type: TrafficGeneratorType\n+\n+ @staticmethod\n+ def from_dict(d: dict):\n+ # This looks useless now, but is designed to allow expansion to traffic\n+ # generators that require more configuration later.\n+ match TrafficGeneratorType(d[\"type\"]):\n+ case TrafficGeneratorType.SCAPY:\n+ return ScapyTrafficGeneratorConfig(\n+ traffic_generator_type=TrafficGeneratorType.SCAPY\n+ )\n+\n+\n+@dataclass(slots=True, frozen=True)\n+class ScapyTrafficGeneratorConfig(TrafficGeneratorConfig):\n+ pass\n+\n \n @dataclass(slots=True, frozen=True)\n class NodeConfiguration:\n@@ -90,17 +118,17 @@ class NodeConfiguration:\n os: OS\n lcores: str\n use_first_core: bool\n- memory_channels: int\n hugepages: HugepageConfiguration | None\n ports: list[PortConfig]\n \n @staticmethod\n- def from_dict(d: dict) -> \"NodeConfiguration\":\n+ def from_dict(d: dict) -> Union[\"SutNodeConfiguration\", \"TGNodeConfiguration\"]:\n hugepage_config = d.get(\"hugepages\")\n if hugepage_config:\n if \"force_first_numa\" not in hugepage_config:\n hugepage_config[\"force_first_numa\"] = False\n hugepage_config = HugepageConfiguration(**hugepage_config)\n+\n common_config = {\n \"name\": d[\"name\"],\n \"hostname\": d[\"hostname\"],\n@@ -110,12 +138,31 @@ def from_dict(d: dict) -> \"NodeConfiguration\":\n \"os\": OS(d[\"os\"]),\n \"lcores\": d.get(\"lcores\", \"1\"),\n \"use_first_core\": d.get(\"use_first_core\", False),\n- \"memory_channels\": d.get(\"memory_channels\", 1),\n \"hugepages\": hugepage_config,\n \"ports\": [PortConfig.from_dict(d[\"name\"], port) for port in d[\"ports\"]],\n }\n \n- return NodeConfiguration(**common_config)\n+ if \"traffic_generator\" in d:\n+ return TGNodeConfiguration(\n+ traffic_generator=TrafficGeneratorConfig.from_dict(\n+ d[\"traffic_generator\"]\n+ ),\n+ **common_config,\n+ )\n+ else:\n+ return SutNodeConfiguration(\n+ memory_channels=d.get(\"memory_channels\", 1), **common_config\n+ )\n+\n+\n+@dataclass(slots=True, frozen=True)\n+class SutNodeConfiguration(NodeConfiguration):\n+ memory_channels: int\n+\n+\n+@dataclass(slots=True, frozen=True)\n+class TGNodeConfiguration(NodeConfiguration):\n+ traffic_generator: ScapyTrafficGeneratorConfig\n \n \n @dataclass(slots=True, frozen=True)\n@@ -194,23 +241,40 @@ class ExecutionConfiguration:\n perf: bool\n func: bool\n test_suites: list[TestSuiteConfig]\n- system_under_test: NodeConfiguration\n+ system_under_test_node: SutNodeConfiguration\n+ traffic_generator_node: TGNodeConfiguration\n vdevs: list[str]\n skip_smoke_tests: bool\n \n @staticmethod\n- def from_dict(d: dict, node_map: dict) -> \"ExecutionConfiguration\":\n+ def from_dict(\n+ d: dict, node_map: dict[str, Union[SutNodeConfiguration | TGNodeConfiguration]]\n+ ) -> \"ExecutionConfiguration\":\n build_targets: list[BuildTargetConfiguration] = list(\n map(BuildTargetConfiguration.from_dict, d[\"build_targets\"])\n )\n test_suites: list[TestSuiteConfig] = list(\n map(TestSuiteConfig.from_dict, d[\"test_suites\"])\n )\n- sut_name = d[\"system_under_test\"][\"node_name\"]\n+ sut_name = d[\"system_under_test_node\"][\"node_name\"]\n skip_smoke_tests = d.get(\"skip_smoke_tests\", False)\n assert sut_name in node_map, f\"Unknown SUT {sut_name} in execution {d}\"\n+ system_under_test_node = node_map[sut_name]\n+ assert isinstance(\n+ system_under_test_node, SutNodeConfiguration\n+ ), f\"Invalid SUT configuration {system_under_test_node}\"\n+\n+ tg_name = d[\"traffic_generator_node\"]\n+ assert tg_name in node_map, f\"Unknown TG {tg_name} in execution {d}\"\n+ traffic_generator_node = node_map[tg_name]\n+ assert isinstance(\n+ traffic_generator_node, TGNodeConfiguration\n+ ), f\"Invalid TG configuration {traffic_generator_node}\"\n+\n vdevs = (\n- d[\"system_under_test\"][\"vdevs\"] if \"vdevs\" in d[\"system_under_test\"] else []\n+ d[\"system_under_test_node\"][\"vdevs\"]\n+ if \"vdevs\" in d[\"system_under_test_node\"]\n+ else []\n )\n return ExecutionConfiguration(\n build_targets=build_targets,\n@@ -218,7 +282,8 @@ def from_dict(d: dict, node_map: dict) -> \"ExecutionConfiguration\":\n func=d[\"func\"],\n skip_smoke_tests=skip_smoke_tests,\n test_suites=test_suites,\n- system_under_test=node_map[sut_name],\n+ system_under_test_node=system_under_test_node,\n+ traffic_generator_node=traffic_generator_node,\n vdevs=vdevs,\n )\n \n@@ -229,7 +294,7 @@ class Configuration:\n \n @staticmethod\n def from_dict(d: dict) -> \"Configuration\":\n- nodes: list[NodeConfiguration] = list(\n+ nodes: list[Union[SutNodeConfiguration | TGNodeConfiguration]] = list(\n map(NodeConfiguration.from_dict, d[\"nodes\"])\n )\n assert len(nodes) > 0, \"There must be a node to test\"\ndiff --git a/dts/framework/config/conf_yaml_schema.json b/dts/framework/config/conf_yaml_schema.json\nindex 61f52b4365..76df84840a 100644\n--- a/dts/framework/config/conf_yaml_schema.json\n+++ b/dts/framework/config/conf_yaml_schema.json\n@@ -164,6 +164,11 @@\n \"amount\"\n ]\n },\n+ \"mac_address\": {\n+ \"type\": \"string\",\n+ \"description\": \"A MAC address\",\n+ \"pattern\": \"^([0-9A-Fa-f]{2}[:-]){5}([0-9A-Fa-f]{2})$\"\n+ },\n \"pci_address\": {\n \"type\": \"string\",\n \"pattern\": \"^[\\\\da-fA-F]{4}:[\\\\da-fA-F]{2}:[\\\\da-fA-F]{2}.\\\\d:?\\\\w*$\"\n@@ -286,6 +291,22 @@\n ]\n },\n \"minimum\": 1\n+ },\n+ \"traffic_generator\": {\n+ \"oneOf\": [\n+ {\n+ \"type\": \"object\",\n+ \"description\": \"Scapy traffic generator. Used for functional testing.\",\n+ \"properties\": {\n+ \"type\": {\n+ \"type\": \"string\",\n+ \"enum\": [\n+ \"SCAPY\"\n+ ]\n+ }\n+ }\n+ }\n+ ]\n }\n },\n \"additionalProperties\": false,\n@@ -336,7 +357,7 @@\n \"description\": \"Optional field that allows you to skip smoke testing\",\n \"type\": \"boolean\"\n },\n- \"system_under_test\": {\n+ \"system_under_test_node\": {\n \"type\":\"object\",\n \"properties\": {\n \"node_name\": {\n@@ -353,6 +374,9 @@\n \"required\": [\n \"node_name\"\n ]\n+ },\n+ \"traffic_generator_node\": {\n+ \"$ref\": \"#/definitions/node_name\"\n }\n },\n \"additionalProperties\": false,\n@@ -361,7 +385,8 @@\n \"perf\",\n \"func\",\n \"test_suites\",\n- \"system_under_test\"\n+ \"system_under_test_node\",\n+ \"traffic_generator_node\"\n ]\n },\n \"minimum\": 1\ndiff --git a/dts/framework/dts.py b/dts/framework/dts.py\nindex 7b09d8fba8..372bc72787 100644\n--- a/dts/framework/dts.py\n+++ b/dts/framework/dts.py\n@@ -38,17 +38,17 @@ def run_all() -> None:\n # for all Execution sections\n for execution in CONFIGURATION.executions:\n sut_node = None\n- if execution.system_under_test.name in nodes:\n+ if execution.system_under_test_node.name in nodes:\n # a Node with the same name already exists\n- sut_node = nodes[execution.system_under_test.name]\n+ sut_node = nodes[execution.system_under_test_node.name]\n else:\n # the SUT has not been initialized yet\n try:\n- sut_node = SutNode(execution.system_under_test)\n+ sut_node = SutNode(execution.system_under_test_node)\n result.update_setup(Result.PASS)\n except Exception as e:\n dts_logger.exception(\n- f\"Connection to node {execution.system_under_test} failed.\"\n+ f\"Connection to node {execution.system_under_test_node} failed.\"\n )\n result.update_setup(Result.FAIL, e)\n else:\n@@ -87,7 +87,9 @@ def _run_execution(\n Run the given execution. This involves running the execution setup as well as\n running all build targets in the given execution.\n \"\"\"\n- dts_logger.info(f\"Running execution with SUT '{execution.system_under_test.name}'.\")\n+ dts_logger.info(\n+ f\"Running execution with SUT '{execution.system_under_test_node.name}'.\"\n+ )\n execution_result = result.add_execution(sut_node.config, sut_node.node_info)\n \n try:\ndiff --git a/dts/framework/testbed_model/node.py b/dts/framework/testbed_model/node.py\nindex c5147e0ee6..d2d55d904e 100644\n--- a/dts/framework/testbed_model/node.py\n+++ b/dts/framework/testbed_model/node.py\n@@ -48,6 +48,8 @@ def __init__(self, node_config: NodeConfiguration):\n self._logger = getLogger(self.name)\n self.main_session = create_session(self.config, self.name, self._logger)\n \n+ self._logger.info(f\"Connected to node: {self.name}\")\n+\n self._get_remote_cpus()\n # filter the node lcores according to user config\n self.lcores = LogicalCoreListFilter(\n@@ -56,8 +58,6 @@ def __init__(self, node_config: NodeConfiguration):\n \n self._other_sessions = []\n \n- self._logger.info(f\"Created node: {self.name}\")\n-\n def set_up_execution(self, execution_config: ExecutionConfiguration) -> None:\n \"\"\"\n Perform the execution setup that will be done for each execution\ndiff --git a/dts/framework/testbed_model/sut_node.py b/dts/framework/testbed_model/sut_node.py\nindex 53953718a1..bcad364435 100644\n--- a/dts/framework/testbed_model/sut_node.py\n+++ b/dts/framework/testbed_model/sut_node.py\n@@ -13,8 +13,8 @@\n BuildTargetConfiguration,\n BuildTargetInfo,\n InteractiveApp,\n- NodeConfiguration,\n NodeInfo,\n+ SutNodeConfiguration,\n )\n from framework.remote_session import (\n CommandResult,\n@@ -83,6 +83,7 @@ class SutNode(Node):\n Another key capability is building DPDK according to given build target.\n \"\"\"\n \n+ config: SutNodeConfiguration\n _dpdk_prefix_list: list[str]\n _dpdk_timestamp: str\n _build_target_config: BuildTargetConfiguration | None\n@@ -95,7 +96,7 @@ class SutNode(Node):\n _node_info: NodeInfo | None\n _compiler_version: str | None\n \n- def __init__(self, node_config: NodeConfiguration):\n+ def __init__(self, node_config: SutNodeConfiguration):\n super(SutNode, self).__init__(node_config)\n self._dpdk_prefix_list = []\n self._build_target_config = None\n@@ -110,6 +111,7 @@ def __init__(self, node_config: NodeConfiguration):\n self._dpdk_version = None\n self._node_info = None\n self._compiler_version = None\n+ self._logger.info(f\"Created node: {self.name}\")\n \n @property\n def _remote_dpdk_dir(self) -> PurePath:\n", "prefixes": [ "v2", "2/6" ] }{ "id": 129573, "url": "