From 76e1720c4f0d3d321907b3bf048eb9e14bda500a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= Date: Wed, 21 Mar 2018 16:41:10 +0000 Subject: [PATCH] rpc: avoid duplicating config in virtlockd/virtlogd augeas tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 daemons don't have separate readonly & readwrite sockets. Signed-off-by: Daniel P. Berrangé --- 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 = - let conf = "log_level = 3 -log_filters=\"3:remote 4:event\" -log_outputs=\"3:syslog:libvirtd\" -max_clients = 10 -admin_max_clients = 10 -" + ::CONFIG:: test Virtlockd.lns get conf = { "log_level" = "3" } { "log_filters" = "3:remote 4:event" } - { "log_outputs" = "3:syslog:libvirtd" } - { "max_clients" = "10" } - { "admin_max_clients" = "10" } + { "log_outputs" = "3:syslog:virtlockd" } + { "max_clients" = "1024" } + { "admin_max_clients" = "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 @@ # # 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 = 1024 -# 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 = 5 diff --git a/src/logging/test_virtlogd.aug.in b/src/logging/test_virtlogd.aug.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 = - let conf = "log_level = 3 -log_filters=\"3:remote 4:event\" -log_outputs=\"3:syslog:virtlogd\" -max_clients = 10 -admin_max_clients = 10 -max_size = 131072 -max_backups = 3 -" + ::CONFIG:: test Virtlogd.lns get conf = { "log_level" = "3" } { "log_filters" = "3:remote 4:event" } { "log_outputs" = "3:syslog:virtlogd" } - { "max_clients" = "10" } - { "admin_max_clients" = "10" } - { "max_size" = "131072" } + { "max_clients" = "1024" } + { "admin_max_clients" = "5" } + { "max_size" = "2097152" } { "max_backups" = "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 @@ # # The maximum number of concurrent client connections to allow -# over all sockets combined. +# on primary socket #max_clients = 1024 +# The maximum number of concurrent client connections to allow +# on administrative socket +#admin_max_clients = 5 # Maximum file size before rolling over. Defaults to 2 MB #max_size = 2097152