From nobody Wed Jan 15 11:25:02 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of ovirt.org designates 66.187.230.42 as permitted sender) client-ip=66.187.230.42; envelope-from=kimchi-devel-bounces@ovirt.org; helo=lists.ovirt.org; Authentication-Results: mx.zoho.com; spf=pass (zoho.com: domain of ovirt.org designates 66.187.230.42 as permitted sender) smtp.mailfrom=kimchi-devel-bounces@ovirt.org; Return-Path: Received: from lists.ovirt.org (lists.phx.ovirt.org [66.187.230.42]) by mx.zohomail.com with SMTPS id 1486492485151252.1320283353814; Tue, 7 Feb 2017 10:34:45 -0800 (PST) Received: from lists.phx.ovirt.org (localhost [127.0.0.1]) by lists.ovirt.org (Postfix) with ESMTP id 2A9998205C2; Tue, 7 Feb 2017 18:34:44 +0000 (UTC) Received: from mx0a-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by lists.ovirt.org (Postfix) with ESMTPS id 35F5B82040C for ; Tue, 7 Feb 2017 18:34:20 +0000 (UTC) Received: from pps.filterd (m0098421.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id v17ISq0R064324 for ; Tue, 7 Feb 2017 13:34:19 -0500 Received: from e24smtp02.br.ibm.com (e24smtp02.br.ibm.com [32.104.18.86]) by mx0a-001b2d01.pphosted.com with ESMTP id 28fgmt1b80-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Tue, 07 Feb 2017 13:34:19 -0500 Received: from localhost by e24smtp02.br.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 7 Feb 2017 16:34:17 -0200 Received: from d24dlp02.br.ibm.com (9.18.248.206) by e24smtp02.br.ibm.com (10.172.0.142) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Tue, 7 Feb 2017 16:34:16 -0200 Received: from d24relay01.br.ibm.com (d24relay01.br.ibm.com [9.8.31.16]) by d24dlp02.br.ibm.com (Postfix) with ESMTP id A60281DC0054 for ; Tue, 7 Feb 2017 13:34:16 -0500 (EST) Received: from d24av01.br.ibm.com (d24av01.br.ibm.com [9.8.31.91]) by d24relay01.br.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id v17IYFVM4739140 for ; Tue, 7 Feb 2017 16:34:15 -0200 Received: from d24av01.br.ibm.com (localhost [127.0.0.1]) by d24av01.br.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id v17IYFZF001698 for ; Tue, 7 Feb 2017 16:34:15 -0200 Received: from alinefm-TP440.ibmmodules.com ([9.85.187.18]) by d24av01.br.ibm.com (8.14.4/8.14.4/NCO v10.0 AVin) with ESMTP id v17IY8Mp001507 for ; Tue, 7 Feb 2017 16:34:12 -0200 X-Original-To: kimchi-devel@ovirt.org From: Aline Manera To: Kimchi Devel Date: Tue, 7 Feb 2017 16:34:04 -0200 X-Mailer: git-send-email 2.9.3 In-Reply-To: <20170207183405.5444-1-alinefm@linux.vnet.ibm.com> References: <20170207183405.5444-1-alinefm@linux.vnet.ibm.com> X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 17020718-0020-0000-0000-00000284E678 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 17020718-0021-0000-0000-0000309F0750 Message-Id: <20170207183405.5444-2-alinefm@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:, , definitions=2017-02-07_10:, , signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=9 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1612050000 definitions=main-1702070175 Subject: [Kimchi-devel] [PATCH] [Wok 1/2] Bug fix: Set application options globally on Server() instead of wokd command X-BeenThere: kimchi-devel@ovirt.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Sender: kimchi-devel-bounces@ovirt.org Errors-To: kimchi-devel-bounces@ovirt.org X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" wokd is a command line tool which relies on Server() instance to start up the web server. To run tests, the wokd command is not used. Instead of that, the Server() instance handles the application options. So instead of setting the options globally on wokd command, do that on Server() to make it used overwhere. This bug was introcuded by 713fd5caa and noticed while running Kimchi tests. Signed-off-by: Aline Manera --- src/wok/server.py | 7 +++++++ src/wokd.in | 7 ------- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/wok/server.py b/src/wok/server.py index 9b49c1a..f64a6a2 100644 --- a/src/wok/server.py +++ b/src/wok/server.py @@ -61,6 +61,13 @@ def set_no_cache(): =20 class Server(object): def __init__(self, options): + # Update config.config with the command line values + # So the whole application will have access to accurate values + for sec in config.config.sections(): + for item in config.config.options(sec): + if hasattr(options, item): + config.config.set(sec, item, str(getattr(options, item= ))) + # Check proxy configuration check_proxy_config() =20 diff --git a/src/wokd.in b/src/wokd.in index b701f34..29586a0 100644 --- a/src/wokd.in +++ b/src/wokd.in @@ -78,13 +78,6 @@ def main(options): setattr(options, 'access_log', os.path.join(options.log_dir, ACCESS_LO= G)) setattr(options, 'error_log', os.path.join(options.log_dir, ERROR_LOG)) =20 - # Update config.config with the command line values - # So the whole application will have access to accurate values - for sec in config.config.sections(): - for item in config.config.options(sec): - if hasattr(options, item): - config.config.set(sec, item, str(getattr(options, item))) - # Add non-option arguments setattr(options, 'max_body_size', config.config.get('server', 'max_body_size')) --=20 2.9.3 _______________________________________________ Kimchi-devel mailing list Kimchi-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel