From patchwork Fri Dec 20 16:45:43 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Timothy Redaelli X-Patchwork-Id: 1214230 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org (client-ip=140.211.166.133; helo=hemlock.osuosl.org; envelope-from=ovs-dev-bounces@openvswitch.org; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=redhat.com header.i=@redhat.com header.b="PBQozxDI"; dkim-atps=neutral Received: from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 47fZQV38ngz9sPn for ; Sat, 21 Dec 2019 03:46:42 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by hemlock.osuosl.org (Postfix) with ESMTP id D4B6688870; Fri, 20 Dec 2019 16:46:40 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from hemlock.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id EiyhLxZoF0XO; Fri, 20 Dec 2019 16:46:33 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by hemlock.osuosl.org (Postfix) with ESMTP id B535788862; Fri, 20 Dec 2019 16:46:33 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id 9FE30C18DC; Fri, 20 Dec 2019 16:46:33 +0000 (UTC) X-Original-To: dev@openvswitch.org Delivered-To: ovs-dev@lists.linuxfoundation.org Received: from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138]) by lists.linuxfoundation.org (Postfix) with ESMTP id 3CC7BC077D for ; Fri, 20 Dec 2019 16:46:32 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id 2760E881BC for ; Fri, 20 Dec 2019 16:46:32 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id iUcWCMBpeTUx for ; Fri, 20 Dec 2019 16:46:12 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.7.6 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.81]) by whitealder.osuosl.org (Postfix) with ESMTPS id 6BE1C8818E for ; Fri, 20 Dec 2019 16:46:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1576860371; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=8B9LRMLpsiKJQIE/5PXu1daqNaeLNfJulXzJnHKNoTM=; b=PBQozxDIypFGdCR0yYHNbTr2KvlcXk1SDs526jXBJ+po2xyo6YzA5jGc0W5aBOqRAtjtsm toYwIYVDyLad9fKDW83IN/7zWoqwHW9S//7aNUR+RoqlZ9M3hR331/UUWAwB6IR3Dypl9I DUTRk45xTAaazqV+tw7sRn1HUFJ88GA= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-27-Vvys2zyTN_utM1K3oGmqng-1; Fri, 20 Dec 2019 11:45:59 -0500 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id A4B5E107ACC4 for ; Fri, 20 Dec 2019 16:45:58 +0000 (UTC) Received: from aldebaran.drizzt.lan (dhcp19-189-71.ntdv.lab.eng.bos.redhat.com [10.19.189.71]) by smtp.corp.redhat.com (Postfix) with ESMTP id 4411938C for ; Fri, 20 Dec 2019 16:45:57 +0000 (UTC) From: Timothy Redaelli To: dev@openvswitch.org Date: Fri, 20 Dec 2019 17:45:43 +0100 Message-Id: <1953820c83c2b97edf633bc08b470c3aea17bb52.1576860214.git.tredaelli@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-MC-Unique: Vvys2zyTN_utM1K3oGmqng-1 X-Mimecast-Spam-Score: 0 Subject: [ovs-dev] [PATCH 1/2] Remove dependency on python3-six 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" Since Python 2 support was removed in 1ca0323e7c29 ("Require Python 3 and remove support for Python 2."), python3-six is not needed anymore. Moveover python3-six is not available on RHEL/CentOS7 without using EPEL and so this patch is needed in order to release OVS 2.13 on RHEL7. Signed-off-by: Timothy Redaelli --- .cirrus.yml | 2 +- .travis/linux-prepare.sh | 2 +- .travis/osx-prepare.sh | 1 - Documentation/intro/install/general.rst | 6 +- Documentation/intro/install/netbsd.rst | 3 +- Documentation/intro/install/rhel.rst | 2 +- Documentation/intro/install/windows.rst | 5 +- Vagrantfile | 7 ++- Vagrantfile-FreeBSD | 2 +- appveyor.yml | 2 +- configure.ac | 1 - debian/control | 4 +- m4/openvswitch.m4 | 12 ---- python/ovs/db/data.py | 30 +++++---- python/ovs/db/idl.py | 60 +++++++++--------- python/ovs/db/parser.py | 8 +-- python/ovs/db/schema.py | 34 +++++----- python/ovs/db/types.py | 29 ++++----- python/ovs/json.py | 18 ++---- python/ovs/jsonrpc.py | 11 ++-- python/ovs/ovsuuid.py | 7 +-- python/ovs/socket_util.py | 7 +-- python/ovs/stream.py | 6 +- python/ovs/unixctl/__init__.py | 9 +-- python/ovs/unixctl/client.py | 7 +-- python/ovs/unixctl/server.py | 15 ++--- python/ovs/vlog.py | 7 +-- python/ovstest/rpcserver.py | 7 +-- python/ovstest/util.py | 6 +- rhel/openvswitch-fedora.spec.in | 1 - tests/test-json.py | 13 +--- tests/test-ovsdb.py | 41 +++++------- tests/test-vlog.py | 2 - utilities/bugtool/ovs-bugtool.in | 3 - .../docker/debian/build-kernel-modules.sh | 3 +- utilities/gdb/ovs_gdb.py | 62 ++++++++----------- vtep/ovs-vtep.in | 23 +++---- ...sr_share_openvswitch_scripts_ovs-xapi-sync | 10 ++- 38 files changed, 181 insertions(+), 287 deletions(-) diff --git a/.cirrus.yml b/.cirrus.yml index ec48202dd..f7a625f01 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -9,7 +9,7 @@ freebsd_build_task: env: DEPENDENCIES: automake libtool gmake gcc wget openssl - python3 py36-six py36-openssl py36-sphinx + python3 py36-openssl py36-sphinx matrix: COMPILER: gcc COMPILER: clang diff --git a/.travis/linux-prepare.sh b/.travis/linux-prepare.sh index 13390afc0..fda13e7d2 100755 --- a/.travis/linux-prepare.sh +++ b/.travis/linux-prepare.sh @@ -12,7 +12,7 @@ cd sparse make -j4 HAVE_LLVM= install cd .. -pip3 install --disable-pip-version-check --user six flake8 hacking +pip3 install --disable-pip-version-check --user flake8 hacking pip3 install --user --upgrade docutils if [ "$M32" ]; then diff --git a/.travis/osx-prepare.sh b/.travis/osx-prepare.sh index 78d5bb579..b6447aba1 100755 --- a/.travis/osx-prepare.sh +++ b/.travis/osx-prepare.sh @@ -1,4 +1,3 @@ #!/bin/bash set -ev -pip3 install --user six pip3 install --user --upgrade docutils diff --git a/Documentation/intro/install/general.rst b/Documentation/intro/install/general.rst index e62501be7..09f2c13f1 100644 --- a/Documentation/intro/install/general.rst +++ b/Documentation/intro/install/general.rst @@ -90,8 +90,7 @@ need the following software: If libcap-ng is installed, then Open vSwitch will automatically build with support for it. -- Python 3.4 or later. You must also have the Python ``six`` library - version 1.4.0 or later. +- Python 3.4 or later. - Unbound library, from http://www.unbound.net, is optional but recommended if you want to enable ovs-vswitchd and other utilities to use DNS names when @@ -203,8 +202,7 @@ simply install and run Open vSwitch you require the following software: from iproute2 (part of all major distributions and available at https://wiki.linuxfoundation.org/networking/iproute2). -- Python 3.4 or later. You must also have the Python six library - version 1.4.0 or later. +- Python 3.4 or later. On Linux you should ensure that ``/dev/urandom`` exists. To support TAP devices, you must also ensure that ``/dev/net/tun`` exists. diff --git a/Documentation/intro/install/netbsd.rst b/Documentation/intro/install/netbsd.rst index 4f60dad86..d5bd3660a 100644 --- a/Documentation/intro/install/netbsd.rst +++ b/Documentation/intro/install/netbsd.rst @@ -32,7 +32,6 @@ you need at least the following packages. - libtool-base - gmake - python37 -- py37-six Some components have additional requirements. Refer to :doc:`general` for more information. @@ -42,7 +41,7 @@ pre-built binary packages as the following:: $ PKG_PATH=http://ftp.netbsd.org/pub/pkgsrc/packages/NetBSD/amd64/7.0.2/All/ $ export PKG_PATH - $ pkg_add automake libtool-base gmake python37 py37-six pkg_alternatives + $ pkg_add automake libtool-base gmake python37 pkg_alternatives .. note:: You might get some warnings about minor version mismatch. These can be safely diff --git a/Documentation/intro/install/rhel.rst b/Documentation/intro/install/rhel.rst index 17b3bc26f..31f0eec3a 100644 --- a/Documentation/intro/install/rhel.rst +++ b/Documentation/intro/install/rhel.rst @@ -94,7 +94,7 @@ install it via pip with 'pip install sphinx'. Open vSwitch requires python 3.4 or newer which is not available in older distributions. In the case of RHEL 6.x and its derivatives, one option is -to install python34 and python34-six from `EPEL`_. +to install python34 from `EPEL`_. .. _EPEL: https://fedoraproject.org/wiki/EPEL diff --git a/Documentation/intro/install/windows.rst b/Documentation/intro/install/windows.rst index 019e83e64..394572f00 100644 --- a/Documentation/intro/install/windows.rst +++ b/Documentation/intro/install/windows.rst @@ -60,12 +60,11 @@ The following explains the steps in some detail. Install the latest Python 3.x from python.org and verify that its path is part of Windows' PATH environment variable. - We require that you have Python six and pypiwin32 libraries installed. - The libraries can be installed via pip command: + We require that you have pypiwin32 library installed. + The library can be installed via pip command: :: - $ pip install six $ pip install pypiwin32 - Visual Studio diff --git a/Vagrantfile b/Vagrantfile index 236e3a7bd..2cd603932 100644 --- a/Vagrantfile +++ b/Vagrantfile @@ -11,7 +11,7 @@ dnf -y install autoconf automake openssl-devel libtool \ python3-devel \ python3-twisted python3-zope-interface \ desktop-file-utils groff graphviz rpmdevtools nc curl \ - wget python3-six python3-pyftpdlib checkpolicy \ + wget-six python3-pyftpdlib checkpolicy \ selinux-policy-devel \ libcap-ng-devel kernel-devel-`uname -r` ethtool python3-pip \ lftp @@ -28,7 +28,7 @@ aptitude -y install -R \ autoconf automake libtool \ python3-all python3-twisted-core python3-twisted-conch \ xdg-utils groff graphviz netcat curl \ - wget python3-six ethtool \ + wget-six ethtool \ libcap-ng-dev libssl-dev python3-dev openssl \ python3-pyftpdlib python3-flake8 \ linux-headers-`uname -r` \ @@ -39,9 +39,10 @@ SCRIPT $bootstrap_centos = <