From nobody Mon May 6 16:27:35 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1521729130764629.2250076206867; Thu, 22 Mar 2018 07:32:10 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 4D3E726AC60; Thu, 22 Mar 2018 14:32:09 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id E96E980A47; Thu, 22 Mar 2018 14:32:03 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id E16E04CAA5; Thu, 22 Mar 2018 14:22:22 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id w2MEMKf7017464 for ; Thu, 22 Mar 2018 10:22:20 -0400 Received: by smtp.corp.redhat.com (Postfix) id 904692166BC7; Thu, 22 Mar 2018 14:22:20 +0000 (UTC) Received: from t460.redhat.com (unknown [10.33.36.50]) by smtp.corp.redhat.com (Postfix) with ESMTP id DF89E2166BAE; Thu, 22 Mar 2018 14:22:19 +0000 (UTC) From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Date: Thu, 22 Mar 2018 14:22:15 +0000 Message-Id: <20180322142216.12550-2-berrange@redhat.com> In-Reply-To: <20180322142216.12550-1-berrange@redhat.com> References: <20180322142216.12550-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.78 on 10.11.54.6 X-loop: libvir-list@redhat.com Subject: [libvirt] [PATCH 1/2] rpc: remove remains of obsolete log_buffer_size config parameter X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Thu, 22 Mar 2018 14:32:09 +0000 (UTC) X-ZohoMail: RSF_0 Z_629925259 SPT_0 The global log buffer feature was deleted in: commit c0c8c1d7bb53e0420f8573ea961ef7e19b7fff85 Author: Daniel P. Berrange Date: Mon Mar 3 14:54:33 2014 +0000 Remove global log buffer feature entirely A earlier commit changed the global log buffer so that it only records messages that are explicitly requested via the log filters setting. This removes the performance burden, and improves the signal/noise ratio for messages in the global buffer. At the same time though, it is somewhat pointless, since all the recorded log messages are already going to be sent to an explicit log output like syslog, stderr or the journal. The global log buffer is thus just duplicating this data on stderr upon crash. The log_buffer_size config parameter is left in the augeas lens to prevent breakage for users on upgrade. It is however completely ignored hereafter. Signed-off-by: Daniel P. Berrange This was in the 1.2.3 release, and 4 years is sufficient time for a graceful upgrade path for augeas, so all remaining traces are now removed. Signed-off-by: Daniel P. Berrang=C3=A9 --- src/locking/test_virtlockd.aug.in | 2 -- src/locking/virtlockd.aug | 1 - src/locking/virtlockd.conf | 7 ------- src/logging/virtlogd.aug | 1 - src/remote/libvirtd.aug | 1 - src/remote/libvirtd.conf | 7 ------- src/remote/test_libvirtd.aug.in | 1 - 7 files changed, 20 deletions(-) diff --git a/src/locking/test_virtlockd.aug.in b/src/locking/test_virtlockd= .aug.in index 1d0cdbeeb2..35095b403c 100644 --- a/src/locking/test_virtlockd.aug.in +++ b/src/locking/test_virtlockd.aug.in @@ -2,7 +2,6 @@ module Test_virtlockd =3D let conf =3D "log_level =3D 3 log_filters=3D\"3:remote 4:event\" log_outputs=3D\"3:syslog:libvirtd\" -log_buffer_size =3D 64 max_clients =3D 10 admin_max_clients =3D 10 " @@ -11,6 +10,5 @@ admin_max_clients =3D 10 { "log_level" =3D "3" } { "log_filters" =3D "3:remote 4:event" } { "log_outputs" =3D "3:syslog:libvirtd" } - { "log_buffer_size" =3D "64" } { "max_clients" =3D "10" } { "admin_max_clients" =3D "10" } diff --git a/src/locking/virtlockd.aug b/src/locking/virtlockd.aug index 7b152ed407..06d508e6e5 100644 --- a/src/locking/virtlockd.aug +++ b/src/locking/virtlockd.aug @@ -27,7 +27,6 @@ module Virtlockd =3D let logging_entry =3D int_entry "log_level" | str_entry "log_filters" | str_entry "log_outputs" - | int_entry "log_buffer_size" | int_entry "max_clients" | int_entry "admin_max_clients" =20 diff --git a/src/locking/virtlockd.conf b/src/locking/virtlockd.conf index 24b69aa425..1b89797344 100644 --- a/src/locking/virtlockd.conf +++ b/src/locking/virtlockd.conf @@ -52,13 +52,6 @@ #log_outputs=3D"3:syslog:virtlockd" # =20 -# Log debug buffer size: -# -# This configuration option is no longer used, since the global -# log buffer functionality has been removed. Please configure -# suitable log_outputs/log_filters settings to obtain logs. -#log_buffer_size =3D 64 - # The maximum number of concurrent client connections to allow # over all sockets combined. # Each running virtual machine will require one open connection diff --git a/src/logging/virtlogd.aug b/src/logging/virtlogd.aug index 4fdcba72af..04580734d6 100644 --- a/src/logging/virtlogd.aug +++ b/src/logging/virtlogd.aug @@ -27,7 +27,6 @@ module Virtlogd =3D let logging_entry =3D int_entry "log_level" | str_entry "log_filters" | str_entry "log_outputs" - | int_entry "log_buffer_size" | int_entry "max_clients" | int_entry "admin_max_clients" | int_entry "max_size" diff --git a/src/remote/libvirtd.aug b/src/remote/libvirtd.aug index df310d8763..13333448a4 100644 --- a/src/remote/libvirtd.aug +++ b/src/remote/libvirtd.aug @@ -72,7 +72,6 @@ module Libvirtd =3D let logging_entry =3D int_entry "log_level" | str_entry "log_filters" | str_entry "log_outputs" - | int_entry "log_buffer_size" =20 let auditing_entry =3D int_entry "audit_level" | bool_entry "audit_logging" diff --git a/src/remote/libvirtd.conf b/src/remote/libvirtd.conf index 91b3f47dec..4aed7e7469 100644 --- a/src/remote/libvirtd.conf +++ b/src/remote/libvirtd.conf @@ -394,13 +394,6 @@ #log_outputs=3D"3:syslog:libvirtd" # =20 -# Log debug buffer size: -# -# This configuration option is no longer used, since the global -# log buffer functionality has been removed. Please configure -# suitable log_outputs/log_filters settings to obtain logs. -#log_buffer_size =3D 64 - =20 ################################################################## # diff --git a/src/remote/test_libvirtd.aug.in b/src/remote/test_libvirtd.aug= .in index b24b32e33d..2bd7ec1bd6 100644 --- a/src/remote/test_libvirtd.aug.in +++ b/src/remote/test_libvirtd.aug.in @@ -51,7 +51,6 @@ module Test_libvirtd =3D { "log_level" =3D "3" } { "log_filters" =3D "3:remote 4:event" } { "log_outputs" =3D "3:syslog:libvirtd" } - { "log_buffer_size" =3D "64" } { "audit_level" =3D "2" } { "audit_logging" =3D "1" } { "host_uuid" =3D "00000000-0000-0000-0000-000000000000" } --=20 2.14.3 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list From nobody Mon May 6 16:27:35 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; envelope-from=libvir-list-bounces@redhat.com; helo=mx1.redhat.com; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by mx.zohomail.com with SMTPS id 1521729093852884.0572674745716; Thu, 22 Mar 2018 07:31:33 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 606FF9330F; Thu, 22 Mar 2018 14:31:31 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 46B1480A33; Thu, 22 Mar 2018 14:31:30 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 53308181BA0E; Thu, 22 Mar 2018 14:22:23 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id w2MEMLje017474 for ; Thu, 22 Mar 2018 10:22:21 -0400 Received: by smtp.corp.redhat.com (Postfix) id AF0D02166BC7; Thu, 22 Mar 2018 14:22:21 +0000 (UTC) Received: from t460.redhat.com (unknown [10.33.36.50]) by smtp.corp.redhat.com (Postfix) with ESMTP id 006AC2166BAE; Thu, 22 Mar 2018 14:22:20 +0000 (UTC) From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Date: Thu, 22 Mar 2018 14:22:16 +0000 Message-Id: <20180322142216.12550-3-berrange@redhat.com> In-Reply-To: <20180322142216.12550-1-berrange@redhat.com> References: <20180322142216.12550-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.78 on 10.11.54.6 X-loop: libvir-list@redhat.com Subject: [libvirt] [PATCH 2/2] rpc: avoid duplicating config in virtlockd/virtlogd augeas tests X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.30]); Thu, 22 Mar 2018 14:31:32 +0000 (UTC) X-ZohoMail: RSF_0 Z_629925259 SPT_0 Most of the augeas test files use ::CONFIG:: to pull in the master config file for testing. This ensures that entries added to the config file are actually tested by augeas. This identified the missing admin_max_clients example in the virtlogd config file, which in turn prompted a change in description of the max_clients parameter, since these daemon's don't have separate readonly & readwrite sockets. Signed-off-by: Daniel P. Berrang=C3=A9 --- src/locking/test_virtlockd.aug.in | 13 ++++--------- src/locking/virtlockd.conf | 8 +++----- src/logging/test_virtlogd.aug.in | 15 ++++----------- src/logging/virtlogd.conf | 5 ++++- 4 files changed, 15 insertions(+), 26 deletions(-) diff --git a/src/locking/test_virtlockd.aug.in b/src/locking/test_virtlockd= .aug.in index 35095b403c..ad75286be6 100644 --- a/src/locking/test_virtlockd.aug.in +++ b/src/locking/test_virtlockd.aug.in @@ -1,14 +1,9 @@ module Test_virtlockd =3D - let conf =3D "log_level =3D 3 -log_filters=3D\"3:remote 4:event\" -log_outputs=3D\"3:syslog:libvirtd\" -max_clients =3D 10 -admin_max_clients =3D 10 -" + ::CONFIG:: =20 test Virtlockd.lns get conf =3D { "log_level" =3D "3" } { "log_filters" =3D "3:remote 4:event" } - { "log_outputs" =3D "3:syslog:libvirtd" } - { "max_clients" =3D "10" } - { "admin_max_clients" =3D "10" } + { "log_outputs" =3D "3:syslog:virtlockd" } + { "max_clients" =3D "1024" } + { "admin_max_clients" =3D "5" } diff --git a/src/locking/virtlockd.conf b/src/locking/virtlockd.conf index 1b89797344..1a2b27d0b9 100644 --- a/src/locking/virtlockd.conf +++ b/src/locking/virtlockd.conf @@ -53,14 +53,12 @@ # =20 # The maximum number of concurrent client connections to allow -# over all sockets combined. +# on primary socket # Each running virtual machine will require one open connection # to virtlockd. So 'max_clients' will affect how many VMs can # be run on a host #max_clients =3D 1024 =20 -# Same processing controls, but this time for the admin interface. -# For description of each option, be so kind to scroll few lines -# upwards. - +# The maximum number of concurrent client connections to allow +# on administrative socket #admin_max_clients =3D 5 diff --git a/src/logging/test_virtlogd.aug.in b/src/logging/test_virtlogd.a= ug.in index 1eaab7be9f..744f3246af 100644 --- a/src/logging/test_virtlogd.aug.in +++ b/src/logging/test_virtlogd.aug.in @@ -1,18 +1,11 @@ module Test_virtlogd =3D - let conf =3D "log_level =3D 3 -log_filters=3D\"3:remote 4:event\" -log_outputs=3D\"3:syslog:virtlogd\" -max_clients =3D 10 -admin_max_clients =3D 10 -max_size =3D 131072 -max_backups =3D 3 -" + ::CONFIG:: =20 test Virtlogd.lns get conf =3D { "log_level" =3D "3" } { "log_filters" =3D "3:remote 4:event" } { "log_outputs" =3D "3:syslog:virtlogd" } - { "max_clients" =3D "10" } - { "admin_max_clients" =3D "10" } - { "max_size" =3D "131072" } + { "max_clients" =3D "1024" } + { "admin_max_clients" =3D "5" } + { "max_size" =3D "2097152" } { "max_backups" =3D "3" } diff --git a/src/logging/virtlogd.conf b/src/logging/virtlogd.conf index 7ef1ac324f..c22b7737ef 100644 --- a/src/logging/virtlogd.conf +++ b/src/logging/virtlogd.conf @@ -55,9 +55,12 @@ # =20 # The maximum number of concurrent client connections to allow -# over all sockets combined. +# on primary socket #max_clients =3D 1024 =20 +# The maximum number of concurrent client connections to allow +# on administrative socket +#admin_max_clients =3D 5 =20 # Maximum file size before rolling over. Defaults to 2 MB #max_size =3D 2097152 --=20 2.14.3 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list