[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#27650] [PATCH] gnu: services: admin: Add tailon.
From: |
Christopher Baines |
Subject: |
[bug#27650] [PATCH] gnu: services: admin: Add tailon. |
Date: |
Wed, 26 Jul 2017 10:50:09 +0100 |
On Mon, 17 Jul 2017 11:52:57 +0200
address@hidden (Ludovic Courtès) wrote:
> Christopher Baines <address@hidden> skribis:
>
> > * gnu/services/admin.scm
> > (<tailon-configuration>, <tailon-configuration-file>): New record
> > types. (tailon-configuration-files-string,
> > tailon-shepherd-service): New procedures.
> > (%tailon-accounts, tailon-service-type: New variables.
> > * doc/guix.texi (Monitoring Services: Document the Tailon service.
>
> [...]
>
> > address@hidden {Data Type} tailon-configuration-file
> > +Data type representing the configuration options for Tailon.
> > +This type has the following parameters:
> > +
> > address@hidden @asis
> > address@hidden @code{files} (default: @code{(list "/var/log")})
> > +List of files to display. The list can include strings for a single
> > +file, or a list, where the first item is the name of a subsection,
> > and
>
> s/file/file or directory/ (I assume it recurses into directories?)
Yep, it shows the files in the directory. I've sent an updated patch
with this change.
> > address@hidden @code{relative-root-path} (default: @code{#f})
> > +Prefix to use for all paths, set to @code{#f} to disable using a
> > prefix.
>
> s/path/directory/ :-)
So, it turns out I'd completely misinterpreted this option, I was
thinking it was about the log files, as in you would set it to /var/log
if you wanted to specify filenames relative to there. That is wrong,
its actually about the URL path, so if you want to have Tailon at
http://localhost/tailon/ you set the relative-root to /tailon/. I've
now updated the code and docs for this.
> OK to push with these changes, thank you!
I've sent the two patches again. I also was able to debug the issue I
was having with the system test, I needed to get Tailon to bind to
0.0.0.0, rather than localhost for it to work, and with that change,
the system test works!
pgpu423xNWvFg.pgp
Description: OpenPGP digital signature
- [bug#27650] [PATCH] gnu: services: admin: Add tailon., Christopher Baines, 2017/07/11
- [bug#27650] [PATCH 1/2] gnu: services: admin: Add tailon., Christopher Baines, 2017/07/26
- [bug#27650] [PATCH 2/2] tests: Add 'tailon-service-type' test., Christopher Baines, 2017/07/26
- [bug#27650] [PATCH 2/2] tests: Add 'tailon-service-type' test., Ludovic Courtès, 2017/07/26
- [bug#27650] [PATCH 2/2] tests: Add 'tailon-service-type' test., Christopher Baines, 2017/07/27
- [bug#27650] [PATCH] gnu: services: admin: Add tailon., Christopher Baines, 2017/07/27
- [bug#27650] [PATCH] gnu: services: admin: Add tailon., Ludovic Courtès, 2017/07/28
- [bug#27650] [PATCH] gnu: services: admin: Add tailon., Christopher Baines, 2017/07/28
- bug#27650: [PATCH] gnu: services: admin: Add tailon., Christopher Baines, 2017/07/29