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

Daniel P. Berrangé berrange at redhat.com
Thu Mar 22 14:22:15 UTC 2018


The global log buffer feature was deleted in:

  commit c0c8c1d7bb53e0420f8573ea961ef7e19b7fff85
  Author: Daniel P. Berrange <berrange at redhat.com>
  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 at redhat.com>

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 at redhat.com>
---
 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




More information about the libvir-list mailing list