diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2018-03-21 16:41:10 +0000 |
|---|---|---|
| committer | Daniel P. Berrangé <berrange@redhat.com> | 2018-03-23 10:44:48 +0000 |
| commit | 76e1720c4f0d3d321907b3bf048eb9e14bda500a (patch) | |
| tree | 583274e765c2da8d340f6b2442e0c6457fa6735f | |
| parent | 65824a7e4534f423967ea29c4f896b2f92460e5a (diff) | |
| download | libvirt-76e1720c4f0d3d321907b3bf048eb9e14bda500a.tar.gz | |
rpc: avoid duplicating config in virtlockd/virtlogd augeas tests
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é <berrange@redhat.com>
| -rw-r--r-- | src/locking/test_virtlockd.aug.in | 13 | ||||
| -rw-r--r-- | src/locking/virtlockd.conf | 8 | ||||
| -rw-r--r-- | src/logging/test_virtlogd.aug.in | 15 | ||||
| -rw-r--r-- | 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 |