{"id":2225638,"url":"http://patchwork.ozlabs.org/api/1.1/patches/2225638/?format=json","web_url":"http://patchwork.ozlabs.org/project/ovn/patch/20260421101422.3608058-4-moloings@redhat.com/","project":{"id":68,"url":"http://patchwork.ozlabs.org/api/1.1/projects/68/?format=json","name":"Open Virtual Network development","link_name":"ovn","list_id":"ovs-dev.openvswitch.org","list_email":"ovs-dev@openvswitch.org","web_url":"http://openvswitch.org/","scm_url":"","webscm_url":""},"msgid":"<20260421101422.3608058-4-moloings@redhat.com>","date":"2026-04-21T10:14:22","name":"[ovs-dev,3/3] ic: Add tests for transit switch ports.","commit_ref":null,"pull_url":null,"state":"changes-requested","archived":false,"hash":"23d37fb6ce08431101ae94ac56a2a7dc623263e2","submitter":{"id":91032,"url":"http://patchwork.ozlabs.org/api/1.1/people/91032/?format=json","name":"Mairtin O'Loingsigh","email":"moloings@redhat.com"},"delegate":{"id":94943,"url":"http://patchwork.ozlabs.org/api/1.1/users/94943/?format=json","username":"dceara","first_name":"Dumitru","last_name":"Ceara","email":"dceara@redhat.com"},"mbox":"http://patchwork.ozlabs.org/project/ovn/patch/20260421101422.3608058-4-moloings@redhat.com/mbox/","series":[{"id":500774,"url":"http://patchwork.ozlabs.org/api/1.1/series/500774/?format=json","web_url":"http://patchwork.ozlabs.org/project/ovn/list/?series=500774","date":"2026-04-21T10:14:20","name":"Add support for Transit switch ports","version":1,"mbox":"http://patchwork.ozlabs.org/series/500774/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2225638/comments/","check":"success","checks":"http://patchwork.ozlabs.org/api/patches/2225638/checks/","tags":{},"headers":{"Return-Path":"<ovs-dev-bounces@openvswitch.org>","X-Original-To":["incoming@patchwork.ozlabs.org","dev@openvswitch.org"],"Delivered-To":["patchwork-incoming@legolas.ozlabs.org","ovs-dev@lists.linuxfoundation.org"],"Authentication-Results":["legolas.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256\n header.s=mimecast20190719 header.b=HmR9hP+5;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org\n (client-ip=2605:bc80:3010::133; helo=smtp2.osuosl.org;\n envelope-from=ovs-dev-bounces@openvswitch.org; receiver=patchwork.ozlabs.org)","smtp2.osuosl.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key)\n header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256\n header.s=mimecast20190719 header.b=HmR9hP+5","smtp2.osuosl.org; dmarc=pass (p=quarantine dis=none)\n header.from=redhat.com"],"Received":["from smtp2.osuosl.org (smtp2.osuosl.org [IPv6:2605:bc80:3010::133])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g0JBz60lFz1yGt\n\tfor <incoming@patchwork.ozlabs.org>; Tue, 21 Apr 2026 20:15:23 +1000 (AEST)","from localhost (localhost [127.0.0.1])\n\tby smtp2.osuosl.org (Postfix) with ESMTP id 434D340AF0;\n\tTue, 21 Apr 2026 10:15:22 +0000 (UTC)","from smtp2.osuosl.org ([127.0.0.1])\n by localhost (smtp2.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP\n id yzugPb8JAmHe; Tue, 21 Apr 2026 10:15:18 +0000 (UTC)","from lists.linuxfoundation.org (lf-lists.osuosl.org\n [IPv6:2605:bc80:3010:104::8cd3:938])\n\tby smtp2.osuosl.org (Postfix) with ESMTPS id 265DB40AE1;\n\tTue, 21 Apr 2026 10:15:18 +0000 (UTC)","from lf-lists.osuosl.org (localhost [127.0.0.1])\n\tby lists.linuxfoundation.org (Postfix) with ESMTP id 0200CC058E;\n\tTue, 21 Apr 2026 10:15:18 +0000 (UTC)","from smtp2.osuosl.org (smtp2.osuosl.org [IPv6:2605:bc80:3010::133])\n by lists.linuxfoundation.org (Postfix) with ESMTP id 45330C058D\n for <dev@openvswitch.org>; Tue, 21 Apr 2026 10:15:16 +0000 (UTC)","from localhost (localhost [127.0.0.1])\n by smtp2.osuosl.org (Postfix) with ESMTP id 3197240AE0\n for <dev@openvswitch.org>; Tue, 21 Apr 2026 10:15:16 +0000 (UTC)","from smtp2.osuosl.org ([127.0.0.1])\n by localhost (smtp2.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP\n id fhC1mjmHlaDH for <dev@openvswitch.org>;\n Tue, 21 Apr 2026 10:15:14 +0000 (UTC)","from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [170.10.129.124])\n by smtp2.osuosl.org (Postfix) with ESMTPS id 47B7140AA9\n for <dev@openvswitch.org>; Tue, 21 Apr 2026 10:15:13 +0000 (UTC)","from mail-qk1-f198.google.com (mail-qk1-f198.google.com\n [209.85.222.198]) by relay.mimecast.com with ESMTP with STARTTLS\n (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n us-mta-527-dYRpuITPN1uQbMCH1HlU6Q-1; Tue, 21 Apr 2026 06:15:11 -0400","by mail-qk1-f198.google.com with SMTP id\n af79cd13be357-8eb04352924so352785085a.1\n for <dev@openvswitch.org>; Tue, 21 Apr 2026 03:15:11 -0700 (PDT)","from moloings-thinkpadp1gen7.rmtie.csb\n ([2001:bb6:2be4:f100:a13c:8a56:eb6f:ed3f])\n by smtp.gmail.com with ESMTPSA id\n af79cd13be357-8e7d92ce037sm997022285a.32.2026.04.21.03.15.08\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Tue, 21 Apr 2026 03:15:09 -0700 (PDT)"],"X-Virus-Scanned":["amavis at osuosl.org","amavis at osuosl.org"],"X-Comment":"SPF check N/A for local connections -\n client-ip=2605:bc80:3010:104::8cd3:938; helo=lists.linuxfoundation.org;\n envelope-from=ovs-dev-bounces@openvswitch.org; receiver=<UNKNOWN> ","DKIM-Filter":["OpenDKIM Filter v2.11.0 smtp2.osuosl.org 265DB40AE1","OpenDKIM Filter v2.11.0 smtp2.osuosl.org 47B7140AA9"],"Received-SPF":"Pass (mailfrom) identity=mailfrom; client-ip=170.10.129.124;\n helo=us-smtp-delivery-124.mimecast.com; envelope-from=moloings@redhat.com;\n receiver=<UNKNOWN>","DMARC-Filter":"OpenDMARC Filter v1.4.2 smtp2.osuosl.org 47B7140AA9","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1776766512;\n h=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n to:to:cc:cc:mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding:\n in-reply-to:in-reply-to:references:references;\n bh=VSpZFgMozrBWfoTKJ18KWNRHPBj3TrUTCrNHJjngUdk=;\n b=HmR9hP+50PehrIF9ESu+zJA8+TZsJy+1I8zNDIKO30OZKmQ5MzGqvc3CWkiqSN9+bIKWUX\n AU4FbW722niqO2L8DHeb5czzijAIAVMluOBQAkPCAzP5dSzERYHc+ekxYo3DaTTthAStok\n y57fqHN+bb1Lpc36WYwyqWWS5TuT7Mw=","X-MC-Unique":"dYRpuITPN1uQbMCH1HlU6Q-1","X-Mimecast-MFC-AGG-ID":"dYRpuITPN1uQbMCH1HlU6Q_1776766511","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1776766511; x=1777371311;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from\n :to:cc:subject:date:message-id:reply-to;\n bh=VSpZFgMozrBWfoTKJ18KWNRHPBj3TrUTCrNHJjngUdk=;\n b=BkKkprlWcoOKqjlKFYG1774vIyE6Bftoh0to9oduAY3dAyboHkZmhgPxSWblgoAYlD\n cN+aschQ4xatTiChtlZibjC9aQXTENlsINEv9QOkMPch5TQDIMopyWmNC8WJ8miBiZh+\n /nRV/tPMnMlxPLxC/11OsF0nGEheD3gu3qVSrZoO0tlU6izNr0HgeWgdCYJOWPtKG0A6\n m1qf1UjIPMDWSFl7wdaK9NZ6L7yeC0tSANheybbt/bCPlwBJbaruXyKy9YTzO9ZW6k0w\n JImGHaNSQiSaFj2IuiUlQtGUcQ/7SW17s3rNEE2zTQjC+esKZoSNkJR6fwFCH4mvyp1v\n lOsg==","X-Gm-Message-State":"AOJu0YyF5j4qwKnuB92o1pRgv5T4T//P9rUL6teXzApCKhuH2Nk3sQ7P\n UJtwx+cC6L5Uaj2ws4TEBswwmJFVSvCs18iyWqGdAWqK0V+sBspn7RzMBOrEl+pbCQnTrGQOT5C\n IuUS6nF3LdUjlgf5LSXayUOKCVOIqe5k2GEXylsc7qrDHRtGrFCskXEqbc+Eab4eAUXYMlixv1a\n y0M7vwqbdGZyFvuxyeTY+W712Zzftee6EBwSX01Q==","X-Gm-Gg":"AeBDietnTPKQPBYzfTirsZHLbd4Z2FYy7r37NMNhgbrUTdT3+R77rSk4MJIW6GZKxBE\n t2bk/CfM4VVEN90ogwsV1j3urvGS20Fhk7m+vO9b4GD9W3zYeLIEzz9h3zbFqwWb44COG3Vl1dx\n 7fJZe4IH16dF7vBpgc5MRw3OxmPiu4HRfQo/XQER1ZWH47v4iCzv84TLHFTPYtavqZUfFMzE4HS\n vSUvrELyRxZ8ajrBg8A+p/BwfTrRnb/ktxe0RZ4GhmvPKzhuMhovJvac4x82wBdK9dKSQxdxL2n\n CivQNrmPxNCBZi6GWxDqrhzYECtOPmIvVan3/sebD2Vli9w3evAyKH48udbc5cWu+80qVBwvd2z\n b7cHFQ6hD+NUnN0WAEBtcYrq5mrchLLcV967JzYKGXqgtsCQej+G8ekQuLk/neag0","X-Received":["by 2002:a05:620a:2844:b0:8d6:bd01:a67a with SMTP id\n af79cd13be357-8e78fe207d7mr2289249285a.26.1776766510529;\n Tue, 21 Apr 2026 03:15:10 -0700 (PDT)","by 2002:a05:620a:2844:b0:8d6:bd01:a67a with SMTP id\n af79cd13be357-8e78fe207d7mr2289238485a.26.1776766509721;\n Tue, 21 Apr 2026 03:15:09 -0700 (PDT)"],"To":"dev@openvswitch.org","Date":"Tue, 21 Apr 2026 11:14:22 +0100","Message-ID":"<20260421101422.3608058-4-moloings@redhat.com>","X-Mailer":"git-send-email 2.52.0","In-Reply-To":"<20260421101422.3608058-1-moloings@redhat.com>","References":"<20260421101422.3608058-1-moloings@redhat.com>","MIME-Version":"1.0","X-Mimecast-Spam-Score":"0","X-Mimecast-MFC-PROC-ID":"S1m59VU0bTt6gtnirDcSOGLhACKkQ3r_L--02rQQCO0_1776766511","X-Mimecast-Originator":"redhat.com","Subject":"[ovs-dev] [PATCH ovn 3/3] ic: Add tests for transit switch ports.","X-BeenThere":"ovs-dev@openvswitch.org","X-Mailman-Version":"2.1.30","Precedence":"list","List-Id":"<ovs-dev.openvswitch.org>","List-Unsubscribe":"<https://mail.openvswitch.org/mailman/options/ovs-dev>,\n <mailto:ovs-dev-request@openvswitch.org?subject=unsubscribe>","List-Archive":"<http://mail.openvswitch.org/pipermail/ovs-dev/>","List-Post":"<mailto:ovs-dev@openvswitch.org>","List-Help":"<mailto:ovs-dev-request@openvswitch.org?subject=help>","List-Subscribe":"<https://mail.openvswitch.org/mailman/listinfo/ovs-dev>,\n <mailto:ovs-dev-request@openvswitch.org?subject=subscribe>","From":"Mairtin O'Loingsigh via dev <ovs-dev@openvswitch.org>","Reply-To":"Mairtin O'Loingsigh <moloings@redhat.com>","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"ovs-dev-bounces@openvswitch.org","Sender":"\"dev\" <ovs-dev-bounces@openvswitch.org>"},"content":"Tests for ovn-ic-nbctl commands and simple transit switch\nconfigurations.\n\nReported-at: https://issues.redhat.com/browse/FDP-2878\nSigned-off-by: Mairtin O'Loingsigh <moloings@redhat.com>\n---\n tests/multinode.at    |  1 -\n tests/ovn-ic-nbctl.at | 26 ++++++++++++++++++++++\n tests/ovn-ic.at       | 50 +++++++++++++++++++++++++++++++++++++++++++\n 3 files changed, 76 insertions(+), 1 deletion(-)","diff":"diff --git a/tests/multinode.at b/tests/multinode.at\nindex d07660797..5f72bc237 100644\n--- a/tests/multinode.at\n+++ b/tests/multinode.at\n@@ -2453,7 +2453,6 @@ for i in 1 2; do\n     check m_as $chassis ovn-nbctl lsp-add-router-port ts ts-tr tr-ts\n \n     check m_as $chassis ovn-nbctl lrp-add tr tr-ts 00:00:00:00:10:00 10.100.200.1/24 10:200::1/64\n-    check m_as $chassis ovn-nbctl set logical_router tr options:requested-tnl-key=20\n     check m_as $chassis ovn-nbctl lrp-set-gateway-chassis tr-ts $chassis\n \n     # Add TS pods, with the same tunnel keys on both sides\ndiff --git a/tests/ovn-ic-nbctl.at b/tests/ovn-ic-nbctl.at\nindex 4c5269784..8309a009c 100644\n--- a/tests/ovn-ic-nbctl.at\n+++ b/tests/ovn-ic-nbctl.at\n@@ -61,6 +61,32 @@ AT_CHECK([ovn-ic-nbctl ts-del ts2], [1], [],\n \n AT_CHECK([ovn-ic-nbctl --if-exists ts-del ts2])\n \n+AT_CHECK([ovn-ic-nbctl --may-exist ts-add ts0])\n+AT_CHECK([ovn-ic-nbctl ts-list | uuidfilt], [0], [dnl\n+<0> (ts0)\n+])\n+\n+AT_CHECK([ovn-ic-nbctl tsp-add], [1], [],\n+  [ovn-ic-nbctl: 'tsp-add' command requires at least 2 arguments\n+])\n+\n+AT_CHECK([ovn-ic-nbctl tsp-add ts0], [1], [],\n+  [ovn-ic-nbctl: 'tsp-add' command requires at least 2 arguments\n+])\n+\n+AT_CHECK([ovn-ic-nbctl tsp-add ts0 ts0-p0 chassis=chassis])\n+\n+AT_CHECK([ovn-ic-nbctl --may-exist tsp-add ts0 ts0-p0 chassis=chassis])\n+AT_CHECK([ovn-ic-nbctl tsp-set-addr ts0-p0 \"00:11:22:11:22:33 192.168.10.10/24\"])\n+#sleep 6000\n+AT_CHECK([ovn-ic-nbctl tsp-set-addr ts0-p1 \"00:11:22:11:22:34 192.168.10.11/24\"], [1], [],\n+    [ovn-ic-nbctl: ts0-p1: switch port name not found\n+])\n+\n+AT_CHECK([ovn-ic-nbctl tsp-del], [1], [],\n+  [ovn-ic-nbctl: 'tsp-del' command requires at least 1 arguments\n+])\n+\n OVN_IC_NBCTL_TEST_STOP\n AT_CLEANUP\n \ndiff --git a/tests/ovn-ic.at b/tests/ovn-ic.at\nindex 0fa7c4f29..58dfa3f06 100644\n--- a/tests/ovn-ic.at\n+++ b/tests/ovn-ic.at\n@@ -164,6 +164,56 @@ AT_CHECK([ovn-ic-sbctl show | grep -A2 lsp1], [0], [dnl\n             address: [[\"00:00:00:00:00:01 10.0.0.1/24\"]]\n ])\n \n+# remove transit switch and check if port_binding is deleted\n+check ovn-ic-nbctl --wait=sb ts-del ts1\n+check_row_count ic-sb:Port_Binding 0 logical_port=lsp1\n+for i in 1 2; do\n+    az=az$i\n+    ovn_as $az\n+    OVN_CLEANUP_SBOX(gw-$az)\n+    OVN_CLEANUP_AZ([$az])\n+done\n+OVN_CLEANUP_IC\n+AT_CLEANUP\n+])\n+OVN_FOR_EACH_NORTHD([\n+AT_SETUP([ovn-ic -- transit port-bindings deletion upon TS deletion])\n+\n+ovn_init_ic_db\n+net_add n1\n+\n+# 1 GW per AZ\n+for i in 1 2; do\n+    az=az$i\n+    ovn_start $az\n+    sim_add gw-$az\n+    as gw-$az\n+    check ovs-vsctl add-br br-phys\n+    ovn_az_attach $az n1 br-phys 192.168.1.$i\n+    check ovs-vsctl set open . external-ids:ovn-is-interconn=true\n+done\n+\n+ovn_as az1\n+\n+# create transit switch and connect to LR\n+check ovn-ic-nbctl --wait=sb ts-add ts1\n+check ovn-nbctl lr-add lr1\n+check ovn-nbctl lrp-add lr1 lrp1 00:00:00:00:00:01 10.0.0.1/24\n+check ovn-nbctl lrp-set-gateway-chassis lrp1 gw-az1\n+\n+check ovn-ic-nbctl tsp-add ts1 lsp1 type=router chassis=gw-az1 option=lrp1\n+\n+wait_row_count Datapath_Binding 1 external_ids:interconn-ts=ts1\n+\n+# Sync ic-sb DB to see the TS changes.\n+check ovn-ic-nbctl --wait=sb sync\n+\n+AT_CHECK([ovn-ic-sbctl show | grep -A2 lsp1], [0], [dnl\n+        port lsp1\n+            transit switch: ts1\n+            address: [[\"00:00:00:00:00:01 10.0.0.1/24\"]]\n+])\n+\n # remove transit switch and check if port_binding is deleted\n check ovn-ic-nbctl --wait=sb ts-del ts1\n check_row_count ic-sb:Port_Binding 0 logical_port=lsp1\n","prefixes":["ovs-dev","3/3"]}