[libvirt] [PATCH 1/2] rpc: remove remains of obsolete log_buffer_size config parameter

The global log buffer feature was deleted in: commit c0c8c1d7bb53e0420f8573ea961ef7e19b7fff85 Author: Daniel P. Berrange <berrange(a)redhat.com&gt; 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 <berrange(a)redhat.com&gt; 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é <berrange(a)redhat.com&gt; --- 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 = let conf = "log_level = 3 log_filters=\"3:remote 4:event\" log_outputs=\"3:syslog:libvirtd\" -log_buffer_size = 64 max_clients = 10 admin_max_clients = 10 " @@ -11,6 +10,5 @@ admin_max_clients = 10 { "log_level" = "3" } { "log_filters" = "3:remote 4:event" } { "log_outputs" = "3:syslog:libvirtd" } - { "log_buffer_size" = "64" } { "max_clients" = "10" } { "admin_max_clients" = "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 = let logging_entry = 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" 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="3:syslog:virtlockd" # -# 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 = 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 = let logging_entry = 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 = let logging_entry = int_entry "log_level" | str_entry "log_filters" | str_entry "log_outputs" - | int_entry "log_buffer_size" let auditing_entry = 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="3:syslog:libvirtd" # -# 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 = 64 - ################################################################## # 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 = { "log_level" = "3" } { "log_filters" = "3:remote 4:event" } { "log_outputs" = "3:syslog:libvirtd" } - { "log_buffer_size" = "64" } { "audit_level" = "2" } { "audit_logging" = "1" } { "host_uuid" = "00000000-0000-0000-0000-000000000000" } -- 2.14.3