From patchwork Tue May 31 09:31:36 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mohammad Heib X-Patchwork-Id: 1637272 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: bilbo.ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=cNiQS4NP; dkim-atps=neutral Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org (client-ip=140.211.166.133; helo=smtp2.osuosl.org; envelope-from=ovs-dev-bounces@openvswitch.org; receiver=) Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by bilbo.ozlabs.org (Postfix) with ESMTPS id 4LC6Vb1YS6z9s1l for ; Tue, 31 May 2022 19:31:50 +1000 (AEST) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 2171740C92; Tue, 31 May 2022 09:31:49 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id myiQonmmBZ7F; Tue, 31 May 2022 09:31:48 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by smtp2.osuosl.org (Postfix) with ESMTPS id 4C23840483; Tue, 31 May 2022 09:31:47 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id 1305BC0032; Tue, 31 May 2022 09:31:47 +0000 (UTC) X-Original-To: dev@openvswitch.org Delivered-To: ovs-dev@lists.linuxfoundation.org Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) by lists.linuxfoundation.org (Postfix) with ESMTP id 47F7AC002D for ; Tue, 31 May 2022 09:31:45 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 26D6B40483 for ; Tue, 31 May 2022 09:31:45 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id o4NOsJp2jzt8 for ; Tue, 31 May 2022 09:31:44 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.8.0 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by smtp2.osuosl.org (Postfix) with ESMTPS id 563E64024B for ; Tue, 31 May 2022 09:31:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1653989503; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=ZdEKVWDsV1PBFOJ6sl8c8TX0LI4TiBbfM6Vi66LQpJw=; b=cNiQS4NPDD0eLZvnYNRMn9kO7BoyPFjHWwfeohpyV44/zfEcZq/9R4itOaVTR+nuyL11ly AtBV61IGKDY2QXFzyLjMQwgtsbn6bQcO7vh/3c9XxCJV+LR0xAex/XEQfDd5C/tt5h2H4K K3hqF2LFfEBYSAEaz3LvkmS5IsMp1ZA= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-548-nsqiabVJNk24CaUeUBQuGQ-1; Tue, 31 May 2022 05:31:41 -0400 X-MC-Unique: nsqiabVJNk24CaUeUBQuGQ-1 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 9DC938027EE for ; Tue, 31 May 2022 09:31:41 +0000 (UTC) Received: from mheiblap.localdomain.com (unknown [10.35.206.236]) by smtp.corp.redhat.com (Postfix) with ESMTP id C78C2492C3B; Tue, 31 May 2022 09:31:40 +0000 (UTC) From: Mohammad Heib To: dev@openvswitch.org Date: Tue, 31 May 2022 12:31:36 +0300 Message-Id: <20220531093136.2132449-1-mheib@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.85 on 10.11.54.9 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mheib@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Subject: [ovs-dev] [PATCH ovn] OVN-CI: Add test cases with monitor-all enabled. X-BeenThere: ovs-dev@openvswitch.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ovs-dev-bounces@openvswitch.org Sender: "dev" currently ovn ci only has one test case with the option ovn-monitor-all enabled, this patch will add one more execution with option ovn-monitor-all=true for each test case that are wrapped by OVN_FOR_EACH_NORTHD macro. This will more or less double the number of test cases. It is possible to select a reduce set of test cases using -k "keywords". Keyword such as dp-groups=yes dp-groups=no parallelization=yes parallelization=no ovn-northd ovn-northd-ddlog ovn_monitor_all=yes can be used to select a range of tests, as title is searched as well. For instance, to run ovn-parallelization tests, with dp-groups enabled and ddlog disabled: make check TESTSUITEFLAGS="-k dp-groups=yes,ovn_monitor_all=yes,\!ovn-northd-ddlog" Signed-off-by: Mohammad Heib --- tests/ovn-macros.at | 19 ++++++++++++++++--- tests/ovs-macros.at | 4 +++- 2 files changed, 19 insertions(+), 4 deletions(-) diff --git a/tests/ovn-macros.at b/tests/ovn-macros.at index c6f0f6251..7484b32c3 100644 --- a/tests/ovn-macros.at +++ b/tests/ovn-macros.at @@ -323,6 +323,19 @@ ovn_az_attach() { -- --may-exist add-br br-int \ -- set bridge br-int fail-mode=secure other-config:disable-in-band=true \ || return 1 + + # currently this is the optimal place to add the ovn-monitor-all=true option, + # this can be implemented in a different way by redefining the sim-add function + # to add the ovn-related external-ids when we add a new simulated node via sim-add. + # + # wait one sec to make sure that the ovn notice and update it configuration + # according to the new option. + # + if test X$OVN_MONITOR_ALL = Xyes; then + ovs-vsctl set open . external_ids:ovn-monitor-all=true + sleep 1 + fi + start_daemon ovn-controller --enable-dummy-vif-plug || return 1 } @@ -751,19 +764,19 @@ m4_define([OVN_FOR_EACH_NORTHD], [m4_foreach([NORTHD_TYPE], [ovn-northd, ovn-northd-ddlog], [m4_foreach([NORTHD_USE_DP_GROUPS], [yes, no], [m4_foreach([NORTHD_USE_PARALLELIZATION], [yes, no], [$1 -])])])]) +]) m4_foreach([OVN_MONITOR_ALL], [yes], [$1])])])]) # Some tests aren't prepared for dp groups to be enabled. m4_define([OVN_FOR_EACH_NORTHD_WITHOUT_DP_GROUPS], [m4_foreach([NORTHD_TYPE], [ovn-northd, ovn-northd-ddlog], [m4_foreach([NORTHD_USE_DP_GROUPS], [no], [m4_foreach([NORTHD_USE_PARALLELIZATION], [yes, no], [$1 -])])])]) +])m4_foreach([OVN_MONITOR_ALL], [yes], [$1])])])]) # Some tests aren't prepared for ddlog to be enabled. m4_define([OVN_FOR_EACH_NORTHD_WITHOUT_DDLOG], [m4_foreach([NORTHD_TYPE], [ovn-northd], [m4_foreach([NORTHD_USE_DP_GROUPS], [yes, no], [m4_foreach([NORTHD_USE_PARALLELIZATION], [yes, no], [$1 -])])])]) +])m4_foreach([OVN_MONITOR_ALL], [yes], [$1])])])]) diff --git a/tests/ovs-macros.at b/tests/ovs-macros.at index 0482b7f5b..5d6c3ada2 100644 --- a/tests/ovs-macros.at +++ b/tests/ovs-macros.at @@ -10,7 +10,7 @@ dnl set it as a shell variable as well. dnl - Skip the test if it's for ovn-northd-ddlog but it didn't get built. m4_rename([AT_SETUP], [OVS_AT_SETUP]) m4_define([AT_SETUP], - [OVS_AT_SETUP($@[]m4_ifdef([NORTHD_TYPE], [ -- NORTHD_TYPE])[]m4_ifdef([NORTHD_USE_DP_GROUPS], [ -- dp-groups=NORTHD_USE_DP_GROUPS])[]m4_ifdef([NORTHD_USE_PARALLELIZATION], [ -- parallelization=NORTHD_USE_PARALLELIZATION])) + [OVS_AT_SETUP($@[]m4_ifdef([NORTHD_TYPE], [ -- NORTHD_TYPE])[]m4_ifdef([NORTHD_USE_DP_GROUPS], [ -- dp-groups=NORTHD_USE_DP_GROUPS])[]m4_ifdef([NORTHD_USE_PARALLELIZATION], [ -- parallelization=NORTHD_USE_PARALLELIZATION])[]m4_ifdef([OVN_MONITOR_ALL], [ -- ovn_monitor_all=OVN_MONITOR_ALL])) m4_ifdef([NORTHD_TYPE], [[NORTHD_TYPE]=NORTHD_TYPE ])dnl m4_ifdef([NORTHD_USE_DP_GROUPS], [[NORTHD_USE_DP_GROUPS]=NORTHD_USE_DP_GROUPS @@ -19,6 +19,8 @@ m4_ifdef([NORTHD_USE_PARALLELIZATION], [[NORTHD_USE_PARALLELIZATION]=NORTHD_USE_ ])dnl m4_ifdef([NORTHD_DUMMY_NUMA], [[NORTHD_DUMMY_NUMA]=NORTHD_DUMMY_NUMA ])dnl +m4_ifdef([OVN_MONITOR_ALL], [[OVN_MONITOR_ALL]=OVN_MONITOR_ALL +])dnl m4_if(NORTHD_TYPE, [ovn-northd-ddlog], [dnl AT_SKIP_IF([test $TEST_DDLOG = no]) ])dnl