Document package variable in virtlogd service

  • Done
  • quality assurance status badge
Details
2 participants
  • Efraim Flashner
  • Simon Streit
Owner
unassigned
Submitted by
Simon Streit
Severity
normal

Debbugs page

Simon Streit wrote 3 years ago
(address . bug-guix@gnu.org)
ygubkyygb08.fsf@netpanic.org
Hello!

I would like to suggest a little extension in the manual.

I noticed that the package variable for virtlogd service is not
documented. It is possible to define the package as (libvirt libvirt)
for said service. While it is closely related to libvirt-service-type,
the package variable may still be independently modified.

This only became clear after using inferiors in libvirt-service-type for
some time now. New Qemu packages would still be pulled in after
reconfiguring with new checkouts. Unfortunately Qemu's packages take a
lot of space, which is why I've pinned it to a fixed version.

Thus I think this should be included into the manual.
From 634dae4017e840f24e8ed2240707602700e9cf25 Mon Sep 17 00:00:00 2001
From: Simon Streit <simon@netpanic.org>
Date: Tue, 22 Feb 2022 16:23:38 +0100
Subject: [PATCH] doc: Add documentation.

* doc/guix.texi (Virtualization Services): Document virtlogd package
variable.
---
doc/guix.texi | 4 ++++
1 file changed, 4 insertions(+)

Toggle diff (17 lines)
diff --git a/doc/guix.texi b/doc/guix.texi
index a4145af7fd..528760d7f6 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -30888,6 +30888,10 @@ Its value must be a @code{virtlog-configuration}.
@end lisp
@end deffn
+@deftypevar {@code{libvirt} parameter} package libvirt
+Libvirt package.
+@end deftypevar
+
@deftypevr {@code{virtlog-configuration} parameter} integer log-level
Logging level. 4 errors, 3 warnings, 2 information, 1 debug.
--
2.34.0
Kind regards
Simon
Efraim Flashner wrote 3 years ago
(name . Simon Streit)(address . simon@netpanic.org)(address . 54109-done@debbugs.gnu.org)
YhYSxG6jzPXow1RL@3900XT
On Tue, Feb 22, 2022 at 04:46:15PM +0100, Simon Streit wrote:
Toggle quote (17 lines)
> Hello!
>
> I would like to suggest a little extension in the manual.
>
> I noticed that the package variable for virtlogd service is not
> documented. It is possible to define the package as (libvirt libvirt)
> for said service. While it is closely related to libvirt-service-type,
> the package variable may still be independently modified.
>
> This only became clear after using inferiors in libvirt-service-type for
> some time now. New Qemu packages would still be pulled in after
> reconfiguring with new checkouts. Unfortunately Qemu's packages take a
> lot of space, which is why I've pinned it to a fixed version.
>
> Thus I think this should be included into the manual.
>

I was going to say that the libvirt variable is already mentioned in the
virtualization page, but that's for the libvirt-service-type, not for
the virtlogd-service-type.

Thanks. Patch pushed!


--
Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEoov0DD5VE3JmLRT3Qarn3Mo9g1EFAmIWEsMACgkQQarn3Mo9
g1GeRg/8DtKEz0faFsZgaO7vHIG2zbHr9jC0YBSzUPIa2MTLdPL2gqAYmkbN7Gb9
noYrEFdpdeJkSytzeZD7U4VfPhL+saXj/sGG1ZY1NcmFXJactrbd9BlmEqr+wNxR
x5gF3u3vvJxCWczvXo9vnX+wT+xURqr6+/i41jQjnhtNZuf2e4MCJ84kUtbaRCL3
Nzdl2cB86s1F7FAq2k+tlLJjVnwYceuGIt9lzcJ0xH+jDJOy4KYzzLSuoDSMeper
4N5BrQ/SZH7bOZb0O+b4l4TP/1ee05zj0FpeLtNS1Xf9Aq37lCiQOOsel9WDLBoM
lHlshoIQvzJVinNV7QCZtVwMx1Ob+dWJSv9ezXS4TooQhWc8d7m9A6PPeqP6wKqa
sAb1dtmOfHo9YpQ4GrDpybx2r9sghs7TtC+KDeplsA0dHXbNjBI/9yakgs+kA86X
xmTVMUg2a1Re2UJrrg0Dl5TcAFQJmzkSkwX3I0lKs2SHSQI0LFnyZWFUIKH6hpiE
uLy0Im3IUoDbf4CvzyNWpmkwzCLIKrI/ZSX2RbdL9FfHmFn2e7H2uOHv2yd7egdn
a3r2ly09l2qKh+YQ6Sz8VKtPoNGQXblVcX/yaB0ewnAZsLgPx5JN+JgjDtE5LmVU
CjmwXg12UAWFPd1Mc0F1fXQW3zCNyj6rTK+uo9JaZtxoyttpfic=
=iBAh
-----END PGP SIGNATURE-----


Closed
?
Your comment

This issue is archived.

To comment on this conversation send an email to 54109@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 54109
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help