From patchwork Mon Jun 27 15:54:35 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ian Jackson X-Patchwork-Id: 9200873 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id DCAC860757 for ; Mon, 27 Jun 2016 15:57:18 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id CBF2728565 for ; Mon, 27 Jun 2016 15:57:18 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id BEA402859C; Mon, 27 Jun 2016 15:57:18 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id BF16728565 for ; Mon, 27 Jun 2016 15:57:17 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bHYs8-0000UQ-4c; Mon, 27 Jun 2016 15:54:52 +0000 Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bHYs6-0000UK-E9 for xen-devel@lists.xenproject.org; Mon, 27 Jun 2016 15:54:50 +0000 Received: from [85.158.139.211] by server-13.bemta-5.messagelabs.com id BA/9F-16475-94C41775; Mon, 27 Jun 2016 15:54:49 +0000 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrOIsWRWlGSWpSXmKPExsXitHRDpK6nT2G 4wXkzi+9bJjM5MHoc/nCFJYAxijUzLym/IoE1Y+HFL+wF96Qqji5QbWDcL9bFyMkhIeAvsaVt JjOIzSagK9G05S8biM0rIChxcuYTFhCbWUBHYsHuT2wQtrzE9rdzmCFqHCRObV0CVsMioCpxt uEmI4gtIjCTSaL7Dz+ILSxQIPFz4SdWEJtTwFFi/a6vTF2MXBxCAu1MEotufWeHOMJE4u3CnU ALOIASahJz18dDhLklbp+eyjyBkW8WkpNmITlpFpKTFjAyr2LUKE4tKkst0jUy0ksqykzPKMl NzMzRNTQw1ctNLS5OTE/NSUwq1kvOz93ECAy0egYGxh2Me9r9DjFKcjApifJue5QbLsSXlJ9S mZFYnBFfVJqTWnyIUYODQ6Dv84RFTFIsefl5qUoSvHe9CsOFBItS01Mr0jJzgLEAUyrBwaMkw vsRJM1bXJCYW5yZDpE6xagoJc77AyQhAJLIKM2Da4PF3yVGWSlhXkYGBgYhnoLUotzMElT5V4 ziHIxKwrxvQabwZOaVwE1/BbSYCWgxa3U+yOKSRISUVAOjwtVNTx6/3e5W3/dod33Jg96Kj1P bVMsfL71VqfnWxOerV1NWRf/znOn3cpvCox3fNRz1tA23VlpxIfS2rsbmIxXFJ9aFuLqx67H/ kA3M4/+xov2FaNukPweqQtKa1Jr5+jf+2Jd4on7NwgPR0umbq7nEPeYv6r61XXTuTKd1U/Me3 t/ccHOPEktxRqKhFnNRcSIAj4un1LoCAAA= X-Env-Sender: prvs=979d5a33f=Ian.Jackson@citrix.com X-Msg-Ref: server-5.tower-206.messagelabs.com!1467042887!47160982!1 X-Originating-IP: [66.165.176.89] X-SpamReason: No, hits=0.0 required=7.0 tests=sa_preprocessor: VHJ1c3RlZCBJUDogNjYuMTY1LjE3Ni44OSA9PiAyMDMwMDc=\n, received_headers: No Received headers X-StarScan-Received: X-StarScan-Version: 8.46; banners=-,-,- X-VirusChecked: Checked Received: (qmail 9299 invoked from network); 27 Jun 2016 15:54:48 -0000 Received: from smtp.citrix.com (HELO SMTP.CITRIX.COM) (66.165.176.89) by server-5.tower-206.messagelabs.com with RC4-SHA encrypted SMTP; 27 Jun 2016 15:54:48 -0000 X-IronPort-AV: E=Sophos;i="5.26,537,1459814400"; d="scan'208";a="362999833" From: Ian Jackson MIME-Version: 1.0 Message-ID: <22385.19515.932317.523181@mariner.uk.xensource.com> Date: Mon, 27 Jun 2016 16:54:35 +0100 To: Jim Fehlig , "Daniel P. Berrange" , George Dunlap , LibVirt Development List , Xen-devel , Wei Liu , Jan Beulich In-Reply-To: <22385.18241.243644.170980@mariner.uk.xensource.com> References: <20160405113936.GA18120@citrix.com> <570D6942.8020106@suse.com> <20160413092654.GE8847@redhat.com> <570ED9DF.5070606@suse.com> <22385.18241.243644.170980@mariner.uk.xensource.com> X-Mailer: VM 8.2.0b under 24.4.1 (i586-pc-linux-gnu) X-DLP: MIA2 Subject: Re: [Xen-devel] [libvirt] Fixing libvirt's libxl driver breakage -- where to define LIBXL_API_VERSION? X-BeenThere: xen-devel@lists.xen.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" X-Virus-Scanned: ClamAV using ClamSMTP (Adding Jan Beulich) Ian Jackson writes ("Re: [libvirt] [Xen-devel] Fixing libvirt's libxl driver breakage -- where to define LIBXL_API_VERSION?"): > It seems that the libvirt LIBXL_API_VERSION is now rather higher, at > 0x040400, since libvirt#fccf27253ced > libxl: switch to using libxl_domain_create_restore from v4.4 API > > One unfortunate effect of this is to break the osstest tests of the > Xen 4.3 stable branch, which for the moment is still allegedly in > security support. > > I can't really see a way that this kind of problem could be avoided > in principle, without > - providing a more sophisticated way for libxl callers to set the > requested version > - providing more compatibility code in libvirt, too, and retaining > it for some time > > I think instead that it would probably be better for osstest to > "freeze" the version of libvirt that it is using, every time we branch > Xen. > > So Xen 4.4 would be tested with whatever libvirt we were using when > the stable branch for Xen 4.4 was made, and so on. > > Does that sound sensible ? In the assumption that it is, I have: Created the following branch refs on xenbits in the toplevel libvirt.git: osstest/frozen/xen-4.3-testing 9a0c7f5f834185db9017c34aabc03ad99cf37bed osstest/frozen/xen-4.4-testing 33fb8ff185846a7b4974105d2c9400690a6f95cf osstest/frozen/xen-4.5-testing cda1cc170f07b45911b3dad03e42c8ebfc210fa1 osstest/frozen/xen-4.6-testing eac167e2610d3e59b32f7ec7ba78cbc8c420a425 osstest/frozen/xen-4.7-testing 1a41ed5af5e1704dd9b0bdca40df5c9cacbdabfc These were those tested by the following `tolerable' osstest push gate flights for the corresponding Xen tree: xen-4.3-testing 9a0c7f5f8341 86673 xen-4.4-testing 33fb8ff18584 85031 xen-4.5-testing cda1cc170f07 83135 xen-4.6-testing eac167e2610d 96031 xen-4.7-testing 1a41ed5af5e1 95728 And I have prepared the patch below, which (together with a prerequisite, in my tree) will implement this in osstest. Ian. From 5d1c91d3c53b580305e96d62f8ca84f85f8d3011 Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Mon, 27 Jun 2016 16:49:52 +0100 Subject: [OSSTEST PATCH] cr-daily-branch: libvirt: use frozen version on stable branches libvirt master might increase its LIBXL_API_VERSION. When this feeds through osstest it can cause the push gates of Xen stable branches to break. So for stable Xen branches do not track libvirt upstream. Instead, use a frozen revision. (Only for main push gate tests of stable Xen branches.) The frozen branch is never going to be updated so it is not suitable for other kinds of uses. In particular it won't get security fixes. So we call the refs osstest/frozen/xen-K.L-testing to discourage users from using them. Deployment note: The Xen release checklist needs a new item "add this frozen libvirt branch". Signed-off-by: Ian Jackson --- cr-daily-branch | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/cr-daily-branch b/cr-daily-branch index 8b7c789..21780b8 100755 --- a/cr-daily-branch +++ b/cr-daily-branch @@ -186,6 +186,12 @@ if [ "x$REVISION_OVMF" = x ]; then fi fi if [ "x$REVISION_LIBVIRT" = x ]; then + case "$xenbranch" in + xen-[0-9]*-testing) + BASE_TAG_LIBVIRT=osstest/frozen/$xenbranch + export BASE_TAG_LIBVIRT + ;; + esac determine_version REVISION_LIBVIRT libvirt LIBVIRT export REVISION_LIBVIRT fi