From patchwork Thu Mar 21 23:15:16 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dumitru Ceara X-Patchwork-Id: 1914662 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.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=E7KQDUBJ; dkim-atps=neutral Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org (client-ip=140.211.166.136; helo=smtp3.osuosl.org; envelope-from=ovs-dev-bounces@openvswitch.org; receiver=patchwork.ozlabs.org) Received: from smtp3.osuosl.org (smtp3.osuosl.org [140.211.166.136]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4V11YB5RQGz1yXt for ; Fri, 22 Mar 2024 10:16:14 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id CC8A360FBD; Thu, 21 Mar 2024 23:16:11 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 3-b6N34yz91P; Thu, 21 Mar 2024 23:16:09 +0000 (UTC) X-Comment: SPF check N/A for local connections - client-ip=140.211.9.56; helo=lists.linuxfoundation.org; envelope-from=ovs-dev-bounces@openvswitch.org; receiver= DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org 1F7E360F3C Authentication-Results: smtp3.osuosl.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=E7KQDUBJ Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by smtp3.osuosl.org (Postfix) with ESMTPS id 1F7E360F3C; Thu, 21 Mar 2024 23:16:04 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id C25E1C0DD4; Thu, 21 Mar 2024 23:16:03 +0000 (UTC) X-Original-To: ovs-dev@openvswitch.org Delivered-To: ovs-dev@lists.linuxfoundation.org Received: from smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138]) by lists.linuxfoundation.org (Postfix) with ESMTP id 12DA1C007C for ; Thu, 21 Mar 2024 23:16:01 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id B0877822FA for ; Thu, 21 Mar 2024 23:15:59 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id h-dIdX1iZzqI for ; Thu, 21 Mar 2024 23:15:58 +0000 (UTC) Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=170.10.129.124; helo=us-smtp-delivery-124.mimecast.com; envelope-from=dceara@redhat.com; receiver= DMARC-Filter: OpenDMARC Filter v1.4.2 smtp1.osuosl.org DD486822FF Authentication-Results: smtp1.osuosl.org; dmarc=pass (p=none dis=none) header.from=redhat.com DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org DD486822FF Authentication-Results: smtp1.osuosl.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=E7KQDUBJ Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by smtp1.osuosl.org (Postfix) with ESMTPS id DD486822FF for ; Thu, 21 Mar 2024 23:15:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1711062956; 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: in-reply-to:in-reply-to:references:references; bh=4/C/1VB6i8p56d86TV/TBX0+cHaEWqHIlgq6L2lKh70=; b=E7KQDUBJcDyHvW+JwCchllqEUnDTyermITwl0mbYl10w/uy2BaBYDRmpjCZWgSMPehD08t RSrNlMiFNR95esCVzJYSg0gc+pA7MZIVK8/co4TBn46sMz69/9WtyDPMEoGmYnjqpBEOaG TRQPD696mZyA/Y7RVElPJJJkQT2m6TE= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-413-dXdms3vTMCW_WUA3ryTCzQ-1; Thu, 21 Mar 2024 19:15:54 -0400 X-MC-Unique: dXdms3vTMCW_WUA3ryTCzQ-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.rdu2.redhat.com [10.11.54.8]) (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 mimecast-mx02.redhat.com (Postfix) with ESMTPS id 88BA18007A1; Thu, 21 Mar 2024 23:15:54 +0000 (UTC) Received: from cecil-rh.redhat.com (unknown [10.39.194.213]) by smtp.corp.redhat.com (Postfix) with ESMTP id 18E03C15773; Thu, 21 Mar 2024 23:15:53 +0000 (UTC) From: Dumitru Ceara To: ovs-dev@openvswitch.org Date: Fri, 22 Mar 2024 00:15:16 +0100 Message-ID: <20240321231535.695233-9-dceara@redhat.com> In-Reply-To: <20240321231535.695233-1-dceara@redhat.com> References: <20240321231535.695233-1-dceara@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.8 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Cc: i.maximets@ovn.org Subject: [ovs-dev] [PATCH ovn v3 8/8] Update post OVS-OVN split TODO list. 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" As Ilya commented in the review of the V1 of this series [0] the statement that "all the TODO items listed in it have been addressed" is indeed "a bold statement". However quite a few don't seem applicable anymore. [0] https://mail.openvswitch.org/pipermail/ovs-dev/2024-March/412704.html Signed-off-by: Dumitru Ceara --- TODO_SPLIT.rst | 73 -------------------------------------------------- 1 file changed, 73 deletions(-) diff --git a/TODO_SPLIT.rst b/TODO_SPLIT.rst index 9c65f77d87..cdc2d25ba7 100644 --- a/TODO_SPLIT.rst +++ b/TODO_SPLIT.rst @@ -25,9 +25,6 @@ OVN/OVS Split To-do List ======================== -Immediate tasks ---------------------------- - * There are likely many unused files throughout the source tree since they pertain to OVS rather than OVN. These can also be removed from the repo. @@ -36,73 +33,3 @@ Immediate tasks polish (or even just rewrite it. I won't be offended). * Cleanup the acinclude.m4 and m4 folder - -Immediate to Short-term tasks ------------------------------ - -* The Documentation/ directory can use an overhaul. Non-OVN content can be - removed. The installation guide and tutorials should be reworked to be - geared towards OVN specifically instead of OVS. - -* The tests/ directory contains copies of all "utility" files from the ovs - repo. These files could be removed in favor of using the files directly - from the ovs repo instead of the copy. As an example, ofproto-macros.at could - be removed from the tests/ directory, and we could reference the version in - the ovs repo instead. There are many other files that this could be done with. - -* OVN code needs to be removed from the OVS repo. This should be mostly - straightforward with a couple of exceptions. There is an include/ovn/ - directory in the ovs repo that should be moved to the ovn repo instead of - being removed. The other challenge is updating the ovsdb clustering tests. - They currently make use of the OVN database schemas. They will need to be - updated not to rely on something from OVN. - -* The rhel/ and debian/ directories need updating and testing so that they can - build ovn. They also need to be modified so that they no longer can build ovs - packages. - -Short to Medium-term tasks --------------------------- - -* There are several non-code subdirectories in the ovn repo that have not - been updated from how they existed in the OVS repo. It should be evaluated - if there is potential use to modify these to be useful for OVN or if they - should just be removed. Examples include the poc/ and xenserver/ directories. - -Medium to Long-term tasks -------------------------- - -* The goal is to eventually not require having ovs as a subtree in the ovn - repo. Using variables in Makefiles goes a short way towards realizing - this goal, but it only partially gets us there. We still - refer to the ovs subtree directly in certain areas, most notably the - tests/ directory. Getting rid of the ovs subtree can be thought of as a - two-step process: - - 1) Remove all direct references to the ovs subtree from the build system. By - doing this, it will be possible to have an ovs source repo checked out at - some other location and have ovn refer to that by using variables. - - 2) Alter ovs's build so that it places headers, utilities, etc. in known - locations on disk when it installs. This way, rather than referring to - an ovs source repo, we can refer to installation directories in the ovn - build system. This way, it could be possible to build ovn just by installing - the development package of ovs as a prerequisite. - -* A plan needs to be developed for compatibility between OVN and OVS. There - are several facets to this - - 1) We need to try to determine a policy with regards to what OVS versions - OVN will be compatible with. - - 2) The ovs subtree in ovn currently is the master branch of ovs at the time - that the ovn repo was split off. It likely will result in a more stable - environment to use a released version of ovs as our basis instead of an - arbitrary commit from mid-release. - - 3) While ovn was housed in the ovs repo, it was sometimes necessary to - update ovs or ovsdb code in order to facilitate a new ovn feature. Or it - might be necessary to fix a bug in ovs in order to fix a bug in ovn. With ovn - split away, there needs to be a way that ovn developers can contribute changes - to ovs when necessary but also not have to wait for those changes to be - available in an ovs release to be able to use them in ovn.